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

Remove rancher-kubeconfigs feature #1033

Merged

Conversation

Danil-Grigorev
Copy link
Contributor

What this PR does / why we need it:

Current rancher version used with turtles uses CAPI 1.9, so the feature is no longer needed and thus removed. It was disabled by default for a while.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #1031

Special notes for your reviewer:

Checklist:

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

@Danil-Grigorev Danil-Grigorev requested a review from a team as a code owner January 22, 2025 10:43
@furkatgofurov7
Copy link
Contributor

@Danil-Grigorev please close the related issue rancher/turtles-docs#112 as unplanned in docs once this is merged

@Danil-Grigorev Danil-Grigorev force-pushed the remove-rancher-kubeconfig-label branch from 70497bb to 6822bc8 Compare January 22, 2025 10:46
furkatgofurov7
furkatgofurov7 previously approved these changes Jan 22, 2025
anmazzotti
anmazzotti previously approved these changes Jan 23, 2025
@Danil-Grigorev Danil-Grigorev force-pushed the remove-rancher-kubeconfig-label branch 3 times, most recently from 77720bb to 313be12 Compare January 23, 2025 09:07
Current rancher version used with turtles uses CAPI 1.9, so the feature
is no longer needed and thus removed.

Signed-off-by: Danil-Grigorev <[email protected]>
@Danil-Grigorev Danil-Grigorev force-pushed the remove-rancher-kubeconfig-label branch from 313be12 to 0fd81f6 Compare January 23, 2025 09:11
@alexander-demicev alexander-demicev merged commit 612a3ad into rancher:main Jan 23, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

features.rancher-kubeconfigs can be deprecated, the use case is no longer valid.
5 participants