Skip to content
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

Add support to check LTSS repos #2780

Merged
merged 1 commit into from Apr 28, 2017
Merged

Conversation

mitiao
Copy link
Contributor

@mitiao mitiao commented Apr 24, 2017

lib/utils.pm Outdated
@@ -588,6 +588,9 @@ sub validatelr {
$product = 'SLE-Module-Web-Scripting';
$version = '12';
}
if ($product =~ /LTSS/) {
$version = '';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you add a comment why this should be a good idea?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added

lib/utils.pm Outdated
@@ -743,6 +752,10 @@ sub validate_repos_sle {
for my $product_channel ("Pool", "Updates", "Debuginfo-Pool", "Debuginfo-Updates", "Source-Pool") {
# Toolchain module doesn't have Source-Pool channel
next if (($scc_product eq 'SLE-TCM') && ($product_channel eq 'Source-Pool'));
# LTSS doesn't have Pool, Debuginfo and Sourcer-Pool channels
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/Sourcer/Source/

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

lib/utils.pm Outdated
# LTSS doesn't have Pool, Debuginfo and Sourcer-Pool channels
next if (($scc_product =~ /LTSS/) && ($product_channel eq 'Pool'));
next if (($scc_product =~ /LTSS/) && ($product_channel eq 'Debuginfo-Pool'));
next if (($scc_product =~ /LTSS/) && ($product_channel eq 'Source-Pool'));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how about

next if $scc_product =~ /LTSS/ and $product_channel =~ /(|Debuginfo-|Source-)Pool/;

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, this better

@okurz okurz merged commit f36f208 into os-autoinst:master Apr 28, 2017
@mitiao mitiao deleted the ltss_repos_check branch May 2, 2017 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants