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 for UBI 8 base image to Pbench Server container image build #3582

Merged
merged 1 commit into from
Dec 8, 2023

Conversation

webbnh
Copy link
Member

@webbnh webbnh commented Dec 7, 2023

During a side trip trying to get the Pbench 1.0 Server deployed on RHEL 8.9, we tried to build the Server container image on a UBI 8 base instead of the usual UBI 9 one and ran into troubles. This PR makes the build work.

It turns out that -8 uses a different version of EPEL from what -9 uses, so this changes the container-build.sh script to add the necessary logic to support that.

Also, we tumbled to the fact that the Server no longer depends on util-linux-core (which we couldn't locate easily in the -8 EPEL), so this change removes that dependency from the RPM spec file.

For the record, this change received only the lightest testing: it passes the CI but mostly because the changes are unused in normal operation. If someone actually tries to use a UBI 8-based Server, they will run into problems because the version of SSL is too old to produce the TLS certificates in the conventional way and the version of Python is 3.6 which is too old to run the Server code.

@webbnh webbnh added the Server label Dec 7, 2023
@webbnh webbnh requested a review from dbutenhof December 7, 2023 15:21
@webbnh webbnh self-assigned this Dec 7, 2023
dbutenhof
dbutenhof previously approved these changes Dec 7, 2023
Copy link
Member

@dbutenhof dbutenhof left a comment

Choose a reason for hiding this comment

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

What is there to say? The proof will be in the taste of the pudding.

@webbnh webbnh changed the title Switch Pbench Server container to UBI8 base Add support for UBI 8 base image to Pbench Server container image build Dec 7, 2023
@webbnh webbnh merged commit b5bee97 into distributed-system-analysis:main Dec 8, 2023
4 checks passed
@webbnh webbnh deleted the ubi8_server branch December 8, 2023 01:20
@webbnh
Copy link
Member Author

webbnh commented Dec 12, 2023

For those who do not study it (or who cannot remember it), history is doomed to repeat itself: #3461. (Apparently, I didn't carry forward from that quite as much as I should have!...ah, well, second time's the charm.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants