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
MGMT-17106: Disable ironic vlan creation when static networking config is present #6055
MGMT-17106: Disable ironic vlan creation when static networking config is present #6055
Conversation
Previously the ironic agent would define vlan interfaces when it detected that vlans were present using LLDP This change only allows this type of reconfiguration if the user didn't provide static networking config. If a user does provide static networking config and requires vlans it's expected that they will configure them manually rather than relying on LLDP-based configuration from the ironic agent. Resolves https://issues.redhat.com/browse/MGMT-17106
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: carbonin 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 |
@carbonin: This pull request references MGMT-17106 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 bug 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. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6055 +/- ##
==========================================
+ Coverage 68.39% 68.40% +0.01%
==========================================
Files 239 239
Lines 35454 35457 +3
==========================================
+ Hits 24247 24253 +6
+ Misses 9101 9099 -2
+ Partials 2106 2105 -1
|
/lgtm |
@carbonin: all tests passed! 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. |
[ART PR BUILD NOTIFIER] This PR has been included in build ose-agent-installer-api-server-container-v4.16.0-202403061739.p0.g3864109.assembly.stream.el8 for distgit ose-agent-installer-api-server. |
/cherry-pick release-ocm-2.10 |
@gamli75: new pull request created: #6081 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/test-infra repository. |
Previously the ironic agent would define vlan interfaces when it
detected that vlans were present using LLDP
This change only allows this type of reconfiguration if the user didn't
provide static networking config. If a user does provide static
networking config and requires vlans it's expected that they will
configure them manually rather than relying on LLDP-based configuration
from the ironic agent.
This also requires new ICC code to vendor in the change to accept this
parameter when we create the ironic agent ignition.
List all the issues related to this PR
Resolves https://issues.redhat.com/browse/MGMT-17106
What environments does this code impact?
How was this code tested?
I don't have the environment to test this, unit tests are all I can do for now.
CI will catch any potential regressions and hopefully QE has the kind of environment required to actually test it.
Checklist
docs
, README, etc)Reviewers Checklist