✨Support token reload without restarting#1780
Conversation
|
Hi @12345lcr. Thanks for your PR. I'm waiting for a kubernetes 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. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 12345lcr 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 |
|
/ok-to-test |
c959824 to
2d86b3f
Compare
What this PR does / why we need it:
In paravirtual mode, the CCM watches the mounted supervisor credentials and restarts the pod on any change. Since the credentials are delivered as a single projected Secret volume, routine token rotation triggered an unnecessary restart, even though client-go can reload the token on its own.
Backward compatible with the current non-expiring token and forward compatible with the planned short-lived token.
Which issue this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)format, will close that issue when PR gets merged): fixes #1779Special notes for your reviewer:
Manual Verification Complete
Step 1: Feature Deployment
Step 2: Basic Functionality Validation
Step 3: Resource Generation and Connectivity Verification
Step 4: Token Rotation and Refresh Testing
Step 5: Temporary Disruption Observation
Note: This represents the expected systemic delay before service account regeneration.
Step 6: Recovery Validation
Step 7: Post-Rotation Lifecycle Verification
Step 8: Pod Restart Testing
Step 9: Environmental Recovery
Release note: