-
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-1379] Port certs and restorable key to backup module #1053
Conversation
26e5789
to
a048f67
Compare
a048f67
to
58532e0
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1053 +/- ##
=======================================
Coverage 94.44% 94.44%
=======================================
Files 48 48
Lines 4551 4554 +3
Branches 812 812
=======================================
+ Hits 4298 4301 +3
Misses 178 178
Partials 75 75
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Both the RestorableRpmKey and the PEMCert classes got migrated to their own module inside the backup folder. Signed-off-by: Rodolfo Olivieri <[email protected]>
58532e0
to
7097978
Compare
convert2rhel/unit_tests/actions/pre_ponr_changes/subscription_test.py
Fixed
Show resolved
Hide resolved
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.
Just one comment about an unused import, otherwise looks good
convert2rhel/unit_tests/actions/pre_ponr_changes/subscription_test.py
Fixed
Show resolved
Hide resolved
Signed-off-by: Rodolfo Olivieri <[email protected]>
/packit test --labels sanity |
1 similar comment
/packit test --labels sanity |
Both the RestorableRpmKey and the PEMCert classes got migrated to their own module inside the backup folder.
Jira Issues: RHELC-1379
Checklist
[RHELC-]
is part of the PR titleRelease Pending
if relevant