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
docs: Update cluster mesh instructions #26608
Conversation
michi-covalent
commented
Jul 3, 2023
- Only document using the Helm mode. Time to deprecate the classic mode.
- Remove "clustermesh enable" and "clustermesh connect" output samples. These outputs might change in the future, and I think output samples from the subsequent "clustermesh status" commands give users enough context to confirm that "clustermesh enable" and "clustermesh connect" commands succeeded.
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 from my side, thanks!
I see this is a release blocker for 1.14:
|
Also, I wonder: Would it help to have a note on the change of default mode for the Cilium CLI in the upgrade notes? |
- Only document using the Helm mode. Time to deprecate the classic mode. - Remove "clustermesh enable" and "clustermesh connect" output samples. These outputs might change in the future, and I think output samples from the subsequent "clustermesh status" commands give users enough context to confirm that "clustermesh enable" and "clustermesh connect" commands succeeded. Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
45d10e7
to
e460541
Compare
yeah we are ready. added
yeah we should do that, @giorio94 also pointed this out in #26606 (review). i'll add upgrade notes in #26606 📝 |
Sounds good, thanks! Doc-change only - codeowners reviews are in, and the relevant workflows passed successfully. We're good to merge this. |