Skip to content

[1.28] Enhancement and fix for containerized pytest#3215

Merged
ptoscano merged 2 commits intosubscription-manager-1.28from
mhorky/pytest-ci-1.28
Mar 1, 2023
Merged

[1.28] Enhancement and fix for containerized pytest#3215
ptoscano merged 2 commits intosubscription-manager-1.28from
mhorky/pytest-ci-1.28

Conversation

@m-horky
Copy link
Contributor

@m-horky m-horky commented Feb 23, 2023

First commit addresses one test that was being skipped on Fedora images. By including dnf-plugins-core package there as well, we can run it.

Second commit removes the excluded tests on CentOS 9 Stream, as https://bugzilla.redhat.com/show_bug.cgi?id=2167468 is now fixed.

Backport of PR #3214.

@m-horky m-horky force-pushed the mhorky/pytest-ci-1.28 branch from cdee964 to c3a2f56 Compare February 23, 2023 16:05
@github-actions
Copy link

github-actions bot commented Feb 23, 2023

Coverage

Coverage (computed on Fedora latest) •
FileStmtsMissCoverMissing
TOTAL22713995856% 
report-only-changed-files is enabled. No files were changed during this commit :)

Tests Skipped Failures Errors Time
2340 10 💤 0 ❌ 0 🔥 5m 30s ⏱️

When the package is not available, one test from
test/test_dnf_content_plugin.py is skipped.

(Adapted from a894837)
@m-horky m-horky force-pushed the mhorky/pytest-ci-1.28 branch from c3a2f56 to ea1f58b Compare February 27, 2023 11:05
They were required because of recent bug in systemd:
- https://bugzilla.redhat.com/show_bug.cgi?id=2167468
- systemd/systemd#26366

With this fix, we can remove the test deselections.

(Adapted from ce02424)
@m-horky m-horky force-pushed the mhorky/pytest-ci-1.28 branch from ea1f58b to aaeb7c3 Compare March 1, 2023 10:15
Copy link
Contributor

@ptoscano ptoscano left a comment

Choose a reason for hiding this comment

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

Easy backport, thanks!

@ptoscano ptoscano merged commit acf1eb2 into subscription-manager-1.28 Mar 1, 2023
@ptoscano ptoscano deleted the mhorky/pytest-ci-1.28 branch March 1, 2023 10:36
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