diff --git a/tasks/rh-sign-image/README.md b/tasks/rh-sign-image/README.md index 8b5e8edf7..bf4bfcc33 100644 --- a/tasks/rh-sign-image/README.md +++ b/tasks/rh-sign-image/README.md @@ -20,6 +20,9 @@ Task to create internalrequests or pipelineruns to sign snapshot components | signRegistryAccessPath | The relative path in the workspace to a text file that contains a list of repositories that needs registry.access.redhat.com image references to be signed (i.e. requires_terms=true), one repository string per line, e.g. "rhtas/cosign-rhel9". | No | - | +## Changes in 5.0.2 +* fix linting issues + ## Changes in 5.0.1 * The default for `sign.request` is now always `simple-signing-pipeline` instead of being `hacbs-signing-pipeline` if using InternalRequests diff --git a/tasks/rh-sign-image/rh-sign-image.yaml b/tasks/rh-sign-image/rh-sign-image.yaml index c384397fc..aaddbf627 100644 --- a/tasks/rh-sign-image/rh-sign-image.yaml +++ b/tasks/rh-sign-image/rh-sign-image.yaml @@ -4,7 +4,7 @@ kind: Task metadata: name: rh-sign-image labels: - app.kubernetes.io/version: "5.0.1" + app.kubernetes.io/version: "5.0.2" annotations: tekton.dev/pipelines.minVersion: "0.12.1" tekton.dev/tags: release @@ -128,9 +128,9 @@ spec: default_pipeline_image="quay.io/redhat-isv/operator-pipelines-images:released" pipeline_image=$(jq -r --arg default_pipeline_image ${default_pipeline_image} \ - '.sign.pipelineImage // $default_pipeline_image' ${DATA_FILE}) - config_map_name=$(jq -r '.sign.configMapName // "signing-config-map"' ${DATA_FILE}) - defaultPushSourceContainer=$(jq -r '.mapping.defaults.pushSourceContainer' ${DATA_FILE}) + '.sign.pipelineImage // $default_pipeline_image' "${DATA_FILE}") + config_map_name=$(jq -r '.sign.configMapName // "signing-config-map"' "${DATA_FILE}") + defaultPushSourceContainer=$(jq -r '.mapping.defaults.pushSourceContainer' "${DATA_FILE}") if [[ "$(params.pyxisServer)" == "production" ]] then @@ -158,21 +158,20 @@ spec: export PYXIS_CERT_PATH=/tmp/crt export PYXIS_KEY_PATH=/tmp/key - COMPONENTS_LENGTH=$(jq '.components |length' ${SNAPSHOT_PATH}) + COMPONENTS_LENGTH=$(jq '.components |length' "${SNAPSHOT_PATH}") for (( COMPONENTS_INDEX=0; COMPONENTS_INDEX