Skip to content

Conversation

@rohennes
Copy link
Contributor

@rohennes rohennes commented Feb 3, 2023

TELCODOCS 947: You can create mixed-architectures clusters by the using the Assisted Installer API to add Aarch64 architecture hosts to deployed AMD64 clusters.

Version(s):
TBD - need peer review only

Issue:
https://issues.redhat.com/browse/TELCODOCS-947

Link to docs preview:
https://55537--docspreview.netlify.app/openshift-enterprise/latest/release_notes/ocp-4-12-release-notes.html#ocp-4-12-1-assisted-installer-api-mixed-arch-clusters

QE review:

  • QE has approved this change.

Merging will be handled by different PR. Location in OCP docs may not be accurate, just need a peer review of RN content only.

@openshift-ci openshift-ci bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 3, 2023
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Feb 3, 2023

🤖 Updated build preview is available at:
https://55537--docspreview.netlify.app

Build log: https://circleci.com/gh/ocpdocs-previewbot/openshift-docs/10036

@rohennes rohennes changed the title PR for peer review of RN for TELCODOCS 947 PR for review of RN for TELCODOCS 947 Feb 3, 2023
Copy link

Choose a reason for hiding this comment

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

maybe the better term is available, as openshift not yet fully-supports it (for OCP 4.12)

Suggested change
This feature is only supported by using the {ai-full} API.
This feature is only available by using the {ai-full} API.

Copy link

Choose a reason for hiding this comment

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

in the spirit of the comment below about supports term

Suggested change
{product-title} now supports mixed-architecture clusters, also known as heterogeneous clusters, that feature hosts with both AMD64 and AArch64 CPU architectures. Use the {ai-full} API to create a heterogeneous cluster by adding AArch64 worker nodes to a deployed AMD64 cluster.
{product-title} now allows mixed-architecture clusters, also known as heterogeneous clusters, that feature hosts with both AMD64 and AArch64 CPU architectures. Use the {ai-full} API to create a heterogeneous cluster by adding AArch64 worker nodes to a deployed AMD64 cluster.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just reorganised the content to reflect your comment (we try to avoid the word allows)

@trewest
Copy link

trewest commented Feb 6, 2023

I think the doc makes sense for the SaaS since release images are handled for customers, however in ZTP environments I think it is important to highlight that mixed-architecture clusters are only possible when using multi architecture release images for installation of the day 1 cluster.

Copy link

Choose a reason for hiding this comment

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

Change release image to multi-architecture release image

@rohennes
Copy link
Contributor Author

rohennes commented Feb 6, 2023

/label telco
/label peer-review-needed

@openshift-ci openshift-ci bot added telco Label for all Telco PRs peer-review-needed Signifies that the peer review team needs to review this PR labels Feb 6, 2023
@trewest
Copy link

trewest commented Feb 6, 2023

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Feb 6, 2023
@jeana-redhat jeana-redhat added the peer-review-in-progress Signifies that the peer review team is reviewing this PR label Feb 6, 2023
Copy link
Contributor

@jeana-redhat jeana-redhat left a comment

Choose a reason for hiding this comment

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

This content looks good to me.

It seems to be a new feature added in a z-stream, and the z-stream is also already out as of 1/30. I would consider adding a "New features" heading for this if it is a new feature. I also suspect that this needs to go through the change management process - please confirm with your CS and DPM.

/remove-label peer-review-in-progress
/remove-label peer-review-needed
/label peer-review-done

@openshift-ci openshift-ci bot 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 peer-review-needed Signifies that the peer review team needs to review this PR labels Feb 6, 2023
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Mar 1, 2023
@openshift-ci
Copy link

openshift-ci bot commented Mar 1, 2023

New changes are detected. LGTM label has been removed.

@openshift-ci openshift-ci bot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Mar 1, 2023
@rohennes
Copy link
Contributor Author

Merge of this content completed in #56147

@rohennes rohennes closed this Mar 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

peer-review-done Signifies that the peer review team has reviewed this PR size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. telco Label for all Telco PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants