fix: fp_image_tag now required

InformaticsMatters · Nov 4, 2024 · d08bc91 · d08bc91
1 parent 19f1453
commit d08bc91
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/roles/player/defaults/main.yaml b/roles/player/defaults/main.yaml
@@ -31,7 +31,7 @@ fp_parameter_file: parameters.yaml
 # Details of the fragmentation player container image
 fp_image_registry: docker.io
 fp_image_name: informaticsmatters/fragmentor-player
-fp_image_tag: '2.0.4'
+fp_image_tag: SetMe
 
 # The nextflow version to run.
 # The player image generally contains the 'latest' nextflow version.

diff --git a/roles/player/tasks/main.yaml b/roles/player/tasks/main.yaml
@@ -15,14 +15,20 @@
 - name: Assert authentication
   assert:
     that:
-    - k8s_auth_host|length > 0
-    - k8s_auth_api_key|length > 0
+    - k8s_auth_host | length > 0
+    - k8s_auth_api_key | length > 0
 
 - name: Assert kubeconfig file is named
   assert:
     that:
-    - fp_kubeconfig_file|length > 0
-    - fp_kubeconfig_file!='SetMe'
+    - fp_kubeconfig_file | length > 0
+    - fp_kubeconfig_file != 'SetMe'
+
+- name: Assert player image tag is named
+  assert:
+    that:
+    - fp_image_tag | length > 0
+    - fp_image_tag != 'SetMe'
 
 # Discover the hostname (an IP address) of the kubernetes cluster
 # control plane. We do this to set a host alias in the Player Pod