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

Deprecated ansible.netcommon.ipaddr is still required after ansible update #10813

Closed
Red-GV opened this issue Jan 19, 2024 · 0 comments · Fixed by #10822
Closed

Deprecated ansible.netcommon.ipaddr is still required after ansible update #10813

Red-GV opened this issue Jan 19, 2024 · 0 comments · Fixed by #10822
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@Red-GV
Copy link

Red-GV commented Jan 19, 2024

What happened?

When running the kubernetes_sigs.kubespray.cluster playbook on the latest Kubespray (v2.24.0), I found a piece of the playbook which still required the use of the deprecated ipaddr method.

What did you expect to happen?

Run the ansible playbook without needing to install ansible.netcommon for the deprecated ipaddr method.

How can we reproduce it (as minimally and precisely as possible)?

Download the latest Kubespray and attempt to run the kubernetes_sigs.kubespray.cluster playbook without having ansible.netcommon installed.

OS

N/A

Version of Ansible

2.16.2

Version of Python

3.11.7

Version of Kubespray (commit)

v2.24.0

Network plugin used

calico

Full inventory with variables

No response

Command used to invoke ansible

No response

Output of ansible run

TASK [Check that python netaddr is installed] **************************************************************************************************************************************************************
[DEPRECATION WARNING]: Use 'ansible.utils.ipaddr' module instead. This feature will be removed from ansible.netcommon in a release after 2024-01-01. Deprecation warnings can be disabled by setting 
deprecation_warnings=False in ansible.cfg.

Anything else we need to know

This line (https://github.com/kubernetes-sigs/kubespray/blob/master/playbooks/ansible_version.yml#L24) should be changed to use ansible.utils.ipaddr.

@Red-GV Red-GV added the kind/bug Categorizes issue or PR as related to a bug. label Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant