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

Rocky 9 dockerfiles for PWB (customer submitted!) #659

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from

Conversation

npelikan
Copy link

@npelikan npelikan commented Nov 8, 2023

These Rocky Linux 9 images were submitted to us by an extremely helpful customer! They've deployed these as part of a k8s install and have tested all functionality. They're reporting no issues. Hoping this helps as a starterpoint for RHEL9 or can live on their own.

@CLAassistant
Copy link

CLAassistant commented Nov 8, 2023

CLA assistant check
All committers have signed the CLA.

@ianpittwood
Copy link
Collaborator

@npelikan this is awesome, thank you! Is it fine if this doesn't build as part of our CI yet?

@npelikan
Copy link
Author

npelikan commented Nov 8, 2023

@npelikan this is awesome, thank you! Is it fine if this doesn't build as part of our CI yet?

No problem with that at all! I'll defer to y'all as to how you'd like to review/test/release/support these.

@ianpittwood
Copy link
Collaborator

@npelikan I've thought about this some and I think for now my plan will be to review and merge this as is. This should be a great help for #305!

Copy link
Collaborator

@ianpittwood ianpittwood left a comment

Choose a reason for hiding this comment

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

I've added a few changes to make these more cohesive with the rest of the repository and to fix a few issues with Just/Goss defintions. I tested these locally and everything seems to be in order!

Copy link
Contributor

@melissa-barca melissa-barca left a comment

Choose a reason for hiding this comment

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

@ianpittwood I'm going through PR review requests and found this one that I think should be good to merge?

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.

5 participants