Skip to content

Add in-place updates e2e suite#910

Open
alexander-demicev wants to merge 3 commits into
rancher:mainfrom
alexander-demicev:inplaceupdatestests
Open

Add in-place updates e2e suite#910
alexander-demicev wants to merge 3 commits into
rancher:mainfrom
alexander-demicev:inplaceupdatestests

Conversation

@alexander-demicev
Copy link
Copy Markdown
Contributor

@alexander-demicev alexander-demicev commented May 21, 2026

What this PR does / why we need it:

This PR adds e2e suite and test extension for in-place updates, I also uncovered some issues with the implementation. All code mostly follows kubeadm patterns.

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 #907, #908

Special notes for your reviewer:

Checklist:

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

Signed-off-by: Alex Demicev <alex.demicev@lambdal.com>
Signed-off-by: Alex Demicev <alex.demicev@lambdal.com>
@alexander-demicev alexander-demicev requested a review from a team as a code owner May 21, 2026 10:06
Signed-off-by: Alex Demicev <alex.demicev@lambdal.com>
@furkatgofurov7 furkatgofurov7 added this to the v2.15.0 milestone May 26, 2026
@furkatgofurov7 furkatgofurov7 moved this to PR to be reviewed in CAPI / Turtles May 26, 2026
Comment thread pkg/rke2/desiredstate/desired_state.go
Comment thread test/extension/handlers/inplaceupdate/handlers.go
Copy link
Copy Markdown
Contributor

@anmazzotti anmazzotti left a comment

Choose a reason for hiding this comment

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

Nice work, thank you!

Copy link
Copy Markdown
Contributor

@furkatgofurov7 furkatgofurov7 left a comment

Choose a reason for hiding this comment

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

Thanks a lot, nice work!

@alexander-demicev
Copy link
Copy Markdown
Contributor Author

@furkatgofurov7 @anmazzotti can someone restart the lint job?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: PR to be reviewed

Development

Successfully merging this pull request may close these issues.

Write a mock update extension for CAPRKE2

3 participants