Skip to content
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

Rotate kubeconfig before it expires #521

Merged
merged 2 commits into from
Jan 8, 2025

Conversation

ttreptow
Copy link
Contributor

@ttreptow ttreptow commented Dec 13, 2024

What this PR does / why we need it:

Recreates the kubeconfig after half of the expiration period (1 year, renews 6 months). See similar code in kubeadm control plane here

Which issue(s) this PR fixes

Special notes for your reviewer:

Checklist:

  • squashed commits into logical changes
  • includes documentation
  • adds unit tests
  • adds or updates e2e tests

@ttreptow ttreptow requested a review from a team as a code owner December 13, 2024 14:02
@furkatgofurov7
Copy link
Contributor

@ttreptow thanks for the PR, can you please fix the linter errors, before we take a closer look at it? Thanks!

@ttreptow ttreptow force-pushed the fix-kubeconfig-rotate branch from 74bf0d4 to d835bb9 Compare January 6, 2025 17:59
@ttreptow
Copy link
Contributor Author

ttreptow commented Jan 6, 2025

Updated to fix lints

@alexander-demicev alexander-demicev added the kind/bug Something isn't working label Jan 7, 2025
@ttreptow ttreptow force-pushed the fix-kubeconfig-rotate branch from d9fc050 to 891993a Compare January 7, 2025 15:54
@alexander-demicev alexander-demicev added kind/bug Something isn't working and removed kind/bug Something isn't working labels Jan 8, 2025
ttreptow and others added 2 commits January 8, 2025 17:14
Signed-off-by: Alexandr Demicev <alexandr.demicev@suse.com>
@alexander-demicev alexander-demicev added kind/feature New feature or request and removed kind/bug Something isn't working labels Jan 8, 2025
Copy link
Contributor

@salasberryfin salasberryfin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good pending CI

@alexander-demicev alexander-demicev merged commit 7c0bc0b into rancher:main Jan 8, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature or request
Development

Successfully merging this pull request may close these issues.

5 participants