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 specific versions in dev script #843

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Danil-Grigorev
Copy link
Contributor

@Danil-Grigorev Danil-Grigorev commented Nov 6, 2024

What this PR does / why we need it:

Dev script requires specific versions for CAPI providers.

This PR proposes removing specific version requirements. Current pinned version (v1.7.7) of CAPRKE2 is not compatible with turtles, as agent install command causes continuous remediation of CP machines. This problem was fixed with latest.

Additionally, a pinned version requires regular bumps with updatecli, which increases complexity of the automation, focused only on maintaining dev environment.

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 #

Special notes for your reviewer:

Checklist:

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

--set core=cluster-api:v1.7.7 \
--set controlPlane=rke2:v0.8.0 \
--set bootstrap=rke2:v0.8.0 \
--set infrastructure=docker \
Copy link
Contributor

@furkatgofurov7 furkatgofurov7 Nov 6, 2024

Choose a reason for hiding this comment

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

the only objective I would have is: with this change, as of now we start installing CAPI v1.8.5 (=latest) and CAPRKE2 v0.8.0 (=latest) where latter is compatible only with CAPI v1.7.x series

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.

4 participants