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
Document the ingressClassName
field
#1196
Document the ingressClassName
field
#1196
Conversation
Signed-off-by: Maël Valais <mael@vls.dev>
They should be "name" and "class". Signed-off-by: Maël Valais <mael@vls.dev>
Signed-off-by: Maël Valais <mael@vls.dev>
Signed-off-by: Maël Valais <mael@vls.dev>
Signed-off-by: Maël Valais <mael@vls.dev>
Signed-off-by: Maël Valais <mael@vls.dev>
✅ Deploy Preview for cert-manager-website ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
Looks good @maelvls
Just a few typos and suggestions
### `class` | ||
### `ingressClassName` | ||
|
||
> The field `ingressClassName` was added in cert-manager 1.12. |
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.
Is there an icon we use for these version info boxes?
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 did not find an existing icon. I added "ℹ️" and an info div.
Note that usually, we use the format:
FEATURE STATE: cert-manager 1.12 [GA]
This format doesn't let the user know when the feature was first introduced (can they use the feature, even if in alpha?).
content/docs/installation/upgrading/ingress-class-compatibility.md
Outdated
Show resolved
Hide resolved
Signed-off-by: Maël Valais <mael@vls.dev> Co-Authored-By: Richard Wall <richard.wall@jetstack.io>
Signed-off-by: Maël Valais <mael@vls.dev> Co-Authored-By: Richard Wall <richard.wall@jetstack.io>
Signed-off-by: Maël Valais <mael@vls.dev> Co-Authored-By: Richard Wall <richard.wall@jetstack.io>
Signed-off-by: Maël Valais <mael@vls.dev> Co-Authored-By: Richard Wall <richard.wall@jetstack.io>
66b8655
to
0b99aa0
Compare
I addressed your comments. PTAL @wallrj |
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.
Thanks @maelvls
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: maelvls, wallrj The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Implementation PR: cert-manager/cert-manager#5849