Skip to content

CNV#64393: cloud platforms matrix #95622

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

ousleyp
Copy link
Member

@ousleyp ousleyp commented Jul 3, 2025

Version(s): 4.18+ (should confirm)

Issue: CNV-64393, CNV-63953 & possibly others

Link to docs preview: https://95622--ocpdocs-pr.netlify.app/openshift-enterprise/latest/virt/install/preparing-cluster-for-virt.html

QE review:

  • QE has approved this change.

Additional information: I'm also moving a few things around in this assembly with the intention of adding clarity/logical flow in light of the new section.

@openshift-ci openshift-ci bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jul 3, 2025
@ousleyp ousleyp added this to the Continuous Release milestone Jul 3, 2025
Copy link

@dominikholler dominikholler left a comment

Choose a reason for hiding this comment

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

@ousleyp thanks, this PR will add so much clarity to our docs!

@phoracek I expect that there are restrictions regarding cnv networking on the clouds, but I did not yet recognize them documented in the related kbase articles. Are there any plans to document the restrictions?

| {odf-short}

| {gcp-first}
| Technology Preview

Choose a reason for hiding this comment

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

@ngavrilo the GCP TP is waiting for your ack in jira

.Additional resources
* link:https://github.com/oracle-quickstart/oci-openshift/blob/main/custom_manifests/oci-ccm-csi-drivers/v1.30.0-RWX-LA/openshift-virtualization.md[Installing {VirtProductName} on OCI]
* link:https://access.redhat.com/articles/7118050[{VirtProductName} and Oracle Cloud Infrastructure known issues and limitations]
* https://catalog.redhat.com/search?p=1&target_platforms=Red%20Hat%20OpenShift[{product-title} partners in the Red{nbsp}Hat Ecosystem Catalog]

Choose a reason for hiding this comment

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

Unfortunately the Ecosystem Catalog is not yet helpful.
CC: @ronensdeor

Copy link

@ctomasko ctomasko Jul 9, 2025

Choose a reason for hiding this comment

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

agreed. I looked at RHEC and I could not easily find the cloud partners.
I'm unsure about mentioning RHEC. This was my idea because we typically send customers to RHEC to find our partners, but in this case, it's not easy to find the cloud partners. However, when I applied the cloud filter: https://catalog.redhat.com/en/search?subcategories=Cloud&target_platforms=Red+Hat+OpenShift
I did find links to portworx and such. I defer to Ronen to decide if this is okay to include. IMO, it won't hurt, but I also don't want to introduce confusion.

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll comment it out for now but can add it back in upon request. :)

@ronensdeor
Copy link

ronensdeor commented Jul 7, 2025 via email

@phoracek
Copy link
Member

phoracek commented Jul 9, 2025

@phoracek I expect that there are restrictions regarding cnv networking on the clouds, but I did not yet recognize them documented in the related kbase articles. Are there any plans to document the restrictions?

@dominikholler pardon my ignorance, but could you link the articles? I'm not aware of any. All we have is a recommendation to use one particular type of networking on public clouds https://docs.redhat.com/en/documentation/openshift_container_platform/4.19/html/virtualization/networking#virt-networking.

I do think the network limitations on cloud would merit better doc / KCS.

@dominikholler
Copy link

@phoracek I expect that there are restrictions regarding cnv networking on the clouds, but I did not yet recognize them documented in the related kbase articles. Are there any plans to document the restrictions?

@dominikholler pardon my ignorance, but could you link the articles? I'm not aware of any.

I was referring to the hints about using cnv on a specific cloud, like:

All we have is a recommendation to use one particular type of networking on public clouds https://docs.redhat.com/en/documentation/openshift_container_platform/4.19/html/virtualization/networking#virt-networking.

thanks, this was the hint I was looking for

I do think the network limitations on cloud would merit better doc / KCS.

@ousleyp Might it be helpful for the reader to have a sentence here to let the reader know that there are further cloud specific information in https://docs.redhat.com/en/documentation/openshift_container_platform/4.19/html/virtualization/networking#virt-networking ?

@ousleyp
Copy link
Member Author

ousleyp commented Jul 10, 2025

A quick question: I believe we will have at least one document per cloud
solution, so wouldn't it be better to also include those links in the table
instead of below?

Thanks, @ronensdeor. I was thinking the same, but when I first opened the PR I was only aware of the OCI links. The links are now in the table. Are there specific links I should add for the two GA clouds (AWS/ROSA), or should I mark those cells as N/A?

@openshift-ci openshift-ci bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jul 10, 2025
@ousleyp ousleyp force-pushed the cnv-64393 branch 3 times, most recently from b404b00 to dd47162 Compare July 10, 2025 19:48
@ousleyp ousleyp force-pushed the cnv-64393 branch 2 times, most recently from 9fbba99 to 4f1ece4 Compare July 10, 2025 20:57
@ronensdeor
Copy link

@ousleyp This looks good. Thank you.
We still have the link to "Installing a cluster on AWS with customizations." below the table. Should it move to the table as well?

@ousleyp
Copy link
Member Author

ousleyp commented Jul 14, 2025

@ousleyp This looks good. Thank you. We still have the link to "Installing a cluster on AWS with customizations." below the table. Should it move to the table as well?

I can add that to the table; thanks for pointing that out. (I might also leave it in the list for now since I am not really editing that section, so I apologize for the redundancy.)

@ronensdeor Do you have any recommendations for the ROSA row in the table, which currently has no links? Maybe something like https://docs.aws.amazon.com/rosa/latest/userguide/what-is-rosa.html ? Thanks!

@ousleyp
Copy link
Member Author

ousleyp commented Jul 14, 2025

@ousleyp This looks good. Thank you. We still have the link to "Installing a cluster on AWS with customizations." below the table. Should it move to the table as well?

I can add that to the table; thanks for pointing that out. (I might also leave it in the list for now since I am not really editing that section, so I apologize for the redundancy.)

@ronensdeor Do you have any recommendations for the ROSA row in the table, which currently has no links? Maybe something like https://docs.aws.amazon.com/rosa/latest/userguide/what-is-rosa.html ? Thanks!

Ah, sorry, I just realized that the AWS link is within our docs (an xref) so it can't go in a module. It has to go in the assembly (where it currently is). Is there an external link that would be helpful? I might also change the heading to "External links" so that it is more clear that the table doesn't contain cross-references. Let me know your thoughts. Thanks, @ronensdeor

@ousleyp ousleyp force-pushed the cnv-64393 branch 2 times, most recently from 7380000 to c398070 Compare July 14, 2025 18:50
@ronensdeor
Copy link

@ousleyp maybe we can link to the official ROSA documentation for virtualization: https://docs.redhat.com/en/documentation/red_hat_openshift_service_on_aws/4/html/virtualization/index

@ousleyp ousleyp force-pushed the cnv-64393 branch 2 times, most recently from 461b283 to 1148810 Compare July 16, 2025 16:49

[id="supported-platforms_preparing-cluster-for-virt"]
== Supported platforms
[id="bare-metal-support_preparing-cluster-for-virt"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

🤖 [error] RedHat.TermsErrors: Use 'bare-metal servers' rather than 'bare metal servers'. For more information, see RedHat.TermsErrors.

Copy link

openshift-ci bot commented Jul 16, 2025

@ousleyp: 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-sigs/prow repository. I understand the commands that are listed here.

@ronensdeor
Copy link

@ousleyp I think we should remove the AWS from the "Bare metal support" section as it's already in the "Compatible cloud platforms" table.

@ousleyp
Copy link
Member Author

ousleyp commented Jul 17, 2025

@ousleyp I think we should remove the AWS from the "Bare metal support" section as it's already in the "Compatible cloud platforms" table.

Thanks @ronensdeor . I'll try to follow up on this later today and do the heading changes we discussed as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch/enterprise-4.18 branch/enterprise-4.19 branch/enterprise-4.20 CNV Label for all CNV PRs size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants