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

Freeze for 2+ operators of the same priority #51

Closed
3 tasks
kopf-archiver bot opened this issue Aug 18, 2020 · 0 comments
Closed
3 tasks

Freeze for 2+ operators of the same priority #51

kopf-archiver bot opened this issue Aug 18, 2020 · 0 comments
Labels
archive bug Something isn't working good first issue Good for newcomers

Comments

@kopf-archiver
Copy link

kopf-archiver bot commented Aug 18, 2020

An issue by nolar at 2019-04-30 09:26:15+00:00
Original URL: zalando-incubator/kopf#51
 

Current Behaviour

When 2+ operators with the same priority run (e.g. default of 0), both are working. Which breaks the purpose of the peering to prevent duplicated side-effects.

Expected Behaviour

When 2+ operators with the same priority run, with peering configured and functional, only one should be functioning, others should freeze.

The best case would be that all operators issue a warning and freeze, so that the cluster becomes not served anymore — this will bring people attention sooner than if they negotiate which one should work. 2+ operators with the same priority is obviously a deployment error.

Note: If the operators make a decision on which one runs, which one freezes (e.g. by the most recent pod start time, or by id, or anything else comparable), then it could happen that the old deployment in another namespace serves the cluster, when a new operator is expected to work. It is better to stop serving and bring people attention to resolve the situation.

Suggestion

See the module where it all happens.

Checklist

  • Implementation.
  • Tests — manually (there is no auto-tests for peering yet).
  • Documentation (in docs/peering.rst)

Commented by smileisak at 2019-06-08 19:29:16+00:00
 

nolar is this issue is already treated ?


Commented by nolar at 2019-06-10 12:16:16+00:00
 

smileisak To my knowledge, no, it is free and waiting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
archive bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

0 participants