Skip to content
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

Consul service health check defaults to advertise address #1574

Closed
evan2645 opened this issue Aug 12, 2016 · 3 comments
Closed

Consul service health check defaults to advertise address #1574

evan2645 opened this issue Aug 12, 2016 · 3 comments

Comments

@evan2645
Copy link
Contributor

Nomad version

Nomad v0.4.0

Operating system and Environment details

Ubuntu Xenial

Issue

The consul services published by nomad set the service address to the advertise address. This is expected behavior, however the health check which is registered against that service is also registered against the service address.

This is unexpected because often times the service address is unreachable from the localhost. Since the Consul agent is expected to run locally, and the default address is on the loopback adapter, it makes sense for the default endpoint of all associated ServiceChecks to be configured against the bind address as opposed to the advertise address. Additionally, the address for the ServiceCheck is not configurable.

Reproduction steps

  1. Install Nomad
  2. Configure Addresses with the interface address
  3. Configure Advertise with unreachable IP
  4. Start Nomad

Registered consul checks will fail against unreachable service address, even though Nomad is healthy.

Suggested Solution

Configure all ServiceChecks with endpoint address and port equal to the bind address (config.Addresses.HTTP and so on)

@ranjib
Copy link
Contributor

ranjib commented Aug 15, 2016

@diptanu @dadgar do you have any plan to address this, if its not soon enough (we need this before september end), will you be open to a patch from our side?

@dadgar
Copy link
Contributor

dadgar commented Aug 16, 2016

Thanks for the PR

@github-actions
Copy link

I'm going to lock this issue because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants