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

[cirrus] bump foreman versions #3363

Merged

Conversation

pmoravec
Copy link
Contributor

Bump tested foreman versions - use current latest (3.7) on Ubuntu and CentOS and previous 3.5 and 3.3 to match supported Satellite versions.


Please place an 'X' inside each '[]' to confirm you adhere to our Contributor Guidelines

  • Is the commit message split over multiple lines and hard-wrapped at 72 characters?
  • Is the subject and message clear and concise?
  • Does the subject start with [plugin_name] if submitting a plugin patch or a [section_name] if part of the core sosreport code?
  • Does the commit contain a Signed-off-by: First Lastname [email protected]?
  • Are any related Issues or existing PRs properly referenced via a Closes (Issue) or Resolved (PR) line?

@packit-as-a-service
Copy link

Congratulations! One of the builds has completed. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/sosreport-sos-3363
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

@TurboTurtle
Copy link
Member

Looks like there's a packaging issue for 3.3; I've restarted the job previously and it's failing at the same point early on so I don't think this is a transient failure.

Bump tested foreman versions - use current latest (3.7) on Ubuntu and
CentOS and previous 3.5 and 3.3 to match supported Satellite versions.

This requires enabling proper module streams for foreman-3.3 as well.

Signed-off-by: Pavel Moravec <[email protected]>
@pmoravec pmoravec force-pushed the sos-pmoravec-bump-foreman-in-cirrus branch from 2c57d2e to 850c557 Compare October 10, 2023 07:52
@pmoravec
Copy link
Contributor Author

I had to enable foreman:el8 (and subsequently postgresql:12) modules for foreman-3.3, which our setup misses.

@TurboTurtle TurboTurtle merged commit 362bbb8 into sosreport:main Oct 10, 2023
34 checks passed
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.

2 participants