-
Notifications
You must be signed in to change notification settings - Fork 166
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
Allow multiple replicas to be configured via helm chart #328
Comments
Hi @josh-ferrell, thanks for opening this. |
Having multiples running across zones for resiliency is a policy of ours. I assumed with the leader election that a second wouldn't be reconciling the CRs until they became the leader. TBH I'm not very familiar with k6 yet but if you can advise as to where multiple replicas would clash I can take a look at addressing them. |
@josh-ferrell I'm reviewing multiple replicas setup again and I believe you're right and we won't need to change anything. TBH, the last time I checked multiple replicas workflow was almost a year ago and leases just weren't present then (in our deps). But now it looks like yes, lease solves the issues: only one replica will reconcile the resources while the rest remain on standby. FYI, I'll continue testing this and will come back to you (and PR) a bit later this week. And thanks again for bringing this up to attention! 😄 |
Closed with #329 |
Feature Description
Allow multiple manager replicas to be configured via helm chart for redundancy.
Suggested Solution (optional)
No response
Already existing or connected issues / PRs (optional)
No response
The text was updated successfully, but these errors were encountered: