Fix StatefulSet ordinal election leaving buckets unowned - #3393
shashankxrm wants to merge 1 commit into
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: shashankxrm The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Welcome @shashankxrm! It looks like this is your first PR to knative/pkg 🎉 |
|
Hi @shashankxrm. Thanks for your PR. I'm waiting for a knative member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Changes
STATEFUL_CONTROLLER_ORDINALas the explicit selector for StatefulSet mode. Invalid explicit configuration now fails closed instead of falling back to lease election. Standard lease election is unchanged when the ordinal env is absent.StatefulSet assignment remains 1:1 (
ordinal Nownsbucket N). Bucket names, DNS identities, and valid StatefulSet election behavior are unchanged./kind bug
Fixes #3384
Release Note
Docs