-
Notifications
You must be signed in to change notification settings - Fork 276
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
[PublicCloud] Skip validate repo on SLEM 6+ #20229
base: master
Are you sure you want to change the base?
Conversation
@@ -146,6 +146,10 @@ sub register_openstack { | |||
# Validation for update repos | |||
sub validate_repo { | |||
my ($maintrepo) = @_; | |||
if (is_sle_micro('>=6.0')) { | |||
record_info("NEW MAINT MODE", "New style update, so can't validate repository"); |
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.
Do we really want one info box per repository?
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.
on SL Micro 6 its only one repo anyway
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 as per https://suse.slack.com/archives/C02CLB8TZP1/p1726743970046109?thread_ts=1726741239.519879&cid=C02CLB8TZP1 but one minor nitpick
@@ -146,6 +146,10 @@ sub register_openstack { | |||
# Validation for update repos | |||
sub validate_repo { | |||
my ($maintrepo) = @_; | |||
if (is_sle_micro('>=6.0')) { | |||
record_info("NEW MAINT MODE", "New style update, so can't validate repository"); |
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 message sounds a bit confusing for whom does not know about this. Maybe something like "Skip repo validation. Embargoed updates are not pushed in SL Micro 6+." or something similar.
No description provided.