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
MCO-532: Finish lease type migration #3842
Conversation
@djoshy: This pull request references MCO-532 which is a valid jira issue. In response to this:
Instructions 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/test-infra repository. |
Skipping CI for Draft Pull Request. |
/test all |
those were some weird (and completely unrelated) failures |
/retest-required |
@djoshy: all tests passed! Full PR test history. Your PR dashboard. Instructions 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/test-infra repository. I understand the commands that are listed here. |
Verified using IPI on GCP The controller pod can get the lease quickly even when we delete the pod several times
Same for the operator pod
When we enable the featuregate TechPreviewNoUpgrade, the time needed to acquire the lease is longer. This is a behavior that is not introduced by this PR and it is expected.
We can add the qe-approved label /label qe-approved |
/lgtm looks simple enough :) |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cdoern, djoshy The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…configmap OCP 4.14 moves to use LeasesResourceLock instead of ConfigMapsLeasesResourceLock and it doesn't have this `machine-config-controller` configmap and `crc start` failing on those bundle. This PR make sure if configmap doesn't present then don't send the error. - openshift/machine-config-operator#3842
…configmap OCP 4.14 moves to use LeasesResourceLock instead of ConfigMapsLeasesResourceLock and it doesn't have this `machine-config-controller` configmap and `crc start` failing on those bundle. This PR make sure if configmap doesn't present then don't send the error. - openshift/machine-config-operator#3842
- What I did
Changes the lease lock resource type from ConfigMapsLeases to LeasesResourceLock. We'll do a follow-up PR to clean up the old lease config maps in a later release.
- How to verify it
leader elections should work as before with no complaints, no timing weirdness etc
- Description for the changelog
migrate to LeasesResourceLock for leader elections