Skip to content

Commit

Permalink
Add ignore rule for no-log-password
Browse files Browse the repository at this point in the history
Ansible Galaxy is calling this out as a warning because it's not looking
at the loop_control which avoids printing the password.
  • Loading branch information
nickjj committed Sep 10, 2023
1 parent 9af29c8 commit 4333c44
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -145,8 +145,10 @@
- name: Restart Docker now to make sure `docker login` works
ansible.builtin.meta: "flush_handlers"

# It's safe to ignore no-log-password because we're removing the password from
# the output with the loop_control property.
- name: Manage Docker registry login credentials
community.docker.docker_login:
community.docker.docker_login: # noqa no-log-password
registry_url: "{{ item.registry_url | d(omit) }}"
username: "{{ item.username }}"
password: "{{ item.password }}"
Expand Down

0 comments on commit 4333c44

Please sign in to comment.