fix(KFLUXBUGS-1215): fix use of PREFETCH_INPUT

- quotes within variable are not handled very well by ssh command and it impacts the buildah-remote task (which is based on this task) we only need to know if prefetch was used, not the actual input. Therefore, we simplify the variable. Signed-off-by: Scott Hebert <[email protected]>
konflux-ci · Apr 18, 2024 · 99423df · 99423df
1 parent c5ea8d8
commit 99423df
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/task/buildah-remote/0.1/buildah-remote.yaml b/task/buildah-remote/0.1/buildah-remote.yaml
@@ -258,6 +258,16 @@ spec:
           mkdir -p "$YUM_REPOS_D_FETCHED"
           cp --no-clobber "$prefetched_repo_for_my_arch" "$YUM_REPOS_D_FETCHED"
         fi
+
+        # quotes within variable are not handled very well
+        # by ssh command and it impacts the buildah-remote task (which is based on this task)
+        # we only need to know if prefetch was used, not the actual
+        # input. Therefore, we simplify the variable.
+        #
+        if [ -n "${PREFETCH_INPUT}" ] ; then
+          PREFETCH_INPUT="prefetched"
+        fi
+
       fi
 
       # if yum repofiles stored in git, copy them to mount point outside the source dir

diff --git a/task/buildah/0.1/buildah.yaml b/task/buildah/0.1/buildah.yaml
@@ -210,6 +210,16 @@ spec:
           mkdir -p "$YUM_REPOS_D_FETCHED"
           cp --no-clobber "$prefetched_repo_for_my_arch" "$YUM_REPOS_D_FETCHED"
         fi
+      
+        # quotes within variable are not handled very well
+        # by ssh command and it impacts the buildah-remote task (which is based on this task)
+        # we only need to know if prefetch was used, not the actual
+        # input. Therefore, we simplify the variable.
+        #
+        if [ -n "${PREFETCH_INPUT}" ] ; then
+          PREFETCH_INPUT="prefetched"
+        fi
+
       fi
 
       # if yum repofiles stored in git, copy them to mount point outside the source dir