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

Switch package manager tests from diffutils to another package #3454

Merged
merged 1 commit into from
Jan 8, 2025

Conversation

happz
Copy link
Collaborator

@happz happz commented Jan 8, 2025

diffutils now cannot be removed from coreos images, therefore we are picking another packages to be used for package manager units tests. Not in all cases, some images are not affected.

Pull Request Checklist

  • implement the feature

@happz happz added code | no functional change "No Functional Change" intended. Patch should not change tmt's behavior in any way. test coverage Improvements or additions to test coverage of tmt itself labels Jan 8, 2025
@happz happz added this to the 1.41 milestone Jan 8, 2025
@happz happz added the ci | full test Pull request is ready for the full test execution label Jan 8, 2025
diffutils now cannot be removed from coreos images, therefore we are
picking another packages to be used for package manager units tests. Not
in all cases, some images are not affected.
@happz happz force-pushed the switch-test-download-installation-from-diffutils branch from 67241f7 to 320a55a Compare January 8, 2025 20:31
Copy link
Collaborator

@psss psss left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@psss psss self-assigned this Jan 8, 2025
@psss psss merged commit 0bf5b80 into main Jan 8, 2025
12 of 20 checks passed
@psss psss deleted the switch-test-download-installation-from-diffutils branch January 8, 2025 23:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci | full test Pull request is ready for the full test execution code | no functional change "No Functional Change" intended. Patch should not change tmt's behavior in any way. test coverage Improvements or additions to test coverage of tmt itself
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants