USHIFT-6282: 'brew download-build' command fails#5679
USHIFT-6282: 'brew download-build' command fails#5679openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
Conversation
|
/test e2e-aws-tests-cache |
|
/test e2e-aws-tests-cache |
1 similar comment
|
/test e2e-aws-tests-cache |
|
/test e2e-aws-tests-cache-arm |
|
@agullon: This pull request references USHIFT-6282 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.21.0" 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. |
4480532 to
d52101d
Compare
|
@agullon: This pull request references USHIFT-6282 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.21.0" 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. |
| local pkg_release="${version_and_release#*-}" | ||
|
|
||
| for current_arch in ${arch} noarch; do | ||
| local base_url="https://download-01.beak-001.prod.iad2.dc.redhat.com/rhel-9/brew/packages/microshift/${pkg_version}/${pkg_release}/${current_arch}/" |
There was a problem hiding this comment.
Wondering if this URL is fixed or there are any other instances that might serve the same files?
| local pkg_release="${version_and_release#*-}" | ||
|
|
||
| for current_arch in ${arch} noarch; do | ||
| local base_url="https://download-01.beak-001.prod.iad2.dc.redhat.com/rhel-9/brew/packages/microshift/${pkg_version}/${pkg_release}/${current_arch}/" |
There was a problem hiding this comment.
can we make the rhel-9 configurable so that it would be easy later when new rhel versions are introduced ?
There was a problem hiding this comment.
yes, I missed that, I'll do it
There was a problem hiding this comment.
There are no references in the repo pointing to different RHEL 9 and RHEL 10. Neither there are MicroShift RPMs published for RHEL 10.
I think this should be addressed in a separate PR at the same time as other RHEL 9 references.
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: agullon, kasturinarra 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 |
|
/verified by ci |
|
@ggiguash: 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. |
|
/retest |
|
/retest |
|
/retest |
2 similar comments
|
/retest |
|
/retest |
|
@agullon: all tests passed! 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. |
|
/cherrypick release-4.20 |
|
@agullon: new pull request created: #5694 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. |
Add a fallback mechanism to download the RPMs using
curlcommand with-koption to ignore invalid and self-signed certificate errors.