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: install as simple application #897

Merged
merged 4 commits into from
Jul 11, 2023
Merged

ci: install as simple application #897

merged 4 commits into from
Jul 11, 2023

Conversation

ldevulder
Copy link
Contributor

@ldevulder ldevulder commented Jul 4, 2023 •
edited
Loading

Install a simple application and check it in different upgrade scenari.

NOTE: the test only runs on K3s for now, as there is no load-balancer installed by default on RKE2.

Should fix #700.

Verification runs:

Regression tests, RKE2 shouldn't be impacted:

@ldevulder ldevulder self-assigned this Jul 4, 2023
@ldevulder ldevulder requested a review from a team as a code owner July 4, 2023 14:06
@ldevulder ldevulder marked this pull request as draft July 4, 2023 14:07
@ldevulder ldevulder force-pushed the ci-add-application-check-during-upgrade branch 9 times, most recently from 5d75cc6 to cfe73e9 Compare July 5, 2023 09:57
kkaempf
kkaempf approved these changes Jul 5, 2023
View reviewed changes
Copy link
Contributor

@kkaempf kkaempf left a comment

Choose a reason for hiding this comment

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

lgtm

@ldevulder ldevulder force-pushed the ci-add-application-check-during-upgrade branch 16 times, most recently from 679b8de to dfad1df Compare July 7, 2023 13:07
@ldevulder ldevulder marked this pull request as ready for review July 7, 2023 13:15
@ldevulder ldevulder force-pushed the ci-add-application-check-during-upgrade branch 3 times, most recently from 9e4f4b6 to 95bc42c Compare July 10, 2023 13:01
@ldevulder ldevulder marked this pull request as draft July 10, 2023 13:03
@ldevulder ldevulder marked this pull request as ready for review July 10, 2023 14:12
juadk
juadk requested changes Jul 11, 2023
View reviewed changes
Copy link
Contributor

@juadk juadk left a comment

Choose a reason for hiding this comment

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

LGTM, only a small typo to change.

ldevulder added 4 commits July 11, 2023 09:25
@ldevulder
Fix copyright on some files
78f3bb7 
Signed-off-by: Loic Devulder <ldevulder@suse.com>
@ldevulder
ci: add hardened option on RKE2 tests
5ca13a6 
Signed-off-by: Loic Devulder <ldevulder@suse.com>
@ldevulder
ci/cli: fix Dev update channel
338436e 
Signed-off-by: Loic Devulder <ldevulder@suse.com>
@ldevulder
ci: install as simple application
cd2680b 
Install a simple application and check it in different upgrade scenari.

Signed-off-by: Loic Devulder <ldevulder@suse.com>
@ldevulder ldevulder force-pushed the ci-add-application-check-during-upgrade branch from 95bc42c to cd2680b Compare July 11, 2023 07:26
@ldevulder ldevulder enabled auto-merge (rebase) July 11, 2023 07:26
@ldevulder ldevulder merged commit da86e8d into main Jul 11, 2023
@ldevulder ldevulder deleted the ci-add-application-check-during-upgrade branch July 11, 2023 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kkaempf kkaempf kkaempf approved these changes

@juadk juadk juadk approved these changes

Assignees

@ldevulder ldevulder

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

Successfully merging this pull request may close these issues.

e2e: Check upgrade happens without downtime
3 participants
@ldevulder @kkaempf @juadk