NVIDIA / cloud-native-stack

Run cloud native workloads on NVIDIA GPUs
Apache License 2.0
118 stars 47 forks source link

Ansible 7.0.0 removes "warn" parameter for ansible.legacy.command #35

Closed tmckayus closed 1 year ago

tmckayus commented 1 year ago

"setup.sh install" pip install the latest ansible package, which is currently 7.0.0 (recently released). Apparently support for the "warn" parameter to ansible.legacy.command has been removed.

Suggestion is to remove the "warn" parameters in all *.yaml files, and perhaps pin the ansible version to a specific version in setup.sh to avoid unexpected changes in the future.

As a quick fix for a local checkout, the current CNS will work if ansible 6.0.0 is installed, for example in setup.sh replace the ansible install with the following line:

pip3 install ansible==6.0.0 2>&1 >/dev/null

tmckayus commented 1 year ago

Error from ansible looks like this:

TASK [Install Helm on NVIDIA Cloud Native Stack 6.3 or 7.1 or 8.0] **** failed: [localhost] (item=curl -O https://get.helm.sh/helm-v3.10.0-linux-amd64.tar.gz) => {"ansible_loop_var": "item", "changed": false, "item": "curl -O https://get.helm.sh/helm-v3.10.0-linux-amd64.tar.gz", "msg": "Unsupported parameters for (ansible.legacy.command) module: warn. Supported parameters include: argv, _raw_params, chdir, _uses_shell, executable, removes, creates, strip_empty_ends, stdin_add_newline, stdin."}

angudadevops commented 1 year ago

@tmckayus updated the code where we stick to Ansible 7.0.0 version with supported parameters, please let us know once you test with latest code.

tmckayus commented 1 year ago

@angudadevops thank you! I tested this with cuOpt and it is working well

tmckayus commented 1 year ago

https://github.com/NVIDIA/cuOpt-Resources/pull/27