fix: resolve pyOpenSSL/cryptography build conflicts in AAP 2.5 EE#15
Closed
shebistar wants to merge 1 commit into
Closed
fix: resolve pyOpenSSL/cryptography build conflicts in AAP 2.5 EE#15shebistar wants to merge 1 commit into
shebistar wants to merge 1 commit into
Conversation
The AAP 2.5 base image (ee-minimal-rhel9:latest) was updated and now ships python3.12-pyOpenSSL-26.0.0 (requires typing-extensions >= 4.9) and python3.12-cryptography installed via RPM. This caused two failures: 1. subscription-manager install failed with a dependency resolution error because pyOpenSSL required typing-extensions but something in subscription-manager's dep tree conflicted with that requirement. 2. The ansible-builder assemble script failed trying to uninstall cryptography via pip when it was installed via RPM (no RECORD file). Fix: move the $PKGMGR remove step BEFORE the subscription-manager install, and extend the list to include python3.12-pyOpenSSL and python3.12-cryptography. Removing them first via RPM (microdnf) before pip runs avoids both conflicts entirely. Closes #14
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes the
build-and-push (2.5)failure that is blocking all open PRs in this repo. The AAP 2.5 base image was updated and introduced two dependency conflicts that need to be resolved beforesubscription-manageris installed.Root Cause
Two failures in
openshift-virtualization-migration-ee-2.5/execution-environment.yml:Failure 1 —
subscription-managerinstall (prepend_base phase)The updated AAP 2.5 base image ships
python3.12-pyOpenSSL-26.0.0which requirestyping-extensions >= 4.9. Something insubscription-manager's dependency tree conflicts with that. Because the cleanup$PKGMGR removestep ran after the install, the conflict was never cleared in time.Failure 2 —
cryptographyuninstall (builder/assemble phase)cryptography 46.0.5in the new base image was installed via RPM.ansible-builder's assemble script tries to manage it through pip, which fails with no RECORD file.Fix
In
prepend_base, the$PKGMGR removecleanup is moved before thesubscription-managerinstall, andpython3.12-pyOpenSSL+python3.12-cryptographyare added to the removal list. Removing them via RPM (microdnf) first eliminates both conflicts before eithersubscription-manageror pip attempts to touch them.Files Changed
openshift-virtualization-migration-ee-2.5/execution-environment.ymlImpact
This is the blocker for all other open PRs (#10, #11, #12, #13). Merging this first will unblock them. The 2.4 and 2.6 EE builds are not affected.
Test plan
build-and-push (2.5)passes in this PR's CI runsubscription-managerregistration succeeds whenREDHAT_ORGandREDHAT_ACTIVATION_KEYare setCloses #14
Made with Cursor