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

Unflaky ci tests #1204

Merged
merged 5 commits into from
Mar 25, 2022
Merged

Unflaky ci tests #1204

merged 5 commits into from
Mar 25, 2022

Conversation

mudler
Copy link
Contributor

@mudler mudler commented Mar 25, 2022

  • Disable part of the EFI installer tests which makes it flaky. Keeps one portion for sanity check (that it boots and can install, without performing a reboot)
  • Split up installer tests
  • Tighten up the bootstrap script used in ci to install luet. Adds checksum verification and also installation via docker from our repositories which is then forced to be used in our pipelines.

Fixes #1202

@mudler mudler force-pushed the unflaky_ci branch 2 times, most recently from 989910f to bd90a20 Compare March 25, 2022 10:07
@mudler mudler requested review from Itxaka and davidcassany and removed request for Itxaka March 25, 2022 10:10
mudler added 5 commits March 25, 2022 11:31
See #1202

Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
Also add checksum verification when downloading the blob with curl

Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
@mudler mudler enabled auto-merge (squash) March 25, 2022 10:54
_DOCKER_IMAGE="$REPO_URL:luet-toolchain-$LUET_VERSION"
echo "Using luet bootstrap version from docker image: ${_DOCKER_IMAGE}"
docker run --entrypoint /usr/bin/luet --name luet ${_DOCKER_IMAGE} --version
docker cp luet:/usr/bin/luet ./
Copy link
Contributor

Choose a reason for hiding this comment

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

should we clean the container afterwards?

@mudler mudler merged commit f7a8590 into master Mar 25, 2022
@mudler mudler deleted the unflaky_ci branch March 25, 2022 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
architecture/arm64 status/safe to test Triggers self-hosted actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unflaky CI tests
3 participants