Skip to content

[OSDOCS-7500] multi-architecture IBM Power and IBM Z control planes #71418

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

Merged
merged 1 commit into from
Feb 16, 2024

Conversation

SNiemann15
Copy link
Contributor

@SNiemann15 SNiemann15 commented Feb 9, 2024

@openshift-ci openshift-ci bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Feb 9, 2024
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Feb 9, 2024

🤖 Fri Feb 09 14:52:48 - Prow CI generated the docs preview: https://71418--ocpdocs-pr.netlify.app

Copy link

openshift-ci bot commented Feb 9, 2024

@SNiemann15: 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.

include::_attributes/common-attributes.adoc[]

toc::[]

To create a cluster with multi-architecture compute machines on bare metal, you must have an existing single-architecture bare metal cluster. For more information on bare metal installations, see xref:../../installing/installing_bare_metal/installing-bare-metal.adoc#installing-bare-metal[Installing a user provisioned cluster on bare metal]. You can then add 64-bit ARM compute machines to your {product-title} cluster on bare metal.
To create a cluster with multi-architecture compute machines on bare metal (`x86_64`), {ibm-power-name} (`ppc64le`), or {ibm-z-name} (`s390x`) you must have an existing single-architecture cluster on one of these platforms. Follow the installations procedures for your platform:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left comment directly with Alisha

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Concerned about implications of supported architecture pairs

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@prb112 We have an extra PR from @kelbrown20 with a table to visualize what's possible. See here: #71394
We probably need to further refine the table. Feel free to comment there.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh - that's fantastic!


Before you can add 64-bit ARM nodes to your bare metal cluster, you must upgrade your cluster to one that uses the multi-architecture payload. For more information on migrating to the multi-architecture payload, see xref:../../updating/updating_a_cluster/migrating-to-multi-payload.adoc#migrating-to-multi-payload[Migrating to a cluster with multi-architecture compute machines].
* xref:../../installing/installing_bare_metal/installing-bare-metal.adoc#installing-bare-metal[Installing a user provisioned cluster on bare metal]. You can then add 64-bit ARM compute machines to your {product-title} cluster on bare metal.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* xref:../../installing/installing_bare_metal/installing-bare-metal.adoc#installing-bare-metal[Installing a user provisioned cluster on bare metal]. You can then add 64-bit ARM compute machines to your {product-title} cluster on bare metal.
* xref:../../installing/installing_bare_metal/installing-bare-metal.adoc#installing-bare-metal[Installing a user provisioned cluster on bare metal]. You can then add 64-bit ppc64le or s390x compute machines to your {product-title} cluster on bare metal.

Is that what's intended?

arm isn't supported with power,s390x

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes arm compute node is only supported with bare metal

include::_attributes/common-attributes.adoc[]

toc::[]

To create a cluster with multi-architecture compute machines on bare metal, you must have an existing single-architecture bare metal cluster. For more information on bare metal installations, see xref:../../installing/installing_bare_metal/installing-bare-metal.adoc#installing-bare-metal[Installing a user provisioned cluster on bare metal]. You can then add 64-bit ARM compute machines to your {product-title} cluster on bare metal.
To create a cluster with multi-architecture compute machines on bare metal (`x86_64`), {ibm-power-name} (`ppc64le`), or {ibm-z-name} (`s390x`) you must have an existing single-architecture cluster on one of these platforms. Follow the installations procedures for your platform:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@prb112 Does adding "either" make it more clear?

Suggested change
To create a cluster with multi-architecture compute machines on bare metal (`x86_64`), {ibm-power-name} (`ppc64le`), or {ibm-z-name} (`s390x`) you must have an existing single-architecture cluster on one of these platforms. Follow the installations procedures for your platform:
To create a cluster with multi-architecture compute machines on either bare metal (`x86_64`), {ibm-power-name} (`ppc64le`), or {ibm-z-name} (`s390x`) you must have an existing single-architecture cluster on one of these platforms. Follow the installations procedures for your platform:

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. That's much clearer.

@alishaIBM
Copy link
Contributor

/assign @prb112

@holgwolf
Copy link

LGTM

Copy link

@prb112 prb112 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Feb 15, 2024
@SNiemann15
Copy link
Contributor Author

/label peer-review-needed

@openshift-ci openshift-ci bot added the peer-review-needed Signifies that the peer review team needs to review this PR label Feb 15, 2024
@SNiemann15
Copy link
Contributor Author

/label multi-arch

@openshift-ci openshift-ci bot added the multi-arch Label for all IBM Power and Z PRs label Feb 15, 2024
@skrthomas skrthomas added peer-review-in-progress Signifies that the peer review team is reviewing this PR and removed peer-review-needed Signifies that the peer review team needs to review this PR labels Feb 15, 2024
Copy link
Contributor

@skrthomas skrthomas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small comment that applies globally, but lgtm otherwise.

@@ -15,6 +15,13 @@ Before you can add `ppc64le` nodes to your cluster, you must upgrade your cluste

The following procedures explain how to create a {op-system} compute machine using an ISO image or network PXE booting. This will allow you to add `ppc64le` nodes to your cluster and deploy a cluster with multi-architecture compute machines.

[NOTE]
====
To create an {ibm-power-name} (`ppc64le`) cluster with multi-architecture compute machines on `x86_64`, follow the instructions for
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
To create an {ibm-power-name} (`ppc64le`) cluster with multi-architecture compute machines on `x86_64`, follow the instructions for
To create an {ibm-power-name} (`ppc64le`) cluster with multiarchitecture compute machines on `x86_64`, follow the instructions for

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was brought up to me in another peer review but aparently most words beginning with multi are not supposed to by hypenated. I know there are multiple instances of this in the repo as "multi-architecture" Here's the IBMSG info:
https://www.ibm.com/docs/en/ibm-style?topic=grammar-prefixes
Screenshot 2024-02-12 at 2 17 37 PM

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Multi-architecture compute is a fixed term for this feature.
I assume that spelling was agreed on when this feature was introduced.
@kelbrown20 Is this something we should add to the SSG?

@skrthomas skrthomas added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-in-progress Signifies that the peer review team is reviewing this PR labels Feb 15, 2024
@SNiemann15
Copy link
Contributor Author

/label merge-review-needed

@openshift-ci openshift-ci bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Feb 16, 2024
@snarayan-redhat snarayan-redhat removed the merge-review-needed Signifies that the merge review team needs to review this PR label Feb 16, 2024
@SNiemann15
Copy link
Contributor Author

/label merge-review-needed

@openshift-ci openshift-ci bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Feb 16, 2024
@snarayan-redhat snarayan-redhat removed the merge-review-needed Signifies that the merge review team needs to review this PR label Feb 16, 2024
@snarayan-redhat snarayan-redhat added merge-review-in-progress Signifies that the merge review team is reviewing this PR branch/enterprise-4.15 labels Feb 16, 2024
@snarayan-redhat snarayan-redhat added this to the Planned for 4.15 GA milestone Feb 16, 2024
@snarayan-redhat snarayan-redhat merged commit 83b3175 into openshift:main Feb 16, 2024
@snarayan-redhat
Copy link
Contributor

/cherrypick enterprise-4.15

@openshift-cherrypick-robot

@snarayan-redhat: new pull request created: #71723

In response to this:

/cherrypick enterprise-4.15

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.

@snarayan-redhat snarayan-redhat removed the merge-review-in-progress Signifies that the merge review team is reviewing this PR label Feb 16, 2024
@SNiemann15 SNiemann15 deleted the ibmpz_control_plane branch February 16, 2024 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch/enterprise-4.15 lgtm Indicates that a PR is ready to be merged. multi-arch Label for all IBM Power and Z PRs peer-review-done Signifies that the peer review team has reviewed this PR size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants