Skip to content

[OSDOCS-8215]: 4.14z Updating hosted control planes TP table, adding new features & enhancements#66219

Merged
kcarmichael08 merged 1 commit intoopenshift:enterprise-4.14from
lahinson:osdocs-6698-hcp-table
Nov 21, 2023
Merged

[OSDOCS-8215]: 4.14z Updating hosted control planes TP table, adding new features & enhancements#66219
kcarmichael08 merged 1 commit intoopenshift:enterprise-4.14from
lahinson:osdocs-6698-hcp-table

Conversation

@lahinson
Copy link
Contributor

@lahinson lahinson commented Oct 13, 2023

Version(s): 4.14+

Issue: https://issues.redhat.com/browse/OSDOCS-8215

Link to docs preview: https://66219--docspreview.netlify.app/openshift-enterprise/latest/release_notes/ocp-4-14-release-notes#ocp-4-14-technology-preview (scroll to "Hosted control plane Technology Preview features")

QE review:

  • QE has approved this change.

Additional information:
This PR is for the ACM 2.9/MCE 2.4 release that is scheduled for Nov. 20. Do not merge until this date.
The links to the RHACM docs will not work until Nov. 20, when the docs go live.

This PR has received approvals from stakeholders per the OCP docs change management process.

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

ocpdocs-previewbot commented Oct 13, 2023

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

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

@lahinson lahinson changed the title [OSDOCS-8190]: Updating hosted control planes TP table [OSDOCS-8190]: 4.14Z Updating hosted control planes TP table Oct 20, 2023
@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 Oct 29, 2023
@lahinson lahinson force-pushed the osdocs-6698-hcp-table branch from fd7842f to 11935a6 Compare November 3, 2023 20:27
@lahinson lahinson changed the title [OSDOCS-8190]: 4.14Z Updating hosted control planes TP table [OSDOCS-8215]: 4.14Z Updating hosted control planes TP table Nov 3, 2023
@lahinson lahinson changed the title [OSDOCS-8215]: 4.14Z Updating hosted control planes TP table [OSDOCS-8215]: 4.14z Updating hosted control planes TP table, adding new features & enhancements Nov 9, 2023
@lahinson lahinson force-pushed the osdocs-6698-hcp-table branch from 11935a6 to 513732b Compare November 10, 2023 21:57
@openshift-ci openshift-ci bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Nov 10, 2023
@lahinson lahinson force-pushed the osdocs-6698-hcp-table branch 5 times, most recently from dd2a86f to e0c80f9 Compare November 10, 2023 22:41
@openshift-ci openshift-ci bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Nov 10, 2023
@lahinson lahinson force-pushed the osdocs-6698-hcp-table branch from e0c80f9 to 0e60bab Compare November 10, 2023 23:50
@openshift-ci openshift-ci bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Nov 10, 2023
@lahinson lahinson force-pushed the osdocs-6698-hcp-table branch 5 times, most recently from 2daaa1e to c9184f5 Compare November 11, 2023 00:43
@YuLi517
Copy link

YuLi517 commented Nov 13, 2023

Please add known issue: OCPBUGS-22753. Description can be same as stolostron/rhacm-docs#5777.

[#hosted-cluster-dual-stack]
=== Potential DNS issues in hosted clusters configured for a dual-stack, disconnected environment

When you create a hosted cluster in a disconnected environment that uses the dual-stack network, you might encounter the following DNS-related issues:

* `CrashLoopBackOff` state in the `service-ca-operator` pod: When the pod tries to reach the Kubernetes API server through the hosted control plane, the pod cannot reach the server because the data plane proxy in the `kube-system` namespace cannot resolve the request. This issue occurs because in the HAProxy setup, the front end uses an IP address and the back end uses a DNS name that the pod cannot resolve.
* Pods stuck in `ContainerCreating` state: This issue occurs because the `openshift-service-ca-operator` cannot generate the `metrics-tls` secret that the DNS pods need for DNS resolution. As a result, the pods cannot resolve the Kubernetes API server.

To resolve those issues, configure the DNS server settings by following the guidelines in xref:../hosted_control_planes/dual_stack_dns.adoc#dual-stack-dns[Configuring DNS for a dual stack network].

@lahinson lahinson force-pushed the osdocs-6698-hcp-table branch from c9184f5 to d13deac Compare November 13, 2023 17:25
@lahinson
Copy link
Contributor Author

@YuLi517 I added the known issue as you requested. PTAL when you can and let me know if anything should change. Thanks!

@skrthomas skrthomas added the peer-review-in-progress Signifies that the peer review team is reviewing this PR label Nov 17, 2023
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 note, lgtm otherwise

[id="ocp-4-14-hcp-ibm-power-z"]
==== Hosted control planes on {ibmpowerProductName} and {ibmzProductName} (Technology Preview)
In this release, hosted control planes is available as a Technology Preview feature on {ibmpowerProductName} and {ibmzProductName}.
//Add links to IBM Power and IBM Z docs when available
Copy link
Contributor

Choose a reason for hiding this comment

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

Just a note, still need links here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch! I'll add those.

@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 Nov 17, 2023
@lahinson lahinson force-pushed the osdocs-6698-hcp-table branch from 57b9cba to d92bda9 Compare November 17, 2023 18:03
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Nov 17, 2023
@openshift-ci
Copy link

openshift-ci bot commented Nov 17, 2023

New changes are detected. LGTM label has been removed.

@lahinson lahinson force-pushed the osdocs-6698-hcp-table branch from d92bda9 to c070525 Compare November 17, 2023 18:10
@lahinson
Copy link
Contributor Author

/label merge-review-needed

Note to reviewers: This PR should not be merged until Nov. 20, in alignment with the ACM 2.9 GA. I'm requesting a merge review in advance per the new process: https://docs.google.com/document/d/1fLMpK4bqthtFlCwA36yeo2cI-pExFpX76VMnSi_In6Q/edit#heading=h.x8ydklaezdcq

@openshift-ci openshift-ci bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Nov 17, 2023
@bburt-rh bburt-rh added merge-review-in-progress Signifies that the merge review team is reviewing this PR branch/enterprise-4.14 and removed merge-review-needed Signifies that the merge review team needs to review this PR labels Nov 17, 2023
@bburt-rh bburt-rh added this to the Continuous Release milestone Nov 17, 2023
Copy link
Contributor

@bburt-rh bburt-rh left a comment

Choose a reason for hiding this comment

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

Not sure what's going with the attributes in the TP table not rendering, but the issue needs to be resolved/explained before I can set this PR as good-to-merge. Thanks, @lahinson!

|Technology Preview
|General Availability

|Hosted control planes for {product-title} on {ibmpowerProductName}
Copy link
Contributor

Choose a reason for hiding this comment

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

This attribute is not rendering in the preview. Is it being defined in a different PR that hasn't merged yet?

image

|Not Available
|Technology Preview

|Hosted control planes for {product-title} on {ibmzProductName}
Copy link
Contributor

Choose a reason for hiding this comment

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

This attribute also is not rendering in the preview.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think the attributes file was recently updated. I'll double-check and replace the ibmzProductName attribute with whatever the new attribute is. Thanks!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@bburt-rh The attributes are fixed now. Thanks again!

Copy link
Contributor

@bburt-rh bburt-rh Nov 17, 2023

Choose a reason for hiding this comment

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

Great. Thanks, @lahinson! LGTM now. Giving it the ok-to-merge label

@lahinson lahinson force-pushed the osdocs-6698-hcp-table branch from c070525 to 7d0b9b2 Compare November 17, 2023 21:06
@bburt-rh bburt-rh added ok-to-merge qe-approved Signifies that QE has signed off on this PR and removed merge-review-in-progress Signifies that the merge review team is reviewing this PR labels Nov 17, 2023
@lahinson lahinson marked this pull request as ready for review November 21, 2023 15:53
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 21, 2023
@lahinson
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 Nov 21, 2023
@kcarmichael08 kcarmichael08 added merge-review-in-progress Signifies that the merge review team is reviewing this PR branch/enterprise-4.15 and removed merge-review-needed Signifies that the merge review team needs to review this PR labels Nov 21, 2023
@kcarmichael08 kcarmichael08 merged commit 2f3dc9d into openshift:enterprise-4.14 Nov 21, 2023
@kcarmichael08 kcarmichael08 removed branch/enterprise-4.15 merge-review-in-progress Signifies that the merge review team is reviewing this PR labels Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.14 ok-to-merge peer-review-done Signifies that the peer review team has reviewed this PR qe-approved Signifies that QE has signed off on 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