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
Documentation: explicitly state that KVStoreMesh is beta level as part of the feature title #26868
Conversation
/test |
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.
LGTM for docs
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.
Do we also want the related note under the title? We usually include it for beta features (.. include:: ../../beta.rst
)
Additionally, let's use the ready to be included snipped about the request for feedback instead of a custom message. Let's also remove the redundant beta feature information from the "enable cluster mesh" section, as already displayed as part of the link title: "[...] opt in to KVStoreMesh (beta) when enabling [...]". Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
5468ac6
to
2d17b8b
Compare
The page already reported a similar note (at the end of the paragraph), but I didn't know about the snippet ready to be included. I've updated the PR to also change that, as well as drop a redundant beta level note as now it is already part of the link title. PTAL. |
No description provided.