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

ci: another bunch of fixes #1051

Merged
merged 6 commits into from
Oct 18, 2023
Merged

ci: another bunch of fixes #1051

merged 6 commits into from
Oct 18, 2023

Conversation

ldevulder
Copy link
Contributor

@ldevulder ldevulder commented Oct 17, 2023

  • Bump ele-testhelpers deps
  • ci: wait a bit after installation with Helm
  • ci: don't use IPv6
  • ci: fix some boolean variables
  • ci/cli: create dedicated reset tests
  • ci: sync all and wait a bit before executing virt-install

Verification runs:

NOTE: CLI-K3s-Hardened-Upgrade-Reset-Rancher_Stable and CLI-RKE2-Hardened-Upgrade-Reset-Rancher_Stable may fail because of the reset feature which seems to sporadically fail.

@ldevulder ldevulder self-assigned this Oct 17, 2023
@ldevulder ldevulder requested a review from a team as a code owner October 17, 2023 19:42
@ldevulder ldevulder force-pushed the ci-another-bunch-of-fixes branch 4 times, most recently from d0838fe to 988ea81 Compare October 18, 2023 10:22
This is to (try to) ensure that CRDs are correctly installed

Signed-off-by: Loic Devulder <ldevulder@suse.com>
Needed to use updated helpers functions.

Signed-off-by: Loic Devulder <ldevulder@suse.com>
IPv6 is not configured in the test environment, force the OS to disable
it to avoid sporadic network issues.

Signed-off-by: Loic Devulder <ldevulder@suse.com>
Signed-off-by: Loic Devulder <ldevulder@suse.com>
@ldevulder ldevulder force-pushed the ci-another-bunch-of-fixes branch 2 times, most recently from 5a08723 to 1658f82 Compare October 18, 2023 13:29
Mainly used to not break all CI tests in case of issue in reset.

Signed-off-by: Loic Devulder <ldevulder@suse.com>
To try to avoid sporadic issue where sometimes the media is not found by virt-install.

Signed-off-by: Loic Devulder <ldevulder@suse.com>
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

@ldevulder ldevulder merged commit 826877c into main Oct 18, 2023
11 checks passed
@ldevulder ldevulder deleted the ci-another-bunch-of-fixes branch October 18, 2023 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants