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

Delete unused envs #988

Merged
merged 1 commit into from
Jan 8, 2025
Merged

Conversation

anmazzotti
Copy link
Contributor

What this PR does / why we need it:

Just a little bump to ensure we are by default testing with latest version.

@anmazzotti anmazzotti requested a review from a team as a code owner January 8, 2025 13:58
furkatgofurov7
furkatgofurov7 previously approved these changes Jan 8, 2025
@Danil-Grigorev
Copy link
Contributor

I suggest removing this variable as it has no effect on dev setup.

Signed-off-by: Andrea Mazzotti <[email protected]>
@anmazzotti anmazzotti force-pushed the update_rke2_local_dev branch from 0db60e1 to 6924377 Compare January 8, 2025 15:00
@anmazzotti anmazzotti changed the title Update default RKE2 provider version in local dev environment Delete unused envs Jan 8, 2025
@anmazzotti
Copy link
Contributor Author

Good point @Danil-Grigorev
I also removed the CAPI version that didn't seem to be in use either.

@anmazzotti anmazzotti enabled auto-merge January 8, 2025 15:12
@anmazzotti anmazzotti merged commit 3e2336e into rancher:main Jan 8, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants