-
Notifications
You must be signed in to change notification settings - Fork 87
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
[RHELC-1070] Duplicate package check for preconversion analysis #987
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #987 +/- ##
==========================================
- Coverage 94.34% 94.34% -0.01%
==========================================
Files 47 48 +1
Lines 4580 4595 +15
Branches 818 821 +3
==========================================
+ Hits 4321 4335 +14
Misses 182 182
- Partials 77 78 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
convert2rhel/unit_tests/actions/system_checks/duplicate_packages_test.py
Fixed
Show fixed
Hide fixed
e2aab94
to
1f6480d
Compare
/packit test |
/packit build |
/packit test |
1 similar comment
/packit test |
1f6480d
to
9589867
Compare
convert2rhel/unit_tests/actions/system_checks/duplicate_packages_test.py
Show resolved
Hide resolved
9589867
to
0b1406a
Compare
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.
Few comments, but otherwise, looks good!
convert2rhel/unit_tests/actions/system_checks/duplicate_packages_test.py
Fixed
Show fixed
Hide fixed
c35b1e7
to
a0de28c
Compare
/packit test |
a0de28c
to
a02d991
Compare
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.
LGTM, just minor things. Can be corrected after test runs are done
/packit test |
With the Ethel Stage refresh going on today all tests here will be failing |
/packit build |
Co-authored-by: Rodolfo Olivieri <[email protected]>
/packit test |
4 similar comments
/packit test |
/packit test |
/packit test |
/packit test |
/packit retest-failed |
Once #1095 is merged the integration test failures related to the redhatrelease file backup should be fixed |
…#987) * [RHELC-1070] Duplicate package check for preconversion analysis * Update convert2rhel/actions/system_checks/duplicate_packages.py Co-authored-by: Rodolfo Olivieri <[email protected]> --------- Co-authored-by: Freya Gustavsson <[email protected]> Co-authored-by: Rodolfo Olivieri <[email protected]>
This PR adds a check to preconversion analysis to run "package-cleanup --dupes" to see if the users system contains multiple versions of the same package. If multiple versions of the same package are detected then an error is raised and the user is instructed to remove the duplicate packages with the help of the package-cleanup command.
Jira Issues: RHELC-1070
Checklist
[RHELC-]
is part of the PR titleRelease Pending
if relevant