-
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
service and check are deregistered after a crash #464
Comments
This is currently the expected behavior since the API driven checks/services are not persisted by the local agent. The plan is to eventually persist these out to the local -data-dir so the agent can recover from a crash. |
Might this be planned for 0.5.0? BrentOn Tue, Nov 11, 2014 at 5:36 PM, Armon Dadgar notifications@github.com
|
Most likely! It's a common request. |
that would be excellent! you have my +1. =) |
Remove meshGateway.enableHealthChecks
* Upgrade controller-runtime and controller-gen - Change CRD api-version to v1 - Update PROJECT to v3 from v3-alpha - Update reconciler interface - Update crds to latest interface - Replace deprecated fake.NewFakeClient with fake.ClientBuilder
Hi guys,
First let me say, we are loving consul!
We are starting to use it to advertise services on our mesos cluster, and while testing, discovered that, if consul crashes (simulated with a 'kill -9'), when it is restarted, it rejoins the cluster, and deregisters the service and healthcheck.
it's worth noting that when the service was started, it was registered via the http api, not through a consul.d config file.
Also, it's worth noting that we are using: "rejoin_after_leave": true
Here's a gist of the relevant portion of the log:
https://gist.github.com/brentley/87be2f33ce80e7637e6d
Now, if the service happens to die, and mesos restarts it, naturally, it gets registered via the api and all is good again.
Is this expected behavior, or is there an option we can enable to have it continue with whatever state it was in when it crashed (assuming healthchecks pass, obviously)
Thanks!
Brent
The text was updated successfully, but these errors were encountered: