Skip to content

[bindata] catch skopeo copy failures#1544

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
olliewalsh:bindata_quay_issues
Jul 25, 2025
Merged

[bindata] catch skopeo copy failures#1544
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
olliewalsh:bindata_quay_issues

Conversation

@olliewalsh

Copy link
Copy Markdown
Contributor

If skopeo copy failed (e.g quay.io wobbling) it would be silently ignored and result in missing image references

If skopeo copy failed (e.g quay.io wobbling) it would be silently ignored
and result in missing image references
@olliewalsh olliewalsh requested review from dprince and stuggi July 24, 2025 11:10
@olliewalsh

Copy link
Copy Markdown
Contributor Author

If I change the skopeo command to false:

+ for BUNDLE in $(hack/pin-bundle-images.sh | tr "," " ")
+ set +e
+ n=0
+ RC=1
+ :
+ false
+ RC=1
+ '[' 1 -eq 0 -o 0 -ge 5 ']'
+ n=1
+ echo 'Command failed. retrying ... 1/5'
Command failed. retrying ... 1/5
+ sleep 15
+ :
+ false
+ RC=1
+ '[' 1 -eq 0 -o 1 -ge 5 ']'
+ n=2
+ echo 'Command failed. retrying ... 2/5'
Command failed. retrying ... 2/5
+ sleep 15
+ :
+ false
+ RC=1
+ '[' 1 -eq 0 -o 2 -ge 5 ']'
+ n=3
+ echo 'Command failed. retrying ... 3/5'
Command failed. retrying ... 3/5
+ sleep 15
+ :
+ false
+ RC=1
+ '[' 1 -eq 0 -o 3 -ge 5 ']'
+ n=4
+ echo 'Command failed. retrying ... 4/5'
Command failed. retrying ... 4/5
+ sleep 15
+ :
+ false
+ RC=1
+ '[' 1 -eq 0 -o 4 -ge 5 ']'
+ n=5
+ echo 'Command failed. retrying ... 5/5'
Command failed. retrying ... 5/5
+ sleep 15
+ :
+ false
+ RC=1
+ '[' 1 -eq 0 -o 5 -ge 5 ']'
+ exit 1
make: *** [Makefile:162: bindata] Error 1

@stuggi stuggi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci

openshift-ci Bot commented Jul 24, 2025

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: olliewalsh, stuggi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@stuggi

stuggi commented Jul 24, 2025

Copy link
Copy Markdown
Contributor

/retest

1 similar comment
@stuggi

stuggi commented Jul 24, 2025

Copy link
Copy Markdown
Contributor

/retest

@stuggi

stuggi commented Jul 24, 2025

Copy link
Copy Markdown
Contributor

/cherry-pick 18.0-fr3

@openshift-cherrypick-robot

Copy link
Copy Markdown

@stuggi: once the present PR merges, I will cherry-pick it on top of 18.0-fr3 in a new PR and assign it to you.

Details

In response to this:

/cherry-pick 18.0-fr3

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@olliewalsh

Copy link
Copy Markdown
Contributor Author

/test openstack-operator-build-deploy-kuttl

@openshift-merge-bot openshift-merge-bot Bot merged commit c4bce29 into openstack-k8s-operators:main Jul 25, 2025
10 checks passed
@openshift-cherrypick-robot

Copy link
Copy Markdown

@stuggi: new pull request created: #1546

Details

In response to this:

/cherry-pick 18.0-fr3

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants