Drop unsupported Python and OS versions #578
Merged
Conversation
This file contains 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
Due to Python 3 being the default in all supported OSs, we may remove the logic that dealt with detecting the OSs Python version and consequently installed a different set of packages. - removed dependency on Python 2 packages (python-psycopg2) - rephrased named steps in CI as it is explicitely set in `python-version` - require Ansible 2.12 as it supports the auto-detection of the OS Python interpreter - removed explicit setting of Python version in variables, playbooks and inventory - removed unsupported configurations from test inventory
Ansible 10 uses ansible-core 2.17+ which drops Python 3.6 support and thus makes it incompatible with Rocky Linux 8 or other RHEL 8 derivatives.
|
Would love to get your feedback on the changes I've made. Happy to change/revert commits if needed. |
MrMegaNova
approved these changes
Jul 19, 2024
|
Thank you - Happy to contribute :-) |
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.
The purpose of these changes is to fix CI tests not passing due to the Ansible 10 release being incompatible with supported OSs that have Python 3.6.
I've also made some updates in the documentation and have removed unused configuration and templates.
With these changes the CI passes again.