-
Notifications
You must be signed in to change notification settings - Fork 316
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
Consistent topology per service_group #2005
Comments
I think a common scenario might be a simple web application where you want a leader topology for the web tier but a standalone database. Granted its just a demo, but in https://github.com/habitat-sh/habitat-aspnet-sample, I have such a scenario. I want rolling updates to the web app but don't want the complexity of a multi node db cluster. |
@mwrock Right but it is possible to start some parts of your web app with |
yes that would be undesirable. I see now that I have mixed |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. We value your input and contribution. Please leave a comment if this issue still affects you. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. We value your input and contribution. Please leave a comment if this issue still affects you. |
Currently the topology of a service is not being gossiped. It will need to be gossiped in order to implement #2003.
However first we need to address some inconsistent/undefined behaviour. Currently topology is scoped to a
service
and not aservice_group
this means it is possible for some services in a group to execute logic as if they were instandalone
topology while another part of the groupexecutes code relevant to
leader
topology (eg. here).How should this be dealt with?
I suggest prohibiting services to join a group that are started with a different
--topology
setting as the rest of the group.The text was updated successfully, but these errors were encountered: