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: Slack updates #25723
docs: Slack updates #25723
Conversation
Commit 69a76d30c7efba7b0c9f1e59a07cd3478f5a5182 does not contain "Signed-off-by". Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin |
69a76d3
to
324b5cc
Compare
Commit 914c7fdf2b516547724aaedb0b56e8dfa446d420 does not contain "Signed-off-by". Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin |
914c7fd
to
cb85da5
Compare
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.
😞 Now the spell checker is giving you grief.
Please fix the following spelling mistakes:
* Documentation/community/community.rst:45: (tetragon) #tetragon
* Documentation/community/community.rst:83: (ciliumcon) #ciliumcon
* Documentation/community/community.rst:84: (ctf) #ctf
If the words are not misspelled, run:
Documentation/update-spelling_wordlist.sh ciliumcon ctf tetragon
Do let me know if you want me to push this over the finish line :-)
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.
Please clean-up the description of your first commit, it contains two titles and two identical Signed-off-by
.
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.
Changes lgtm now- thanks!
Commitwise, best to squash the two commits, as well as address Quentin's comment about the commit message.
List some more of the existing Slack channels Clarify that not all Slack channels have to be listed in the docs Document how to request a new Slack channel Signed-off-by: Liz Rice <liz@lizrice.com> Co-authored-by: David Bimmler <david.bimmler@isovalent.com>
5c766af
to
494a5c1
Compare
@@ -41,6 +41,8 @@ Name Purpose | |||
#kubernetes Kubernetes-specific questions | |||
#networkpolicy Questions on network policies | |||
#release Release announcements only | |||
#service-mesh Questions on Cilium Service Mesh |
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.
a lot of community members ares using this slack channel for clustermesh as well, seems like we don't have dedicated channel for clustermesh.
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.
Can you be more explicit please, is this an request for creating the channel and adding to the list in this PR? Or are you OK with the current update?
@lizrice: Have you considered creating #clustermesh?
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.
yeah, it will be great to have a dedicated clustermesh channel 👍, the changes in this PR look good to me.
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.
OK thanks! Now with this PR, I guess you know what to do to request the channel creation 😁
Doc change only, documentation workflow and Netlify preview passed. This is good to go |
Changes to the Slack docs page:
Note: I've suggested that for simply creating a Slack channel this issue could be filed in the cilium/community repo. There is a very similar process for creating a SIG, which gets a Slack channel and regular Zoom meeting, and currently is requested via an issue in cilium/cilium. Should we move that to cilium/community as well?