🌱 E2E: Adjust Ironic kustomization patch #2308
Conversation
We have removed the IP address from the CA in the source manifests. This breaks the replacement because it was selecting ALL certificates. This commit changes it to only select the ironic-certificate. Signed-off-by: Lennart Jern <[email protected]>
metal3-io-bot
added
the
size/XS
|
/hold |
metal3-io-bot
added
the
do-not-merge/hold
|
/test ? |
|
@lentzi90: The following commands are available to trigger required jobs: The following commands are available to trigger optional jobs: Use 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 kubernetes-sigs/prow repository. |
|
/test metal3-e2e-clusterctl-upgrade-test-main |
|
The test has successfully deployed Ironic from the correct overlay: This is the one that is pointed to the branch in metal3-io/baremetal-operator#2222 in the test commit to check that it works. |
|
The test was successful: https://jenkins.nordix.org/view/Metal3/job/metal3-e2e-clusterctl-upgrade-test-main/111/ |
lentzi90
force-pushed
the
lentzi90/ironic-ca-address
branch
from
2adfcdf to
02eebf2
Compare
|
/test metal3-centos-e2e-integration-test-main |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: kashifest 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 |
metal3-io-bot
added
the
approved
|
/hold cancel |
metal3-io-bot
removed
the
do-not-merge/hold
What this PR does / why we need it:
We have removed the IP address from the CA in the source manifests. This
breaks the replacement because it was selecting ALL certificates. This
commit changes it to only select the ironic-certificate.
See metal3-io/baremetal-operator#2222