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

Drop out legacy and add default upgrade e2e tests #398

Merged

Conversation

furkatgofurov7
Copy link
Contributor

@furkatgofurov7 furkatgofurov7 commented Aug 7, 2024

What this PR does / why we need it:
Drop legacy upgrade tests (v0.2.7->v0.3.0) and related code introduced in a7b3b2f#diff-868dcd8976a79a1736e0bfe328a5100bd09c295509f0c3edd3f1d6baa72ff4aa
and add back default upgrade (v0.4.0->v0.5.0) e2e tests.

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

@furkatgofurov7
Copy link
Contributor Author

furkatgofurov7 commented Aug 7, 2024

Strange, e2e tests are terminating (https://github.com/rancher/cluster-api-provider-rke2/actions/runs/10281104489/job/28450906458?pr=398, https://github.com/rancher/cluster-api-provider-rke2/actions/runs/10281104489/job/28449894374?pr=398) out of nowhere in the middle and it fails.. no logs, no traces, nothing 😮‍💨

@furkatgofurov7
Copy link
Contributor Author

New e2e tests are not running in suite, checking it.

@furkatgofurov7 furkatgofurov7 force-pushed the separate-legacy-upgrade-tests branch 10 times, most recently from 3069a52 to 4149e41 Compare August 12, 2024 15:36
@furkatgofurov7 furkatgofurov7 changed the title Separate out legacy and default upgrade e2e tests Drop out legacy and add default upgrade e2e tests Aug 12, 2024
@furkatgofurov7 furkatgofurov7 force-pushed the separate-legacy-upgrade-tests branch 2 times, most recently from 5a0aa17 to d6a51dc Compare August 12, 2024 16:43
Signed-off-by: Furkat Gofurov <furkat.gofurov@suse.com>
e2e image building is already part of the 'make e2e-tests'

Signed-off-by: Furkat Gofurov <furkat.gofurov@suse.com>
@furkatgofurov7 furkatgofurov7 force-pushed the separate-legacy-upgrade-tests branch from d6a51dc to ec2b27b Compare August 12, 2024 17:16
@furkatgofurov7 furkatgofurov7 self-assigned this Aug 13, 2024
@alexander-demicev alexander-demicev merged commit 125d6b8 into rancher:main Aug 13, 2024
6 checks passed
@furkatgofurov7 furkatgofurov7 deleted the separate-legacy-upgrade-tests branch August 13, 2024 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/refactor Denotes a change that is a refactor
Development

Successfully merging this pull request may close these issues.

3 participants