OCPBUGS#37577: Add a note about identity-provider-arn#109594
OCPBUGS#37577: Add a note about identity-provider-arn#109594aksjoshi89 wants to merge 1 commit intoopenshift:enterprise-4.15from
Conversation
|
🤖 Mon Apr 06 13:45:22 - Prow CI generated the docs preview: |
| [role="_abstract"] | ||
| The {cert-manager-operator} is not installed in {product-title} by default. You can install the {cert-manager-operator} by using the web console. | ||
|
|
||
| The operator sets `features.operators.openshift.io/token-auth-aws`, `features.operators.openshift.io/token-auth-azure`, `features.operators.openshift.io/token-auth-gcp` annotations in the operator CSV, and console asks to provide the relevant credentials details when these annotations are set. Currently, the operator does not make use of the values collected by the console and users should follow https://docs.redhat.com/en/documentation/openshift_container_platform/4.20/html/security_and_compliance/cert-manager-operator-for-red-hat-openshift. Users can provide any dummy value when asked for the input. For example, when installing on ROSA, the `identity-provider-arn` is asked and any value can be provided to proceed. |
There was a problem hiding this comment.
🤖 [error] Vale.Terms: Use 'Operators?' instead of 'operator'.
|
@emmajiafan |
| [role="_abstract"] | ||
| The {cert-manager-operator} is not installed in {product-title} by default. You can install the {cert-manager-operator} by using the web console. | ||
|
|
||
| The operator sets `features.operators.openshift.io/token-auth-aws`, `features.operators.openshift.io/token-auth-azure`, `features.operators.openshift.io/token-auth-gcp` annotations in the operator CSV, and console asks to provide the relevant credentials details when these annotations are set. Currently, the operator does not make use of the values collected by the console and users should follow https://docs.redhat.com/en/documentation/openshift_container_platform/4.20/html/security_and_compliance/cert-manager-operator-for-red-hat-openshift. Users can provide any dummy value when asked for the input. For example, when installing on ROSA, the `identity-provider-arn` is asked and any value can be provided to proceed. |
There was a problem hiding this comment.
The text is not enclosed in a Note or Important block. Also in the preview I see the initial intro text appearing twice could you please check.
269d812 to
6300a24
Compare
|
@aksjoshi89: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions 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. |
|
/lgtm |
|
/lgtm |
|
/label merge-review-needed |
lunarwhite
left a comment
There was a problem hiding this comment.
I feel this PR should be opened against master/main branch and cherrypicked onto subsequently supported releases rather than enterprise-4.15 solely. The reported ticket https://redhat.atlassian.net/browse/OCPBUGS-37577 explicitly mentioned 4.15.18 / [enterprise-4.16] probably because at that time 4.16 was pointed to the master/main, but now it's 4.22. We should maintain consistency across different release branches since the cert-manager operator is OCP version agnostic.
Version(s):
4.15
Issue:
OCPBUGS-37577
Link to docs preview:
https://109594--ocpdocs-pr.netlify.app/openshift-enterprise/latest/security/cert_manager_operator/cert-manager-operator-install.html
QE review: