diff options
| author | Chris Evich <cevich@redhat.com> | 2021-04-26 12:54:30 -0400 |
|---|---|---|
| committer | Chris Evich <cevich@redhat.com> | 2021-04-26 14:12:42 -0400 |
| commit | 17c82caac47a3e7f4b3dd22997bb6fb4cd6ffdb3 (patch) | |
| tree | 6e25fdfbb4fd7e146274141109b2654d32b0e36c | |
| parent | 476c76f580d5cd092ff958765af36857b2a68d6c (diff) | |
| download | podman-17c82caac47a3e7f4b3dd22997bb6fb4cd6ffdb3.tar.gz podman-17c82caac47a3e7f4b3dd22997bb6fb4cd6ffdb3.tar.bz2 podman-17c82caac47a3e7f4b3dd22997bb6fb4cd6ffdb3.zip | |
Fix logic for pushing stable multi-arch images
The intention is to only push an image if there is ***NOT*** an existing
tag. The original logic for this condition was inverted.
Also, improve radability of the `{container,podman}_push=true`
statements.
Signed-off-by: Chris Evich <cevich@redhat.com>
| -rw-r--r-- | .github/workflows/multi-arch-build.yaml | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/.github/workflows/multi-arch-build.yaml b/.github/workflows/multi-arch-build.yaml index 41ef3f3a6..e4ab88544 100644 --- a/.github/workflows/multi-arch-build.yaml +++ b/.github/workflows/multi-arch-build.yaml @@ -77,7 +77,7 @@ jobs: docker://$PODMAN_QUAY_REGISTRY/stable | \ jq -r '.Tags[]') PUSH="false" - if fgrep -qx "$VERSION" <<<"$ALLTAGS"; then + if ! fgrep -qx "$VERSION" <<<"$ALLTAGS"; then PUSH="true" fi @@ -85,7 +85,7 @@ jobs: # Only push if version tag does not exist if [[ "$PUSH" == "true" ]]; then echo "Will push $FQIN" - echo "::set-output name=podman_push::${PUSH}" + echo "::set-output name=podman_push::true" echo "::set-output name=podman_fqin::${FQIN}" else echo "Not pushing, $FQIN already exists." @@ -97,7 +97,7 @@ jobs: docker://$CONTAINERS_QUAY_REGISTRY/podman | \ jq -r '.Tags[]') PUSH="false" - if fgrep -qx "$VERSION" <<<"$ALLTAGS"; then + if ! fgrep -qx "$VERSION" <<<"$ALLTAGS"; then PUSH="true" fi @@ -105,7 +105,7 @@ jobs: # Only push if version tag does not exist if [[ "$PUSH" == "true" ]]; then echo "Will push $FQIN" - echo "::set-output name=containers_push::${PUSH}" + echo "::set-output name=containers_push::true" echo "::set-output name=containers_fqin::$FQIN" else echo "Not pushing, $FQIN already exists." |