-
Notifications
You must be signed in to change notification settings - Fork 83
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-1747] Use enablerepo instead of no_rhsm to check custom repos #1288
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1288 +/- ##
=======================================
Coverage 96.51% 96.51%
=======================================
Files 71 71
Lines 5077 5078 +1
Branches 883 883
=======================================
+ Hits 4900 4901 +1
Misses 98 98
Partials 79 79
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
bcdb1b0
to
619c7fa
Compare
convert2rhel/actions/pre_ponr_changes/custom_repos_are_valid.py
Outdated
Show resolved
Hide resolved
0cbbeed
to
64c1e91
Compare
64c1e91
to
8ba6221
Compare
8ba6221
to
702293d
Compare
702293d
to
db2f0ec
Compare
/packit test --labels sanity |
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.
Looks good to me!
db2f0ec
to
d652726
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.
The problem Michal mentioned is fixed, the changes looks good.
We were using the no_rhsm option to skip the check for custom repositories, and since the system can be unregistered or pre-registered, there is no need anymore to use no_rhsm in this action, instead, we can use enablerepo directly to validate if the repositories are valid or not.
Moving the CustomReposAreValid to pre_ponr_changes to be a dependency of SUBSCRIBE_SYSTEM.
d652726
to
fdb8942
Compare
/packit test --labels sanity |
From Adam: The problem Michal mentioned is fixed, the changes looks good.
@r0x0d can you create a Jira issue, update the PR text, and merge? |
We were using the no_rhsm option to skip the check for custom repositories, and since the system can be unregistered or pre-registered, there is no need anymore to use no_rhsm in this action, instead, we can use enablerepo directly to validate if the repositories are valid or not.
Jira Issues:
Checklist
[RHELC-]
or[HMS-]
is part of the PR titleRelease Pending
if relevantTODO