-
Notifications
You must be signed in to change notification settings - Fork 200
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
Switch Katello bats tests to better repo (fix Katello nightly BATS failures) #1806
Switch Katello bats tests to better repo (fix Katello nightly BATS failures) #1806
Conversation
@ianballou looks good, can you address the shell check comments? |
Doing a final full test and then I'll push up the last changes |
d3658f6
to
e53b17f
Compare
@ehelms this is good to go now, finished up my testing. |
bats/export_repos
Outdated
@@ -0,0 +1 @@ | |||
Zoo |
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.
Is this file needed?
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.
Might've snuck in, lemme double check.
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.
alright, it's deleted.
e53b17f
to
334e2bc
Compare
Yea - see theforeman/foreman-packaging#10669 |
Oooh, that's related?! |
Yep, check the output:
|
What a way to find a bug ;) |
rpm-packages-updateinfo
contains an important erratum (the SEA one) that points to the wrong RPMs.rpm-no-comps
correctly points to walrus-5.21.Also added some repo cleanup since the Pulp fixtures reuse RPMs, which causes clashes between the modular and non-modular versions of the RPMs.
Should fixes the Katello nightly pipeline.
Tested against nightly.