Skip to content

Conversation

@emarcusRH
Copy link
Contributor

@emarcusRH emarcusRH commented Aug 15, 2022

Version(s):
enterprise-4.11
enterprise-4.10
enterprise-4.12

Issue:

  • options matrix for OCP on RHV in restricted networks is not rendering correctly in v 4.11 docs
  • matrix not correct regarding IPI and UPI support for RHV

Link to docs preview:

http://file.tlv.redhat.com/emarcus/upi_ipi/installing/installing-preparing.html#supported-installation-methods-for-different-platforms

Additional information:

@openshift-ci openshift-ci bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Aug 15, 2022
@emarcusRH
Copy link
Contributor Author

@engelmi , @janosdebugs
I need someone to double check that the support for UPI on restricted networks exists in both 4.10 and 4.11.
In the installation instructions in the guide, it implies that both are supported - callout description #4 in this sample YAML file indicates that IPI is supported - https://docs.openshift.com/container-platform/4.11/installing/installing_rhv/installing-rhv-restricted-network.html#installation-bare-metal-config-yaml_installing-rhv-restricted-network

@engelmi
Copy link

engelmi commented Aug 16, 2022

Based on the documentation for 4.10 and 4.11, I'd say yes - not sure how to verify this correctly. Can you help? @janosdebugs

@openshift-merge-robot openshift-merge-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Aug 16, 2022
@ghost
Copy link

ghost commented Aug 16, 2022

Sadly, I don't know the answer. I have to assume that the previous version would be correct, but @peterclauterbach would probably know more.

@peterclauterbach
Copy link

I'm not sure I understand the question. Once we support a deployment option, we continue to support it, unless it is explicitly deprecated.
Since it was it was supported in OCP 4.9, it is correct that support continues in OCP 4.10 and OCP 4.11.

@ghost
Copy link

ghost commented Aug 16, 2022

I believe the question is between UPI and IPI: which one do we support for a restricted network installation?

@emarcusRH
Copy link
Contributor Author

I'm not sure I understand the question. Once we support a deployment option, we continue to support it, unless it is explicitly deprecated. Since it was it was supported in OCP 4.9, it is correct that support continues in OCP 4.10 and OCP 4.11.

@peterclauterbach the instructions for OCP on RHV/restricted networks provides the procdure for UPI installations - https://docs.openshift.com/container-platform/4.11/installing/installing_rhv/installing-rhv-restricted-network.html#installation-network-user-infra_installing-rhv-restricted-network
But the support matrix in the general Installation chapter is missing an "X" under UPI/Restricted Networks > RHV
https://docs.openshift.com/container-platform/4.11/installing/installing-preparing.html#supported-installation-methods-for-different-platforms

  • is this simply an oversight that it was not updated?

@peterclauterbach
Copy link

Yes, it's an oversight. We support UPI installations on restricted networks. The bulk of UPI installations are direconnected.

@openshift-merge-robot openshift-merge-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Aug 17, 2022
@openshift-merge-robot openshift-merge-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Aug 22, 2022
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Aug 22, 2022
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Aug 22, 2022
@emarcusRH
Copy link
Contributor Author

too many commit errors - restarting this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants