[jitsi-dev] [jitsi/jitsi-videobridge] Harvesting (#362)


#1

Moves the mapping harvesters' configuration from jitsi-videobridge to ice4j. Fails health checks if STUN mapping harvesters were configured but failed to get a response. Clean-up. Adds a reason for health check failure to HTTP responses.
You can view, comment on, or merge this pull request online at:

  https://github.com/jitsi/jitsi-videobridge/pull/362

-- Commit Summary --

  * feat: Removes HarvesterConfiguration.
  * feat: Removes the code from HarvesterConfiguration.
  * style: Renames variables for simplicity.
  * feat: Initialize the ice4j mapping harvesters early.
  * feat: Supports the old names of some recently moved properties.
  * feat: Fails health checks if STUN address discovery failed.
  * feat: Adds a failure reason as HTTP content to failed health check responses.

-- File Changes --

    M src/main/java/org/jitsi/videobridge/HarvesterConfiguration.java (213)
    M src/main/java/org/jitsi/videobridge/IceUdpTransportManager.java (73)
    M src/main/java/org/jitsi/videobridge/RawUdpTransportManager.java (22)
    M src/main/java/org/jitsi/videobridge/Videobridge.java (33)
    M src/main/java/org/jitsi/videobridge/health/Health.java (33)

-- Patch Links --

https://github.com/jitsi/jitsi-videobridge/pull/362.patch
https://github.com/jitsi/jitsi-videobridge/pull/362.diff

···

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/362


#2

@bgrozev pushed 1 commit.

9ff1708 docs: Updates the documentation on harvesters (points to ice4j).

···


You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/362/files/021818fe1c54ed02ce363b42cef059fc2ae15cb7..9ff17080d3d7f55ef8f49d1384c3d1aa2da9f3bc


#3

Note: don't actually merge this PR until the one in ice4j is merged and jitsi-videobridge's pom.xml is updated.

···

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/362#issuecomment-261316794


#4

@bgrozev pushed 1 commit.

dbbdd4d mvn: Updates ice4j.

···


You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/362/files/9ff17080d3d7f55ef8f49d1384c3d1aa2da9f3bc..dbbdd4dbf4130c3085c1b21ad19f2586aed32af1


#5

Merged #362.

···

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/362#event-867400628