saithal-confluent opened a new pull request, #19598: URL: https://github.com/apache/druid/pull/19598
## Description Allow Druid nodes to advertise a different plaintext port for peer discovery while binding Jetty to the actual service port. ### Motivation When running Druid with a sidecar proxy (e.g. Envoy for east-west mTLS), the port the proxy listens on differs from the port Jetty binds to. Without this change, peer discovery advertises the Jetty bind port, so other nodes try to connect directly — bypassing the sidecar — and mTLS fails. ### Changes - Added `advertisedPlaintextPort` field to `DruidNode` (JSON property: `advertisedPlaintextPort`, `@Max(0xffff)`) - When set to a positive value, `getHostAndPort()` returns the advertised port instead of `plaintextPort` - When unset or ≤ 0, falls back to `plaintextPort` — no behaviour change for existing deployments - Updated `ServiceLocation` to use the advertised port - Added tests covering the new field and its fallback semantics ### Testing - [ ] `mvn test -pl server -Dtest=DruidNodeTest` - [ ] `mvn test -pl server` (full server module) <sub>This is a backport of an internal change validated in production.</sub> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
