-
Notifications
You must be signed in to change notification settings - Fork 115
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
DRPM removal, SRPM fix #14675
DRPM removal, SRPM fix #14675
Conversation
trigger: test-robottelo |
PRT Result
|
does this need to be cherrypicked to earlier version too? @vsedmik |
Probably not necessarily, but I proposed 6.15 + 6.14 CPs just for the sake of the SRPM fix. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK, merge pending on the question
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK
* Get rid of deprecated DRPM repo tests * Fix SRPM repo tests (cherry picked from commit 0cf4cc6)
* Get rid of deprecated DRPM repo tests * Fix SRPM repo tests
Problem Statement
Delta RPMs are not supported in Pulp and Satellite for quite a long time and related tests are being skipped for three and half years, as well as some SRPM tests, which are still supported though.
Solution
Remove the DRPM tests and fix the SRPM ones (they used puppet env instead of LCE).
PRT test Cases example
trigger: test-robottelo
pytest: tests/foreman/api/test_repository.py::TestSRPMRepository