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: fix edge test failures in cs9 and fedora40 #4309

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

yih-redhat
Copy link
Contributor

@yih-redhat yih-redhat commented Aug 16, 2024

This PR fixed test failures on fedora40 and centos-stream9.

  1. disabled edge-commit and edge-installed test on fedora40 due to bug https://bugzilla.redhat.com/show_bug.cgi?id=2294897
  2. disabled minimal-raw test on fedora40 due to bug minimal-raw install podman issue virt-s1/rhel-edge#3531 (this error only happens in upstream, need to use latest snapshot repo)
  3. disabled ostree-remount test case on cs9 due to bug https://issues.redhat.com/browse/RHEL-25249
  4. fixed some test case errors.

Those disabled cases will be added back after the bugs are fixed.

@yih-redhat
Copy link
Contributor Author

@achilleas-k could you please review this pull request? With this pr, all edge test cases will run in testing-farm, includes rhel9.5, cs9 and fedora40.

Copy link
Member

@achilleas-k achilleas-k left a comment

Choose a reason for hiding this comment

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

Could you split the PR into multiple commits? It seems to be doing more than the commit message implies. The changes in minimal-raw.sh and check_ostree.yaml, if they're related, could use a mention in the commit message. If they're unrelated to the move to testing farm, then separate commits would make more sense.

test/data/ansible/check_ostree.yaml Show resolved Hide resolved
@yih-redhat yih-redhat changed the title ci: move edge fedora and centos-stream test to testing-farm ci: fix edge test failures in cs9 and fedora40 Aug 19, 2024
@achilleas-k achilleas-k enabled auto-merge (rebase) August 19, 2024 15:30
@achilleas-k achilleas-k merged commit f27f9a2 into osbuild:main Aug 19, 2024
46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants