New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CORS-3250: images/altinfra: add etcd/kas binaries from containers #8309
Conversation
@r4f4: This pull request references CORS-3250 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 sub-task to target the "4.16.0" version, but no target version was set. In 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. |
Depends on #8196 |
With this we'll only support native installs, that is, it won't be possible yet to deploy an arm64 cluster from an amd64 one. |
In the build log for the
So the binaries are there. Let's see if they are working: /test altinfra-e2e-aws-ovn |
I'm going to send a fix to shellcheck soon. |
/hold |
Only download the binaries if the existing binaries' version is lower than the minimum version (currently 1.28.0). This change will be needed when we copy etcd/kas binaries from existing container images which build them from source.
When building the altinfra image for inclusion in the release payload, we cannot download any binaries as it is currently done for etcd/kas. Instead let's copy those binaries from their existing container images.
shellcheck fixed. /hold cancel |
Actually let's wait to sync this with ART. /hold |
/approve for when we sync with ART. Along with this, I think we may want to always build CAPI providers #8273? |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: patrickdillon The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
I can set |
@r4f4: The following test failed, say
Full PR test history. Your PR dashboard. 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/test-infra repository. I understand the commands that are listed here. |
/hold cancel |
523f573
into
openshift:master
[ART PR BUILD NOTIFIER] This PR has been included in build ose-installer-altinfra-container-v4.16.0-202404261114.p0.g523f573.assembly.stream.el9 for distgit ose-installer-altinfra. |
When building the altinfra image for inclusion in the release payload, we cannot download any binaries as it is currently done for etcd/kas. Instead let's copy those binaries from their existing container images.