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
[release-ocm-2.8] MGMT-15413: Use same installer binary for all platform types (#5334) #5408
[release-ocm-2.8] MGMT-15413: Use same installer binary for all platform types (#5334) #5408
Conversation
@CrystalChun: This pull request references MGMT-15413 which is a valid jira issue. 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. |
1 similar comment
@CrystalChun: This pull request references MGMT-15413 which is a valid jira issue. 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. |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## release-ocm-2.8 #5408 +/- ##
===================================================
- Coverage 67.50% 67.49% -0.01%
===================================================
Files 218 218
Lines 32721 32721
===================================================
- Hits 22088 22086 -2
- Misses 8641 8642 +1
- Partials 1992 1993 +1
|
0be7f7f
to
a46d1e1
Compare
/cc @filanov |
https://issues.redhat.com/browse/MGMT-15413 --- MGMT-15150: Use same installer binary for all platform types (openshift#5334) * MGMT-15150: Use same installer binary for all platform types There's nothing special about platform:none that requires it to use a different installer binary. This was originally done (in f9b2f3d) to avoid a problem with the openshift-baremetal-install binary not being available for non-x86 targets, but this was resolved by 1d025b8 (MGMT-9206). * Remove platform argument when extracting installer Since we no longer use different installer binaries for different platforms, we no longer need to pass the platform type down. This allows us to clean up a lot of function signatures. This partially reverts commit f9b2f3d.
a46d1e1
to
c5c9b78
Compare
/retest-required |
@CrystalChun: 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. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: CrystalChun, filanov 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 |
602f258
into
openshift:release-ocm-2.8
Cherry-pick of #5334
https://issues.redhat.com/browse/MGMT-15413
There's nothing special about platform:none that requires it to use a different installer binary. This was originally done (in f9b2f3d) to avoid a problem with the openshift-baremetal-install binary not being available for non-x86 targets, but this was resolved by 1d025b8 (MGMT-9206).
Since we no longer use different installer binaries for different platforms, we no longer need to pass the platform type down. This allows us to clean up a lot of function signatures.
This partially reverts commit f9b2f3d.
List all the issues related to this PR
What environments does this code impact?
How was this code tested?
Checklist
docs
, README, etc)Reviewers Checklist