Skip to content

fix: Update role with ansible-lint recommendation#76

Open
Heniland wants to merge 1 commit into
NVIDIA:masterfrom
Heniland:master
Open

fix: Update role with ansible-lint recommendation#76
Heniland wants to merge 1 commit into
NVIDIA:masterfrom
Heniland:master

Conversation

@Heniland

@Heniland Heniland commented Sep 12, 2023

Copy link
Copy Markdown

This change includes a few minor changes.

  • Use ansible FQCN names over short names
  • Added become: true on specific modules that required privilege. (Make it clear which module needs privilege escalation.)
  • Add Nouveau driver module to blacklist on Ubuntu when installed without Cuda repo
  • for modprobe i used community.general.modprobe as it is recommended by Ansible Lint (You should use canonical module name community.general.modprobe instead of ansible.builtin.modprobe. )

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