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

fix: Update role with ansible-lint recommendation #76

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Heniland
Copy link

@Heniland Heniland commented Sep 12, 2023

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