Open mkowalski opened 6 months ago
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please assign bfournie for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.
/cc @cybertron
/lgtm
Seems fine. I assume this is related to deploying single stack clusters on dual stack envs?
/hold Until manual testing proves that all works
Yes, the issue with current state is that we do not generate a valid install-config yaml for a single-stack cluster with dual-stack hosts. For such a scenario (allowed by syntax of config.sh) we put dual-stack VIPs in install-config.yaml but only single cluster network. This is illegal and does not pass the basic validations.
With this fix I want to be explicit as the VIP configuration goes. I am still not allowing to have single VIP setup with dual-stack cluster but extending feature set is not my goal, all I want first is to remove a scenario that prevents installation when in general it should work
@mkowalski: The following tests failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
ci/prow/e2e-metal-ipi-ovn-dualstack | 08803c9f491882e3c6e7b576d6a009f06ddab444 | link | false | /test e2e-metal-ipi-ovn-dualstack |
ci/prow/e2e-metal-ipi-serial-ovn-ipv6 | 08803c9f491882e3c6e7b576d6a009f06ddab444 | link | false | /test e2e-metal-ipi-serial-ovn-ipv6 |
Full PR test history. Your PR dashboard.
Skipping CI for Draft Pull Request. If you want CI signal for your change, please convert it to an actual PR. You can still manually trigger a test run with
/test all