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-17196: Fix none existens of static MAC adr. in case of specific network devices on s390x. #6087
MGMT-17196: Fix none existens of static MAC adr. in case of specific network devices on s390x. #6087
Conversation
@AmadeusPodvratnik: This pull request references MGMT-17196 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. |
Hi @AmadeusPodvratnik. Thanks for your PR. I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
54c8bc3
to
d01bcff
Compare
21cbc6b
to
7742f8e
Compare
60975cb
to
bcd286a
Compare
/lgtm |
/ok-to-test |
/approve |
Please rebase squash all commits |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: AmadeusPodvratnik, omertuc 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 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6087 +/- ##
==========================================
- Coverage 68.38% 68.21% -0.18%
==========================================
Files 239 239
Lines 35503 35607 +104
==========================================
+ Hits 24280 24290 +10
- Misses 9115 9212 +97
+ Partials 2108 2105 -3
|
specific network devices on s390x. Signed-off-by: Amadeus Podvratnik <apodvrat@redhat.com> Co-authored-by: Omer Tuchfeld <omertuchfeld@gmail.com> MGMT-17196: Fix none existens of static MAC adr. in case of specific network devices on s390x. Signed-off-by: Amadeus Podvratnik <apodvrat@redhat.com> Update internal/host/hostcommands/install_cmd.go Co-authored-by: Omer Tuchfeld <omertuchfeld@gmail.com> Signed-off-by: Amadeus Podvratnik <apodvrat@redhat.com> Update internal/host/hostcommands/install_cmd.go Co-authored-by: Omer Tuchfeld <omertuchfeld@gmail.com> Update internal/host/hostcommands/install_cmd.go Co-authored-by: Omer Tuchfeld <omertuchfeld@gmail.com> Update internal/host/hostcommands/install_cmd.go Co-authored-by: Omer Tuchfeld <omertuchfeld@gmail.com> Signed-off-by: Amadeus Podvratnik <apodvrat@redhat.com>
bcd286a
to
4e2b253
Compare
/lgtm |
@AmadeusPodvratnik: 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. |
On s390x there are some network configurations where MAC addresses are not static and lead to issues using Assisted Installer, Agent based Installer and HCP (see attached net conf).
For AI and HCP there is a possibility to patch kernel arguments but using the UI there is a separate manual step needed using the API. This will be a bad user experience.
In addition patching the kernel arguments for ABI is not possible.
To solve this, a config override parameter need to be added to the parm file by the user and the ip settings will be automatically passed to the coreos installer regardless what the user configure (DHCP or static IP using nmstate).
See matrix in ticket:
https://issues.redhat.com/browse/MGMT-17196
Doc ticket:
https://issues.redhat.com/browse/MULTIARCH-4537
List all the issues related to this PR
What environments does this code impact?
How was this code tested?
Simple parse test by running unit tests
assisted-test-infra environment
dev-scripts environment
Reviewer's test appreciated
Waiting for CI to do a full test run
Manual (Elaborate on how it was tested)
[] No tests needed
Checklist
docs
, README, etc)Reviewers Checklist