Skip to content

Commit de02020

Browse files
Partial ansible-lint fixes applied
1. Schema Violations - Replaced deprecated include with ansible.builtin.include_tasks - Fixed old-style include directives 2. Naming Violations - Capitalized task names that started with lowercase letters - Added names to plays and tasks that were missing them - Fixed naming issues 3. YAML Truthy Violations - Converted True/False to lowercase true/false 4. Indentation Violations - Corrected wrong indentation - Fixed expected vs found indentation mismatches 5. YAML Formatting Issues - Fixed trailing spaces - Removed excessive empty lines - Corrected colon spacing - Fixed bracket spacing 6. FQCN Violations - Added ansible.builtin prefix to core module calls - Applied to include_tasks, set_fact, debug, etc. 7. Module Docstring YAML Errors - Fixed syntax errors - Removed excess blank lines in documentation strings - Corrected YAML formatting in EXAMPLES sections 8. Miscellaneous Fixes - Fixed no-free-form violations in set_fact calls - Corrected var-naming pattern - Fixed jinja2 spacing
1 parent f802a5c commit de02020

File tree

296 files changed

+4867
-6282
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

296 files changed

+4867
-6282
lines changed

galaxy.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
namespace: cisco
3-
name: dcnm
3+
name: Dcnm
44
version: 3.9.0-dev
55
readme: README.md
66
authors:

playbooks/roles/dcnm_bootflash/create_files.yaml

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,10 @@
88
- name: SETUP - Create files on {{ switch1 }}
99
cisco.nxos.nxos_command:
1010
commands:
11-
- echo 1 > bootflash:/{{ nxos_vars.switch1_file1 }}
12-
- echo 1 > bootflash:/{{ nxos_vars.switch1_file2 }}
13-
- echo 1 > bootflash:/{{ nxos_vars.switch1_file3 }}
14-
- echo 1 > bootflash:/{{ nxos_vars.switch1_file4 }}
11+
- echo 1 > bootflash: /{{ nxos_vars.switch1_file1 }}
12+
- echo 1 > bootflash: /{{ nxos_vars.switch1_file2 }}
13+
- echo 1 > bootflash: /{{ nxos_vars.switch1_file3 }}
14+
- echo 1 > bootflash: /{{ nxos_vars.switch1_file4 }}
1515

1616
- gather_facts: false
1717
hosts:
@@ -22,7 +22,7 @@
2222
- name: SETUP - Create files on {{ switch2 }}
2323
cisco.nxos.nxos_command:
2424
commands:
25-
- echo 1 > bootflash:/{{ nxos_vars.switch2_file1 }}
26-
- echo 1 > bootflash:/{{ nxos_vars.switch2_file2 }}
27-
- echo 1 > bootflash:/{{ nxos_vars.switch2_file3 }}
28-
- echo 1 > bootflash:/{{ nxos_vars.switch2_file4 }}
25+
- echo 1 > bootflash: /{{ nxos_vars.switch2_file1 }}
26+
- echo 1 > bootflash: /{{ nxos_vars.switch2_file2 }}
27+
- echo 1 > bootflash: /{{ nxos_vars.switch2_file3 }}
28+
- echo 1 > bootflash: /{{ nxos_vars.switch2_file4 }}

playbooks/roles/dcnm_bootflash/dcnm_hosts.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@ all:
33
ansible_user: "admin"
44
ansible_password: "password-secret"
55
ansible_python_interpreter: python
6-
ansible_httpapi_validate_certs: False
7-
ansible_httpapi_use_ssl: True
6+
ansible_httpapi_validate_certs: false
7+
ansible_httpapi_use_ssl: true
88
children:
99
dcnm:
1010
vars:

playbooks/roles/dcnm_bootflash/dcnm_tests.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
# setup and uncomment the testcase you want to run.
99
#
1010
- hosts: dcnm
11+
name: Play for dcnm
1112
gather_facts: false
1213
connection: ansible.netcommon.httpapi
1314

playbooks/roles/dcnm_fabric/dcnm_hosts.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@ all:
33
ansible_user: "admin"
44
ansible_password: "password-secret"
55
ansible_python_interpreter: python
6-
ansible_httpapi_validate_certs: False
7-
ansible_httpapi_use_ssl: True
6+
ansible_httpapi_validate_certs: false
7+
ansible_httpapi_use_ssl: true
88
children:
99
dcnm:
1010
vars:
@@ -15,6 +15,6 @@ all:
1515
nxos:
1616
hosts:
1717
n9k-hosta.example.com:
18-
ansible_connection: ansible.netcommon.network_cli
18+
ansible_connection: ansible.netcommon.network_cli
1919
ansible_network_os: cisco.nxos.nxos
2020
ansible_ssh_port: 22

playbooks/roles/dcnm_fabric/dcnm_tests.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
# Fabric Controller -> Admin -> System Settings -> Feature Management
1616
# "Fabric Builder" is checked.
1717
- hosts: dcnm
18+
name: Play for dcnm
1819
gather_facts: false
1920
connection: ansible.netcommon.httpapi
2021

playbooks/roles/dcnm_image_policy/dcnm_hosts.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@ all:
33
ansible_user: "admin"
44
ansible_password: "password-secret"
55
ansible_python_interpreter: python
6-
ansible_httpapi_validate_certs: False
7-
ansible_httpapi_use_ssl: True
6+
ansible_httpapi_validate_certs: false
7+
ansible_httpapi_use_ssl: true
88
children:
99
dcnm:
1010
vars:
@@ -15,6 +15,6 @@ all:
1515
nxos:
1616
hosts:
1717
n9k-hosta.example.com:
18-
ansible_connection: ansible.netcommon.network_cli
18+
ansible_connection: ansible.netcommon.network_cli
1919
ansible_network_os: cisco.nxos.nxos
2020
ansible_ssh_port: 22

playbooks/roles/dcnm_image_policy/dcnm_tests.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
# setup and uncomment the testcase you want to run.
99
#
1010
- hosts: dcnm
11+
name: Play for dcnm
1112
gather_facts: false
1213
connection: ansible.netcommon.httpapi
1314

playbooks/roles/dcnm_image_upgrade/dcnm_hosts.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@ all:
44
ansible_password: "password-ndfc"
55
switch_password: "password-switch"
66
ansible_python_interpreter: python
7-
ansible_httpapi_validate_certs: False
8-
ansible_httpapi_use_ssl: True
7+
ansible_httpapi_validate_certs: false
8+
ansible_httpapi_use_ssl: true
99
children:
1010
ndfc:
1111
vars:

playbooks/roles/dcnm_image_upgrade/dcnm_tests.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
# setup and uncomment the testcase you want to run.
99
#
1010
- hosts: dcnm
11+
name: Play for dcnm
1112
gather_facts: false
1213
connection: ansible.netcommon.httpapi
1314

0 commit comments

Comments
 (0)