Skip to content

Fix triton multicast#13

Merged
lvboudre merged 10 commits intomasterfrom
fix-triton-multicast
Apr 13, 2026
Merged

Fix triton multicast#13
lvboudre merged 10 commits intomasterfrom
fix-triton-multicast

Conversation

@lvboudre
Copy link
Copy Markdown

Current Problem

  • Command line argument triton-multicast-num-threads if set to a number bigger than 1 would cause to join the multicast group N times and receiving the same traffic N times.
  • Missing multicast-subscription-port, when joining the multicast group, we don't specify a multicast subscription port, and create a socket with a random port. So the multicast socket creates bind our procedure would receive nothing. Unless --src-bind-port was not properly set to a port that match on of the joined multicast group, the proxy would receive nothing.
  • packet received by the multicast socket were never added to prometheus observation.

New

  • Removed the triton-multicast-num-threads.
  • Added CLI arguments triton-multicast-subscription-port.

Fixes

  • When provided with CLI arguments to join a multicast group owned by triton, a dedicated socket is created and bind to 0.0.0.0:[triton-multicast-subscription-port].
  • Refactor the code to remove duplicate code everywhere and used a unified approach to be aligned with jito's original code.

@lvboudre lvboudre requested a review from gritsly March 24, 2026 20:01
@lvboudre lvboudre merged commit 9408a58 into master Apr 13, 2026
0 of 2 checks passed
lvboudre added a commit that referenced this pull request Apr 13, 2026
lvboudre added a commit that referenced this pull request Apr 13, 2026
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.

3 participants