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

Adding more moderators to our communication properties #3212

Open
wants to merge 8 commits into
base: master
from

Conversation

@castrojo
Copy link
Contributor

castrojo commented Feb 5, 2019

First cut at expanding our moderator coverage. I wanted to push this early so I can get plenty of review, so there are things missing like an issue template, etc. The general gist:

  • Moderator selection process modelled after the k8s membership process.
  • Required yearly checkups for moderator teams.
  • A proposed set of open slots for moderators, I just educated-guessed what I think a good number of moderators per property should be, so we can definitely adjust there.
  • A looser set of moderator pro tempore positions to allow for larger coverage for people who can't commit to full time moderation.

/hold
/sig contributor-experience
/area community-management

@castrojo

This comment has been minimized.

Copy link
Contributor Author

castrojo commented Feb 5, 2019

Should probably add a section for moderating teams reporting on their property status yearly to contribex as well.

@castrojo

This comment has been minimized.

Copy link
Contributor Author

castrojo commented Feb 5, 2019

This is part of fixing this issue: #3128

@cblecker

This comment has been minimized.

Copy link
Member

cblecker commented Feb 6, 2019

Spot review from me looks good. May also worth calling out that the GitHub Administration Team (/github-management#github-administration-team) is responsible for moderation enforcement on GitHub.

Show resolved Hide resolved communication/moderators.md Outdated
@castrojo

This comment has been minimized.

Copy link
Contributor Author

castrojo commented Feb 6, 2019

Question for the group, not sure if we should have an escalation policy here, or if we should make a generic one that can cover all things, including moderation.

@idvoretskyi
Copy link
Member

idvoretskyi left a comment

/lgtm

@mrbobbytables

This comment has been minimized.

Copy link
Member

mrbobbytables commented Feb 6, 2019

Question for the group, not sure if we should have an escalation policy here, or if we should make a generic one that can cover all things, including moderation.

I think we can have a generic escalation policy covering who, when and how things should be escalated. Those sorts of things should be common across the various communication platforms.

@parispittman

This comment has been minimized.

Copy link
Contributor

parispittman commented Feb 6, 2019

should we put the selection of moderators criteria on moderators.md with the list of moderators? that can be the canonical source of who, how whereas moderation.md is the what

@k8s-ci-robot

This comment has been minimized.

Copy link
Contributor

k8s-ci-robot commented Feb 6, 2019

New changes are detected. LGTM label has been removed.

@k8s-ci-robot k8s-ci-robot removed the lgtm label Feb 6, 2019

Show resolved Hide resolved communication/moderation.md Outdated
Show resolved Hide resolved communication/moderators.md Outdated
Show resolved Hide resolved communication/moderation.md Outdated
Update moderator and moderation policies
This updates the list of moderators to reflect assignments. We've added
empty slots of each k8s property to ensure we're expanding coverage
across the board.

 - Added descriptor for Github admin team
 - Added requirement for yearly reporting
 - Made it explicit that sponsors and moderators are expected to keep up
with platform changes.
 - Link to SIG ContribEx charter for list of projects we
maintain

Signed-off-by: Jorge O. Castro <jorgec@vmware.com>

@castrojo castrojo force-pushed the castrojo:moderator-update branch from 97b2f65 to 0a902ee Feb 7, 2019

Show resolved Hide resolved communication/moderation.md Outdated
Show resolved Hide resolved communication/moderation.md
Show resolved Hide resolved communication/moderators.md
Show resolved Hide resolved communication/moderators.md
Show resolved Hide resolved communication/moderators.md Outdated
Show resolved Hide resolved communication/moderators.md
Added a template for moderator applications and more small fixes from…
… Paris' review

Signed-off-by: Jorge O. Castro <jorgec@vmware.com>
Make it explicit for YouTube moderators to check comments
Signed-off-by: Jorge O. Castro <jorgec@vmware.com>
Final fixes to moderator template
Signed-off-by: Jorge O. Castro <jorgec@vmware.com>

castrojo added some commits Feb 13, 2019

Remove moderator template, I'll put that in a separate PR
Signed-off-by: Jorge O. Castro <jorgec@vmware.com>
Add escalation procedures for moderators
Signed-off-by: Jorge O. Castro <jorgec@vmware.com>
Fix spelling to pass tests
Signed-off-by: Jorge O. Castro <jorgec@vmware.com>
@k8s-ci-robot

This comment has been minimized.

Copy link
Contributor

k8s-ci-robot commented Feb 15, 2019

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: castrojo, cblecker, idvoretskyi

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment