[TOPSAIL] Appraise the linter

openshift-psap · Feb 16, 2025 · cd5022d · cd5022d
1 parent ddbc0a1
commit cd5022d
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 6 deletions.
diff --git a/projects/fine_tuning/toolbox/fine_tuning_ray_fine_tuning_job/tasks/main.yml b/projects/fine_tuning/toolbox/fine_tuning_ray_fine_tuning_job/tasks/main.yml
@@ -292,7 +292,7 @@
   - name: Make the Ray ServiceAccount privileged
     command:
       oc adm policy add-scc-to-user privileged \
-         -z {{ ray_cluster_service_account_name_cmd.stdout}}
+         -z {{ ray_cluster_service_account_name_cmd.stdout }}
          -n {{ fine_tuning_ray_fine_tuning_job_namespace }}
     when: fine_tuning_ray_fine_tuning_job_use_secondary_nic | default('', true) | length > 0
 

diff --git a/projects/fine_tuning/toolbox/fine_tuning_run_fine_tuning_job/tasks/main.yml b/projects/fine_tuning/toolbox/fine_tuning_run_fine_tuning_job/tasks/main.yml
@@ -1,7 +1,7 @@
 ---
 - name: Store the flag for rdma use
   set_fact:
-    rdma_nics: "{{ fine_tuning_run_fine_tuning_job_use_secondary_nic and fine_tuning_run_fine_tuning_job_use_secondary_nic | select('search', 'rdma') | list}}"
+    rdma_nics: "{{ fine_tuning_run_fine_tuning_job_use_secondary_nic and fine_tuning_run_fine_tuning_job_use_secondary_nic | select('search', 'rdma') | list }}"
 
 - name: Create the src directory
   file:
@@ -34,7 +34,7 @@
     {{ fine_tuning_job_fms_config_template }}{%-
 elif fine_tuning_run_fine_tuning_job_workload == 'ilab' -%}
     {{ fine_tuning_job_ilab_config_template }}
-{%- endif %}"
+{%- endif %}" # noqa: jinja[spacing]
     dest: "{{ artifact_extra_logs_dir }}/src/config_base.yaml"
     mode: '0400'
 

diff --git a/projects/jump_ci/toolbox/jump_ci_ensure_lock/tasks/main.yml b/projects/jump_ci/toolbox/jump_ci_ensure_lock/tasks/main.yml
@@ -43,4 +43,4 @@
 
 - name: Fail if the owner is different
   when: lock_owner_content_cmd.stdout | trim != jump_ci_ensure_lock_owner
-  fail: msg="Unexpected lock owner. Found '{{ lock_owner_content_cmd.stdout}}', expected '{{ jump_ci_ensure_lock_owner }}'"
+  fail: msg="Unexpected lock owner. Found '{{ lock_owner_content_cmd.stdout }}', expected '{{ jump_ci_ensure_lock_owner }}'"
diff --git a/projects/jump_ci/toolbox/jump_ci_prepare_topsail/tasks/main.yml b/projects/jump_ci/toolbox/jump_ci_prepare_topsail/tasks/main.yml
@@ -73,7 +73,7 @@
         refs/pull/{{ jump_ci_prepare_topsail_pr_number }}/merge
       {%- else -%}
         main
-      {%- endif -%}"
+      {%- endif -%}" # noqa: jinja[spacing]
 
 - name: Fetch TOPSAIL repository
   block:

diff --git a/projects/jump_ci/toolbox/jump_ci_retrieve_artifacts/tasks/main.yml b/projects/jump_ci/toolbox/jump_ci_retrieve_artifacts/tasks/main.yml
@@ -9,7 +9,7 @@
 
 - name: Compute the path of the directory to retrieve
   set_fact:
-    remote_dir_path: "{% if cluster_lock_dir is defined %}{{ cluster_lock_dir }}{% else %}{{ '/tmp/topsail_'+jump_ci_retrieve_artifacts_cluster}}{% endif %}/{{ jump_ci_retrieve_artifacts_remote_dir }}"
+    remote_dir_path: "{% if cluster_lock_dir is defined %}{{ cluster_lock_dir }}{% else %}{{ '/tmp/topsail_' + jump_ci_retrieve_artifacts_cluster }}{% endif %}/{{ jump_ci_retrieve_artifacts_remote_dir }}"
 
 - name: Get the size of the directory to retrieve
   command: