USHIFT-6986: Add retry and error handling to cfssl downloads in CNCF …#6698
Conversation
…conformance The cfssl binary downloads in configure-pri.sh intermittently fail due to transient network issues, causing silent checksum mismatches because curl saves error responses as the output file. Add --fail to reject HTTP errors, --retry with --retry-all-errors for resilience, and -S to surface errors in logs for debugging.
openshift-ci-robot
added
the
jira/valid-reference
|
@pacevedom: This pull request references USHIFT-6986 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the bug to target the "4.19.z" version, but no target version was set. DetailsIn response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository: openshift/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
openshift-ci
Bot
added
the
approved
|
/retest |
|
/test e2e-aws-tests-bootc |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ggiguash, pacevedom The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/retest |
1 similar comment
|
/retest |
|
/override ci/prow/e2e-aws-tests-bootc The changes in this PR only impact cncf conformance tests, which are not present in the failures seen above, therefore its an unrelated error. |
|
@pacevedom: Overrode contexts on behalf of pacevedom: ci/prow/e2e-aws-tests-bootc, ci/prow/e2e-aws-tests-bootc-arm DetailsIn response to this:
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. |
|
/verified by CI |
openshift-ci-robot
added
the
verified
|
@pacevedom: This PR has been marked as verified by DetailsIn response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
|
/label jira/valid-bug |
openshift-ci
Bot
added
jira/valid-bug
|
@pacevedom: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. I understand the commands that are listed here. |
|
/retest |
openshift-merge-bot
Bot
merged commit 9552c3d
into
openshift:release-4.19
|
/cherry-pick release-4.18 |
|
@pacevedom: new pull request created: #6726 DetailsIn response to this:
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. |
5 participants
…conformance
The cfssl binary downloads in configure-pri.sh intermittently fail due to transient network issues, causing silent checksum mismatches because curl saves error responses as the output file.
Add --fail to reject HTTP errors, --retry with --retry-all-errors for resilience, and -S to surface errors in logs for debugging.