Merge pull request #770 from lcarva/fix-yamllint

Fix yamllint
konflux-ci · Jan 30, 2024 · 970ca0e · 970ca0e
2 parents a316c8e + 61a4f96
commit 970ca0e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/.tekton/tasks/yaml-lint.yaml b/.tekton/tasks/yaml-lint.yaml
@@ -40,7 +40,7 @@ spec:
       script: |
         #!/bin/bash
         for task in $(find task -name '*.yaml'); do
-          if yq '.spec.steps[] | .script' $task | grep -q 'params\.'; then
+          if yq '.spec?.steps[] | .script' $task | grep -q 'params\.'; then
             FAILED_TASKS="$FAILED_TASKS $task"
           fi
         done

diff --git a/task/tkn-bundle/0.1/tkn-bundle.yaml b/task/tkn-bundle/0.1/tkn-bundle.yaml
@@ -44,6 +44,8 @@ spec:
     env:
     - name: CONTEXT
       value: $(params.CONTEXT)
+    - name: STEPS_IMAGE
+      value: $(params.STEPS_IMAGE)
     script: |
       #!/bin/env bash
       set -o errexit
@@ -100,9 +102,9 @@ spec:
         fi
       done
 
-      if [[ -n "$(params.STEPS_IMAGE)" ]]; then
+      if [[ -n "${STEPS_IMAGE}" ]]; then
         for f in "${FILES[@]}"; do
-          yq e '(.spec.steps[] | select(has("image")).image) = "$(params.STEPS_IMAGE)"' -i $f
+          yq e '(.spec.steps[] | select(has("image")).image) = "env(STEPS_IMAGE)"' -i $f
         done
       fi