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

upstream CI: Pin Python version to 3.11 #1157

Merged
merged 1 commit into from
Oct 18, 2023

Conversation

rjeffman
Copy link
Member

@rjeffman rjeffman commented Oct 13, 2023

After Azure updated available Python to version 3.12, test image preparation started to fail.

The fix is to pin python version to the latest is the 3.11 series until we can run tests with Python 3.12.

Note that only the first commit should be applied (55b8729). Other commits come from PR #1148 and are present only to fix test selection.

@rjeffman rjeffman added the DRAFT label Oct 13, 2023
@rjeffman rjeffman force-pushed the ci_fix_docker_install branch 7 times, most recently from 51401bc to ea933bc Compare October 17, 2023 19:10
@rjeffman rjeffman removed the DRAFT label Oct 17, 2023
Azure Ubuntu images have Python 3.12 available, and as we did not pin
the requested Python version, the latest available one was used, causing
image preparation and tests to fail.

This patch pins Python version to 3.11 until test can be executed with
Python 3.12 and later.
@rjeffman rjeffman changed the title Ci fix docker install Upstream CI: Fix test image preparation Oct 17, 2023
@rjeffman rjeffman changed the title Upstream CI: Fix test image preparation upstream CI: Pin Python version to 3.11 Oct 18, 2023
Copy link
Member

@t-woerner t-woerner left a comment

Choose a reason for hiding this comment

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

55b8729 LGTM
Please remove the other commits.

@t-woerner
Copy link
Member

Thanks for the fix!

@t-woerner t-woerner merged commit ee73542 into freeipa:master Oct 18, 2023
11 of 31 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