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-3417: capi/aws: support ignition proxy and CA bundle #8170
CORS-3417: capi/aws: support ignition proxy and CA bundle #8170
Conversation
@r4f4: This pull request references CORS-3417 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 story 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. |
1eb7d85
to
6ddab19
Compare
Update: rebased on top of master. Waiting on #8142 to run the proxy CI job. |
/test altinfra-e2e-aws-ovn-proxy |
/hold cancel |
Make sure ignition configuration respects proxy/noproxy and CA bundle settings.
6ddab19
to
14fa8cc
Compare
Update: fixed merge conflicts. |
https://github.com/kubernetes-sigs/cluster-api-provider-aws/pull/4750/files introduced support for proxies in CAPA, so we can specify the Proxy and CA directly in CAPA rather than ignition |
Isn't this exactly what I'm doing or did I miss something? |
/approve |
[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 |
@r4f4: The following tests 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. |
/test images okd-images |
9a8b820
into
openshift:master
[ART PR BUILD NOTIFIER] This PR has been included in build ose-installer-altinfra-container-v4.16.0-202404071345.p0.g9a8b820.assembly.stream.el8 for distgit ose-installer-altinfra. |
Depends on #8169
/hold