Closed adriengentil closed 1 month ago
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
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: adriengentil
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/test edge-unit-test edge-lint edge-e2e-oci-assisted edge-e2e-metal-assisted-external
@adriengentil: This pull request references MGMT-18121 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 task to target the "4.17.0" version, but no target version was set.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 68.61%. Comparing base (
8c9c437
) to head (5c749ae
). Report is 1 commits behind head on master.
/test edge-unit-test edge-lint edge-e2e-oci-assisted edge-e2e-metal-assisted-external
/test edge-e2e-oci-assisted edge-e2e-metal-assisted-external
@adriengentil: This pull request references MGMT-18121 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 task to target the "4.18.0" version, but no target version was set.
@adriengentil: This pull request references MGMT-18121 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 task to target the "4.18.0" version, but no target version was set.
@adriengentil: This pull request references MGMT-18121 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 task to target the "4.18.0" version, but no target version was set.
@adriengentil: This pull request references MGMT-18121 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 task to target the "4.18.0" version, but no target version was set.
/test edge-unit-test edge-lint
@adriengentil: This pull request references MGMT-18121 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 task to target the "4.18.0" version, but no target version was set.
/cc @eifrach
@adriengentil: The following test 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/edge-e2e-oci-assisted | 573f4c0f1279a42cc2828c805c32c956d43f2358 | link | false | /test edge-e2e-oci-assisted |
Full PR test history. Your PR dashboard.
/lgtm
[ART PR BUILD NOTIFIER]
Distgit: ose-agent-installer-api-server This PR has been included in build ose-agent-installer-api-server-container-v4.18.0-202408132015.p0.g52cb256.assembly.stream.el9. All builds following this will include this PR.
A secondary network interface is required in OCI in order to install clusters on iSCSI boot volumes. Extra interfaces in OCI must be statically set, this change adds a script that configures the secondary network interface on OCI during discovery. We ensure that we execute the script only for machines in OCI with an iBFT configuration.
Since the secondary interface is the one that will be part of the machine networks, we will prevent its configuration with DHCP though the kernel arguments.
Unfortunately, we cannot propagate the network configuration set during discovery to the installed system because
--copy-network
won't work with iSCSI boot volumes: https://github.com/coreos/coreos-installer/issues/1389. As an alternative, the user will need to pass extra custom manifests to configure the network on next boot.For the moment, we'll keep this script to configure OCI networking in assisted-service, but in the future we may want Oracle to own this script and inject it at discovery time. This will be refined in a later stage.