Fix onyx group vars (#50)

* use correct var names for group_vars * remove unnecessary onyx overrides from plays
OpenMPDK · Aug 3, 2023 · 7557482 · 7557482
1 parent 73c919c
commit 7557482
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 23 deletions.
diff --git a/group_vars/onyx.yml b/group_vars/onyx.yml
@@ -2,6 +2,5 @@
 
 ansible_connection: network_cli
 ansible_network_os: onyx
-gather_facts: false
-become: true
-become_method: enable
+ansible_become: true
+ansible_become_method: enable
diff --git a/playbooks/configure_vlans.yml b/playbooks/configure_vlans.yml
@@ -83,12 +83,7 @@
 
 - name: Get Onyx LLDP tables
   hosts: onyx
-  connection: network_cli
   gather_facts: false
-  become: true
-  become_method: enable
-  vars:
-    ansible_network_os: onyx
   roles:
     - get_onyx_lldp
 
@@ -104,11 +99,6 @@
 
 - name: Configure Onyx VLANs
   hosts: onyx
-  connection: network_cli
   gather_facts: false
-  become: true
-  become_method: enable
-  vars:
-    ansible_network_os: onyx
   roles:
     - configure_onyx_vlans
diff --git a/playbooks/remove_vlans.yml b/playbooks/remove_vlans.yml
@@ -47,12 +47,7 @@
 
 - name: Get Onyx LLDP tables
   hosts: onyx
-  connection: network_cli
   gather_facts: false
-  become: true
-  become_method: enable
-  vars:
-    ansible_network_os: onyx
   roles:
     - get_onyx_lldp
 
@@ -67,11 +62,6 @@
 
 - name: Remove Onyx allowed VLANs
   hosts: onyx
-  connection: network_cli
   gather_facts: false
-  become: true
-  become_method: enable
-  vars:
-    ansible_network_os: onyx
   roles:
     - remove_onyx_allowed_vlans