Skip to content

[Camera] Map ports for certification-tool-backend container to listen H.264 packets #576

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: v2.13+summer2025-develop
Choose a base branch
from

Conversation

chulspro
Copy link

Problem

  • For the TestHarness
  • This PR adds expose ports of certification-tool-backend container to listen to H.264 packets produce by camera-controller.
    This is required to forward the H.264 packets to the frontend client.

Change overview

  • Test Harness provides H.264 frames from camera-controller SDK.
  • The camera-controller is opening H.264 packets to the TH.

Testing

  • Excute TestHarness and check WebRTC Requestor/Provider operations
  • Install apk file on Android phone such as Galaxy and Pixel
  • SmartThings App and Hub can commission camera-app and It is possible to test full automation

…packets from camera-controller

Signed-off-by: Sathvik K Gatti <[email protected]>
Signed-off-by: Suyambulingam Rathinasamy Muthupandi <[email protected]>
Signed-off-by: Charles Kim <[email protected]>
@chulspro
Copy link
Author

chulspro commented Apr 23, 2025

@antonio-amjr : Can you review this PR? I'm very thank you for your guidance. And also, Can I add you in the reviewer list? because I couldn't add any review in certification-tool. I want to add me in certification tool memebers. Thank you.

@antonio-amjr
Copy link
Contributor

@antonio-amjr : Can you review this PR? I'm very thank you for your guidance. And also, Can I add you in the reviewer list? because I couldn't add any review in certification-tool. I want to add me in certification tool memebers. Thank you.

Sure, no problem at all. You can always add me in the reviewer list as well.
But anyhow, I already approved it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants