Skip to content

share/ansible/palybook.yml: remove unused reserved name variable - #1726

Merged
alejandro-colomar merged 1 commit into
shadow-maint:masterfrom
ikerexxe:fix-ansible-warnings
Aug 26, 2026
Merged

share/ansible/palybook.yml: remove unused reserved name variable#1726
alejandro-colomar merged 1 commit into
shadow-maint:masterfrom
ikerexxe:fix-ansible-warnings

Conversation

@ikerexxe

Copy link
Copy Markdown
Collaborator

It was generating a warning and since the variable isn't used it's better to remove it.

[WARNING]: Found variable using reserved name 'name'.
Origin: /home/ipedrosa/repos/shadow/share/ansible/playbook.yml:15:9

13       vars:
14         privileged: "{{ privileged_mode | default(false) | bool }}"
15         name: "{{ container_name }}"
           ^ column 9

It was generating a warning and since the variable isn't used it's
better to remove it.
```
[WARNING]: Found variable using reserved name 'name'.
Origin: /home/ipedrosa/repos/shadow/share/ansible/playbook.yml:15:9

13       vars:
14         privileged: "{{ privileged_mode | default(false) | bool }}"
15         name: "{{ container_name }}"
           ^ column 9
```

Signed-off-by: Iker Pedrosa <ipedrosa@redhat.com>
@ikerexxe

Copy link
Copy Markdown
Collaborator Author

CC'ing @silverhadch

@silverhadch silverhadch left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alejandro-colomar alejandro-colomar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@alejandro-colomar
alejandro-colomar merged commit 5e7ae07 into shadow-maint:master Aug 26, 2026
16 checks passed
@ikerexxe
ikerexxe deleted the fix-ansible-warnings branch August 26, 2026 14:36
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.

3 participants