Skip to content

chore: upgrade ansible ee to support ansible-core 2.16+ #188

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

Merged
merged 1 commit into from
Mar 21, 2025

Conversation

syntaxsdev
Copy link
Contributor

@syntaxsdev syntaxsdev commented Mar 20, 2025

closes: #187
should close: #155
On systems with Python 3.12+ and higher, for some reason (even with the EE encompassing its own version) the mirror-registry command fails at the last steps to wait for the host to be live, despite the actual service being up and usable.

This is caused to due to deprecations in parameters for Python3.12 + in the urllib3 or requests libraries, causing the timeout to wait 3 minutes because of a parameter failure and them timeouts, causing workflows depending on the success/zero exit code to fail as well.

It is imperative that the base images are updated, as I have tried this across 20+ automated VM installs with many different versions of the library, it seems that this is the only fix (upgrading to Ansible 2.16+.)

I tested with make build-online-zip on a fresh server install.

@HammerMeetNail HammerMeetNail added the ok-to-test Indicates a non-member PR verified by an org member that is safe to test. label Mar 21, 2025
@HammerMeetNail
Copy link
Contributor

@syntaxsdev Thanks for the PR! I just kicked off the testing, we should be able to get this in pretty easily if everything passes.

@HammerMeetNail HammerMeetNail added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed ok-to-test Indicates a non-member PR verified by an org member that is safe to test. labels Mar 21, 2025
@syntaxsdev
Copy link
Contributor Author

syntaxsdev commented Mar 21, 2025

@HammerMeetNail Thanks for your reply. It seems like 2 of the tests are failing at the remote install but probably unrelated to the image change. Could we rerun these again or are we ok to merge?

@HammerMeetNail HammerMeetNail merged commit 7269bd9 into quay:main Mar 21, 2025
7 checks passed
@syntaxsdev syntaxsdev deleted the chore/upgrade-ansible branch March 22, 2025 04:24
@bcaton85
Copy link
Contributor

bcaton85 commented Apr 9, 2025

/cherry-pick mirror-registry-2.0-rhel-8

@openshift-cherrypick-robot

@bcaton85: new pull request created: #192

In response to this:

/cherry-pick mirror-registry-2.0-rhel-8

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test Indicates a non-member PR verified by an org member that is safe to test.
Development

Successfully merging this pull request may close these issues.

Ansible uri module fails to ignore self-signed certificate despite validate_certs: no
4 participants