-
Notifications
You must be signed in to change notification settings - Fork 4.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change host-based UUIDs to opt-in #3171
Comments
Won't this break people not running in containers if they relied on host based node ids for anything? e.g. nagios alerts if consul agent on a host is down. |
If they relied on host-based IDs they'd have to opt back in, so we will document this as a breaking change for 0.9 that people need to consider when they upgrade. I think for the vast majority of users they are using the node name for things like alerts. The host-based IDs case really just made Consul + Nomad easier to use together because both would use the same IDs for a node, so we can document how to configure them for that. |
Should an issue be opened in the nomad project to highlight this in the Nomad documentation? FWIW, I'm starting to use Nomad and have been using Consul for a while and am really not sure how the host ID actually surfaces in regular usage of either or what would be affected if this wasn't set when using Nomad. |
Hi @nugend Nomad is making a similar default change over on hashicorp/nomad#2735. If you can use host-based IDs, then Nomad and Consul will use the same UUID, so when you see which Nomad node a job is running on you could |
Thanks. Sometimes it's just a little hard to get a handle on some of the concepts from the docs alone. |
hi @slackpad |
We've had a high number of users struggle to deploy newer versions of Consul with host-based UUIDs because of various edge cases of how the host IDs work in Docker, on specially-provisioned machines, etc. We need to change the sense of this to opt-in instead of opt-out by changing the default for
-disable-host-node-id
totrue
.The text was updated successfully, but these errors were encountered: