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

Pin Puppet in CI #279

Merged
merged 2 commits into from
Apr 28, 2023
Merged

Pin Puppet in CI #279

merged 2 commits into from
Apr 28, 2023

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Apr 26, 2023

Puppet 8.0.0 was released with incorrect Ruby compatibility metadata, which confuses bundler. This pins to more specific versions.

In gha-puppet this was already done, but this module has a modified beaker pipeline (to run multiple Pulpcore versions) so doesn't benefit from it.

Puppet 8.0.0 was released with incorrect Ruby compatibility metadata,
which confuses bundler. This pins to more specific versions.

In gha-puppet this was already done, but this module has a modified
beaker pipeline (to run multiple Pulpcore versions) so doesn't benefit
from it.
@ekohl
Copy link
Member Author

ekohl commented Apr 26, 2023

So puppetlabs/puppetlabs-postgresql#1425 is the next thing to chase ...

@evgeni
Copy link
Member

evgeni commented Apr 26, 2023

@evgeni
Copy link
Member

evgeni commented Apr 26, 2023

@evgeni
Copy link
Member

evgeni commented Apr 26, 2023

Huh, why is it installing Puppet 7.21, especially also on Puppet 6 acceptance branches?!

@evgeni
Copy link
Member

evgeni commented Apr 26, 2023

Ooh, it's the puppet that is running "outside" to parse things and launch beaker

@evgeni
Copy link
Member

evgeni commented Apr 27, 2023

the tests run now at least

@ekohl
Copy link
Member Author

ekohl commented Apr 28, 2023

So Pulp 3.17 is just broken: ImportError: /usr/lib64/python3.8/site-packages/_solv.so: undefined symbol: repo_add_deb, version SOLV_1.0

And Puppet 6 is busted because of concat.

@evgeni
Copy link
Member

evgeni commented Apr 28, 2023

Yeah. Shall we merge as-is as it's an improvement? :)

@evgeni
Copy link
Member

evgeni commented Apr 28, 2023

Nothing is using 3.17
And 3.16 was last used with 3.2
https://github.com/theforeman/forklift/blob/master/vagrant/config/versions.yaml

@ekohl
Copy link
Member Author

ekohl commented Apr 28, 2023

Yes, and then revive #277

@ekohl ekohl merged commit 4230d7d into theforeman:master Apr 28, 2023
@ekohl ekohl deleted the pin-ci branch April 28, 2023 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants