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.

Sign up for GitHub

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

Jump to bottom

ci/ui: install operator from cli for ui upgrade #1588

Merged
merged 1 commit into from
Sep 25, 2024
Merged

ci/ui: install operator from cli for ui upgrade #1588

merged 1 commit into from
Sep 25, 2024

Conversation

juadk
Copy link
Contributor

@juadk juadk commented Sep 24, 2024 •
edited
Loading

Fix #1578

  • Install the operator from the command line for upgrade test because registry.suse.com is not always synced with the version we have in the marketplace
  • Improve the script to get the latest stable OS version by differentiating Container and ISO

UI-K3S-Upgrade (test failed for another reason)
We can see that the version matches and the issue does not appear anymore.
image

UI-K3S ✅
CLI-K3s-OBS_Dev ✅

@juadk juadk force-pushed the 1578-install_operator_from_cli_for_ui_upgrade branch 3 times, most recently from 37d30f1 to aa068bf Compare September 25, 2024 07:42
@juadk juadk added the kind/QA label Sep 25, 2024
@juadk juadk self-assigned this Sep 25, 2024
ldevulder
ldevulder reviewed Sep 25, 2024
View reviewed changes
tests/e2e/suite_test.go Outdated Show resolved Hide resolved
ldevulder
ldevulder reviewed Sep 25, 2024
View reviewed changes
tests/e2e/suite_test.go Outdated Show resolved Hide resolved
ldevulder
ldevulder approved these changes Sep 25, 2024
View reviewed changes
Copy link
Contributor

@ldevulder ldevulder left a comment

Choose a reason for hiding this comment

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

LGTM, just a small comment not directly related to this PR.

@juadk juadk force-pushed the 1578-install_operator_from_cli_for_ui_upgrade branch 2 times, most recently from 2c05d0c to 8dc0334 Compare September 25, 2024 09:17
@juadk juadk marked this pull request as ready for review September 25, 2024 12:13
@juadk juadk requested a review from a team as a code owner September 25, 2024 12:13
@juadk juadk force-pushed the 1578-install_operator_from_cli_for_ui_upgrade branch from 8dc0334 to b49faca Compare September 25, 2024 12:14
@juadk juadk merged commit a6b07aa into main Sep 25, 2024
3 checks passed
@juadk juadk deleted the 1578-install_operator_from_cli_for_ui_upgrade branch September 25, 2024 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ldevulder ldevulder ldevulder approved these changes

Assignees

@juadk juadk

Labels
kind/QA
Projects
Elemental
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Install elemental from CLI instead of using marketplace for upgrade tests
2 participants
@juadk @ldevulder