-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add Leader Election Lease Options #3430
Add Leader Election Lease Options #3430
Conversation
Signed-off-by: Will Sugarman <[email protected]>
Signed-off-by: Will Sugarman <[email protected]>
Signed-off-by: Will Sugarman <[email protected]>
I am unsure if this should result in new documentation |
Yeah, we should cover this in this section of docs: https://keda.sh/docs/2.8/operate/cluster/ Thanks! |
Signed-off-by: Will Sugarman <[email protected]>
Signed-off-by: Will Sugarman <[email protected]>
Signed-off-by: Will Sugarman <[email protected]>
Taking another look, perhaps there are too many options and it's better to simply share the same values for both the operator and metrics server? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, it's a lot of new values, but I think that it is okay to keep it like this :)
/run-e2e |
Signed-off-by: Will Sugarman <[email protected]>
Signed-off-by: Will Sugarman <[email protected]>
/run-e2e cron* |
@zroubalik Thank you for your help! |
Add new options for configuring the leader election lease, including:
KEDA_OPERATOR_LEADER_ELECTION_LEASE_DURATION
KEDA_OPERATOR_LEADER_ELECTION_RENEW_DEADLINE
KEDA_OPERATOR_LEADER_ELECTION_RETRY_PERIOD
KEDA_METRICS_LEADER_ELECTION_LEASE_DURATION
KEDA_METRICS_LEADER_ELECTION_RENEW_DEADLINE
KEDA_METRICS_LEADER_ELECTION_RETRY_PERIOD
Checklist
env
elementRelates to #2836
Relates to kedacore/keda-docs#842