Linting

geant/gap_ansible/roles/iptrunk_checks/tasks/check_optical_levels.yaml

@@ -16,13 +16,7 @@
     - name: Populate the lanes data for "{{ ae_member.interface_name }}"
       no_log: "{{ my_nolog }}"
       ansible.builtin.set_fact:
-        lanes: "{{ lanes + [{
-          'lane_id': iface_opt_lane['lane-index'][0]['data'],
-          'laser_current': iface_opt_lane['laser-bias-current'][0]['data'],
-          'laser_tx_mw': iface_opt_lane['laser-output-power'][0]['data'],
-          'laser_tx_dbm': iface_opt_lane['laser-output-power-dbm'][0]['data'],
-          'laser_rx_mw': iface_opt_lane['laser-rx-optical-power'][0]['data'],
-          'laser_rx_dbm': iface_opt_lane['laser-rx-optical-power-dbm'][0]['data'] }] }}"
+        lanes: "{{ lanes + [{'lane_id': iface_opt_lane['lane-index'][0]['data'], 'laser_current': iface_opt_lane['laser-bias-current'][0]['data'], 'laser_tx_mw': iface_opt_lane['laser-output-power'][0]['data'], 'laser_tx_dbm': iface_opt_lane['laser-output-power-dbm'][0]['data'], 'laser_rx_mw': iface_opt_lane['laser-rx-optical-power'][0]['data'], 'laser_rx_dbm': iface_opt_lane['laser-rx-optical-power-dbm'][0]['data']}] }}"
 
       vars:
         obj: "{{ optical_status.stdout_lines[0]['interface-information'][0]['physical-interface'][0]['optics-diagnostics'][0] }}"

geant/gap_ansible/roles/iptrunk_checks/tasks/main.yml

@@ -124,9 +124,10 @@
   loop_control:
     loop_var: ae_member
 
-- name: Write result to the "{{ opt_checks_file}}" file
+- name: Write result to the "{{ opt_checks_file }}" file
   when: check == "optical_pre" or check == "optical_post"
   ansible.builtin.copy:
+    mode: '0644'
     content: "{{ ae_result | to_nice_yaml }}"
     dest: "{{ opt_checks_file }}"