Skip to content

Commit e1dcdd2

Browse files
mayastor-borstiagolobocastro
andcommitted
chore(bors): merge pull request #702
702: ci(pin): don't check if "latest" already exists r=tiagolobocastro a=tiagolobocastro Since it will exist and it's not an error, only if the pinned one exists is an error Co-authored-by: Tiago Castro <[email protected]>
2 parents 155492f + f9bfc1f commit e1dcdd2

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

scripts/helm/pin.sh

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -322,10 +322,13 @@ if [ ! "${UNPIN_CHART:-}" = "true" ] && [[ "$PINNED_VERSION" != "$PINNED_VERSION
322322
exit 0
323323
fi
324324
fi
325-
# Ensure the pinned version really doesn't exist already
326-
exists=$(helm_oci_chart_exists "$PINNED_OCI_CHART" "$PINNED_VERSION")
327-
if [[ "$exists" = "true" ]]; then
328-
log_fatal "Pinned chart $PINNED_VERSION already exists!"
325+
326+
if [ ! "${UNPIN_CHART:-}" = "true" ]; then
327+
# Ensure the pinned version really doesn't exist already
328+
exists=$(helm_oci_chart_exists "$PINNED_OCI_CHART" "$PINNED_VERSION")
329+
if [[ "$exists" = "true" ]]; then
330+
log_fatal "Pinned chart $PINNED_VERSION already exists!"
331+
fi
329332
fi
330333

331334
if [[ "${DRY_RUN:-}" = "true" ]]; then

0 commit comments

Comments
 (0)