Ansible code bot recommendations

roles/run/tasks/health_checks/eos.yaml

@@ -1,7 +1,7 @@
 ---
 - name: Parse bgp summary
   ansible.utils.cli_parse:
-    command: "show ip bgp summary"
+    command: show ip bgp summary
     parser:
       name: ansible.netcommon.content_templates
     set_fact: bgp_health

roles/run/tasks/health_checks/ios.yaml

@@ -1,7 +1,7 @@
 ---
 - name: Parse bgp summary
   ansible.utils.cli_parse:
-    command: "show bgp summary"
+    command: show bgp summary
     parser:
       name: ansible.netcommon.content_templates
     set_fact: bgp_health

roles/run/tasks/health_checks/iosxr.yaml

@@ -1,7 +1,7 @@
 ---
 - name: Parse bgp summary
   ansible.utils.cli_parse:
-    command: "show bgp summary"
+    command: show bgp summary
     parser:
       name: ansible.netcommon.content_templates
     set_fact: bgp_health

roles/run/tasks/health_checks/junos.yaml

@@ -1,12 +1,11 @@
 ---
 - name: Parse bgp summary
   ansible.utils.cli_parse:
-    command: "show bgp summary"
+    command: show bgp summary
     parser:
       name: ansible.netcommon.content_templates
     set_fact: bgp_health
 
-
 - name: Show Summary facts
   ansible.builtin.debug:
     msg: "{{ bgp_health }}"

roles/run/tasks/health_checks/nxos.yaml

@@ -1,7 +1,7 @@
 ---
 - name: Parse bgp summary
   ansible.utils.cli_parse:
-    command: "show ip bgp summary"
+    command: show ip bgp summary
     parser:
       name: ansible.netcommon.content_templates
     set_fact: bgp_health

roles/run/tasks/health_checks/vyos.yaml

@@ -1,7 +1,7 @@
 ---
 - name: Parse bgp summary
   ansible.utils.cli_parse:
-    command: "show ip bgp summary"
+    command: show ip bgp summary
     parser:
       name: ansible.netcommon.content_templates
     set_fact: bgp_health

roles/run/tasks/includes/configure.yaml

@@ -3,4 +3,4 @@
   ansible.builtin.include_role:
     name: network.base.resource_manager
   vars:
-    operation: 'configure'
+    operation: configure

roles/run/tasks/includes/deploy.yaml

@@ -6,5 +6,5 @@
   ansible.builtin.include_role:
     name: network.base.resource_manager
   vars:
-    operation: 'deploy'
+    operation: deploy
     resources: "{{ bgp_resources }}"

roles/run/tasks/includes/detect.yaml

@@ -6,6 +6,6 @@
   ansible.builtin.include_role:
     name: network.base.resource_manager
   vars:
-    operation: 'detect'
-    state: 'replaced'
+    operation: detect
+    state: replaced
     resources: "{{ bgp_resources }}"

roles/run/tasks/includes/gather.yaml

@@ -6,5 +6,5 @@
   ansible.builtin.include_role:
     name: network.base.resource_manager
   vars:
-    operation: 'gather'
+    operation: gather
     resources: "{{ bgp_resources }}"

roles/run/tasks/includes/health_check.yaml

@@ -4,9 +4,9 @@
 
 - name: Set health checks fact
   ansible.builtin.set_fact:
-     health_checks: "{{ bgp_health | network.bgp.health_check_view(operation) }}"
+    health_checks: "{{ bgp_health | network.bgp.health_check_view(operation) }}"
 
 - name: BGP health checks
   ansible.builtin.debug:
-     var: health_checks
+    var: health_checks
   failed_when: "'unsuccessful' == health_checks.status"

roles/run/tasks/includes/health_checks/eos.yaml

@@ -1,7 +1,7 @@
 ---
 - name: Parse bgp summary
   ansible.utils.cli_parse:
-    command: "show ip bgp summary"
+    command: show ip bgp summary
     parser:
       name: ansible.netcommon.content_templates
     set_fact: bgp_health

roles/run/tasks/includes/health_checks/ios.yaml

@@ -1,7 +1,7 @@
 ---
 - name: Parse bgp summary
   ansible.utils.cli_parse:
-    command: "show bgp summary"
+    command: show bgp summary
     parser:
       name: ansible.netcommon.content_templates
     set_fact: bgp_health

roles/run/tasks/includes/health_checks/iosxr.yaml

@@ -1,7 +1,7 @@
 ---
 - name: Parse bgp summary
   ansible.utils.cli_parse:
-    command: "show bgp summary"
+    command: show bgp summary
     parser:
       name: ansible.netcommon.content_templates
     set_fact: bgp_health

roles/run/tasks/includes/health_checks/junos.yaml

@@ -1,12 +1,11 @@
 ---
 - name: Parse bgp summary
   ansible.utils.cli_parse:
-    command: "show bgp summary"
+    command: show bgp summary
     parser:
       name: ansible.netcommon.content_templates
     set_fact: bgp_health
 
-
 - name: Show Summary facts
   ansible.builtin.debug:
     msg: "{{ bgp_health }}"

roles/run/tasks/includes/health_checks/nxos.yaml

@@ -1,7 +1,7 @@
 ---
 - name: Parse bgp summary
   ansible.utils.cli_parse:
-    command: "show ip bgp summary"
+    command: show ip bgp summary
     parser:
       name: ansible.netcommon.content_templates
     set_fact: bgp_health

roles/run/tasks/includes/health_checks/vyos.yaml

@@ -1,7 +1,7 @@
 ---
 - name: Parse bgp summary
   ansible.utils.cli_parse:
-    command: "show ip bgp summary"
+    command: show ip bgp summary
     parser:
       name: ansible.netcommon.content_templates
     set_fact: bgp_health

roles/run/tasks/includes/list.yaml

@@ -3,4 +3,4 @@
   ansible.builtin.include_role:
     name: network.base.resource_manager
   vars:
-    operation: 'list'
+    operation: list

roles/run/tasks/includes/persist.yaml

@@ -6,6 +6,6 @@
   ansible.builtin.include_role:
     name: network.base.resource_manager
   vars:
-    operation: 'persist'
+    operation: persist
     resources: "{{ bgp_resources }}"
     ignore_empty: "{{ persist_empty | default(true) }}"

roles/run/tasks/includes/remediate.yaml

@@ -6,6 +6,6 @@
   ansible.builtin.include_role:
     name: network.base.resource_manager
   vars:
-    operation: 'remediate'
-    state: 'replaced'
+    operation: remediate
+    state: replaced
     resources: "{{ bgp_resources }}"

roles/run/tasks/includes/resources.yaml

@@ -2,14 +2,14 @@
 - name: Set resources
   ansible.builtin.set_fact:
     bgp_resources:
-      - 'bgp_global'
-      - 'bgp_address_family'
-      - 'bgp_neighbor_address_family'
+      - bgp_global
+      - bgp_address_family
+      - bgp_neighbor_address_family
   when: ansible_network_os == 'cisco.nxos.nxos'
 
 - name: Set resources
   ansible.builtin.set_fact:
     bgp_resources:
-      - 'bgp_global'
-      - 'bgp_address_family'
+      - bgp_global
+      - bgp_address_family
   when: ansible_network_os != 'cisco.nxos.nxos'

roles/run/tasks/main.yml

@@ -1,5 +1,5 @@
 ---
-- name: include tasks
+- name: Include tasks
   ansible.builtin.include_tasks: includes/{{operation.name}}.yaml
   loop: "{{operations}}"
   loop_control: