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

fs-integration: Install python modules using pip #54

Merged

Conversation

anoopcs9
Copy link
Collaborator

@anoopcs9 anoopcs9 commented Feb 14, 2024

Very recently on CentOS Stream 8 ansible version got updated to v2.16.3 built with Python 3.12. Due to unavailability of minimum required python 3.12 modules playbook execution started to fail as follows:

An exception occurred during task execution. To see the full traceback, use -vvv. The error was: ansible.errors.AnsibleFilterError: The ipaddr filter requires python's netaddr be installed on the ansible controller
fatal: [localhost]: FAILED! => {"changed": false, "msg": "AnsibleFilterError: The ipaddr filter requires python's netaddr be installed on the ansible controller"}

Therefore switch python module installation from RPMs to pip.

Very recently on CentOS Stream 8 ansible version got updated to v2.16.3
built with Python 3.12. Due to unavailability of minimum required python
3.12 modules playbook execution started to fail as follows:

. . .
An exception occurred during task execution. To see the full traceback,
use -vvv. The error was: ansible.errors.AnsibleFilterError: The ipaddr
filter requires python's netaddr be installed on the ansible controller
fatal: [localhost]: FAILED! => {"changed": false,
 "msg": "AnsibleFilterError: The ipaddr filter requires python's netaddr
  be installed on the ansible controller"}
. . .

Therefore switch python module installation from RPMs to pip.
@anoopcs9 anoopcs9 changed the title fs-integration: Restrict to ansible version built with Py3.11 fs-integration: Install python modules using pip Feb 15, 2024
@anoopcs9
Copy link
Collaborator Author

anoopcs9 commented Feb 19, 2024

Proposed change was manually deployed at jenkins internally for validation of test runs. Expediting the merge as it would be blocking all other pull requests from sit-environment and sit-test-cases along with nightly test runs.

@anoopcs9 anoopcs9 merged commit 097f919 into samba-in-kubernetes:main Feb 20, 2024
2 checks passed
@anoopcs9 anoopcs9 deleted the c8s-restrict-ansible-vers branch February 20, 2024 05:49
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.

1 participant