OSDOCS#14025: Removing note saying that modern profile isn't supporte…

[bergerhoffer]

…d for control plane

Version(s):
4.19

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

Link to docs preview:

• https://92144--ocpdocs-pr.netlify.app/openshift-enterprise/latest/security/tls-security-profiles.html#tls-profiles-kubernetes-configuring_tls-security-profiles
• https://92144--ocpdocs-pr.netlify.app/openshift-enterprise/latest/security/tls-security-profiles.html#tls-profiles-understanding_tls-security-profiles

QE review:

• QE has approved this change.

Additional information:

Do not merge this until the eng PR merges

[ocpdocs-previewbot]

🤖 Wed Apr 16 18:24:49 - Prow CI generated the docs preview:

https://92144--ocpdocs-pr.netlify.app/openshift-dedicated/latest/networking/networking_operators/ingress-operator.html
https://92144--ocpdocs-pr.netlify.app/openshift-enterprise/latest/networking/networking_operators/ingress-operator.html
https://92144--ocpdocs-pr.netlify.app/openshift-enterprise/latest/nodes/nodes/nodes-nodes-tls.html
https://92144--ocpdocs-pr.netlify.app/openshift-enterprise/latest/security/tls-security-profiles.html
https://92144--ocpdocs-pr.netlify.app/openshift-rosa/latest/networking/networking_operators/ingress-operator.html

[bergerhoffer]

@jacobsee @dusk125 Can you PTAL? (I won't merge until we're sure the eng PR makes it)

[dusk125]

/lgtm

[dusk125]

Looks like we need to update the following to include the modern option as well.
Step 2 of the procedure for configuring control plane in modules/tls-profiles-kubernetes-configuring.adoc

<1> Specify the TLS security profile type (`Old`, `Intermediate`, or `Custom`). The default is `Intermediate`.
<2> Specify the appropriate field for the selected type:
* `old: {}`
* `intermediate: {}`
* `custom:`
<3> For the `custom` type, specify a list of TLS ciphers and minimum accepted TLS version.

[bergerhoffer]

@gangwgr I know the eng PR isn't merged yet, but here is the docs PR for TLS 1.3/modern profile for control plane. Can you PTAL when you get a chance? (I'll make sure the eng PR merges before merging this)

@dusk125 noticed that we needed to add modern as an option below the example, but I also see that we didn't have it listed for the ingress controller and kubelet. So I've added it there, because I'm pretty certain it's supported for those. Let me know if this is something you can help confirm, or if we need to pull in someone else to help with that. Thanks!

Previews:

• https://92144--ocpdocs-pr.netlify.app/openshift-enterprise/latest/security/tls-security-profiles.html#tls-profiles-kubernetes-configuring_tls-security-profiles
• https://92144--ocpdocs-pr.netlify.app/openshift-enterprise/latest/security/tls-security-profiles.html#tls-profiles-understanding_tls-security-profiles

[dusk125]

/lgtm

[bergerhoffer]

Preview regeneration seems stuck. Prow jobs look to be backed up, so I don't think retesting will do anything other than putting us farther down in the queue. Will check back in later.

Edit: all good now

[bergerhoffer]

/retest

[openshift-ci]

@bergerhoffer: 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.