-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
modules/virt-cloud-platforms.adoc
Outdated
| {odf-short} | ||
|
||
| {gcp-first} | ||
| Technology Preview |
There was a problem hiding this comment.
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
modules/virt-cloud-platforms.adoc
Outdated
.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] |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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. :)
Hi Pan,
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?
…On Mon, Jul 7, 2025 at 4:19 PM Pan Ousley ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In modules/virt-cloud-platforms.adoc
<#95622 (comment)>
:
> +
+| {gcp-first}
+| Technology Preview
+| {gcp-short} native storage
+
+| {product-dedicated}
+| Technology Preview
+| {gcp-short} native storage
+
+// | IBM Cloud
+// | Technology Preview
+// | IBM Fusion
+|===
+
+[role="_additional-resources"]
+.Additional resources
Thanks! I will add these
—
Reply to this email directly, view it on GitHub
<#95622 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AQZTZAR7ENZHTKECDFYNEIL3HJXUBAVCNFSM6AAAAACAX44546VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDSOJTHA2DKNBUGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@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. |
I was referring to the hints about using cnv on a specific cloud, like:
thanks, this was the hint I was looking for
@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 ? |
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? |
b404b00
to
dd47162
Compare
9fbba99
to
4f1ece4
Compare
@ousleyp This looks good. Thank you. |
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 |
7380000
to
c398070
Compare
@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 |
461b283
to
1148810
Compare
|
||
[id="supported-platforms_preparing-cluster-for-virt"] | ||
== Supported platforms | ||
[id="bare-metal-support_preparing-cluster-for-virt"] |
There was a problem hiding this comment.
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.
@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. |
@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. |
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:
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.