druid icon indicating copy to clipboard operation
druid copied to clipboard

Revert "Use canonical hostname instead of ip by default (#16386)"

Open abhishekagarwal87 opened this issue 1 year ago • 1 comments

This reverts commit 9459722ebf6565d7161edab671d91588ff2c6e1b.

The effect of PR https://github.com/apache/druid/pull/16386 is that by default, services would advertise themselves with localhost instead of their IP address and because of that, services in docker no longer can discover each other. I ran into this issue while verifying the docker image of 31 RC 1. We don't see this being an issue in IT because there, druid.host is already set to the container name.

abhishekagarwal87 avatar Oct 15 '24 13:10 abhishekagarwal87

Instead of reverting, what do you guys think about just adding ENV DRUID_SET_HOST_IP=1 into the Dockerfile? The reasoning here is that when running Druid on Kubernetes with ephemeral IP's certain features such as restoreTasksOnRestart break. Using canonical hostname is also helpful when looking at the Druid console since hostnames are easier to reason about than IP's

aho135 avatar Oct 23 '24 17:10 aho135

This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the [email protected] list. Thank you for your contributions.

github-actions[bot] avatar Dec 23 '24 00:12 github-actions[bot]

ENV DRUID_SET_HOST_IP=1 into the Dockerfile

This is a change in behavior no? Do rolling upgrades work if we do this ?

cryptoe avatar Jan 30 '25 09:01 cryptoe

cc @aho135

cryptoe avatar Jan 30 '25 09:01 cryptoe

This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the [email protected] list. Thank you for your contributions.

github-actions[bot] avatar Apr 01 '25 00:04 github-actions[bot]

This pull request/issue has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time.

github-actions[bot] avatar Apr 30 '25 00:04 github-actions[bot]