Skip to content

Add multicast source filter IP address in ST 2110 connections#384

Merged
ko80 merged 2 commits intoOpenVisualCloud:mainfrom
ko80:add-mcast-sip-addr
Apr 14, 2025
Merged

Add multicast source filter IP address in ST 2110 connections#384
ko80 merged 2 commits intoOpenVisualCloud:mainfrom
ko80:add-mcast-sip-addr

Conversation

@ko80
Copy link
Collaborator

@ko80 ko80 commented Apr 12, 2025

  • Add JSON field "multicastSourceIpAddr".
  • Add FFmpeg argument "-mcast_sip_addr".
  • Rename JSON ST 2110 connection parameters "remoteIpAddr" and "remotePort" to "ipAddr" and "port" respectively to better align with MTL API.
  • Adjust unit tests.
  • Update documentation.
  • Update test scenario descriptions.

* Add JSON field "multicastSourceIpAddr".
* Add FFmpeg argument "-mcast_sip_addr".
* Rename JSON ST 2110 connection parameters "remoteIpAddr"
and "remotePort" to "ipAddr" and "port" respectively to better align
with MTL API.
* Adjust unit tests.
* Update documentation.

Signed-off-by: Konstantin Ilichev <konstantin.ilichev@intel.com>
@ko80 ko80 marked this pull request as ready for review April 13, 2025 09:23
@ko80 ko80 requested review from moleksy and soopel as code owners April 13, 2025 09:23
tszumski
tszumski previously approved these changes Apr 14, 2025
Mionsz
Mionsz previously approved these changes Apr 14, 2025
Copy link
Collaborator

@MateuszGrabuszynski MateuszGrabuszynski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not see any issues looking at the code. :)

zLukas
zLukas previously approved these changes Apr 14, 2025
Signed-off-by: Konstantin Ilichev <konstantin.ilichev@intel.com>
@ko80 ko80 dismissed stale reviews from zLukas, MateuszGrabuszynski, Mionsz, and tszumski via 6b262ea April 14, 2025 09:18
@ko80 ko80 force-pushed the add-mcast-sip-addr branch from 9ac715e to 6b262ea Compare April 14, 2025 09:18
@ko80 ko80 merged commit 0455080 into OpenVisualCloud:main Apr 14, 2025
9 checks passed
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.

5 participants