community icon indicating copy to clipboard operation
community copied to clipboard

update code of conduct charter to address other committee elections

Open salaxander opened this issue 3 years ago • 26 comments

Over the course of the code of conduct committee's existence, we have seen a number of members run for steering while serving on the committee. In the event that they are elected, they resign from code of conduct (serving on both would be a conflict of interest). This does however create the potential for high turnover on code of conduct if many members end up making that a stepping stone to steering.

I am proposing this charter change to encourage elected members to remain with code of conduct through their committed term.

salaxander avatar Nov 02 '22 15:11 salaxander

/hold

salaxander avatar Nov 02 '22 15:11 salaxander

/assign @jeremyrickard @detiber @endocrimes @hlipsig

salaxander avatar Nov 02 '22 16:11 salaxander

/lgtm

detiber avatar Nov 02 '22 16:11 detiber

/lgtm

hlipsig avatar Nov 02 '22 16:11 hlipsig

/hold /cc @kubernetes/steering-committee /label committee/steering

Explicit hold as all charter changes require steering review

cblecker avatar Nov 02 '22 21:11 cblecker

@cblecker: The label(s) `/label committee/steering

cannot be applied. These labels are supported:api-review, tide/merge-method-merge, tide/merge-method-rebase, tide/merge-method-squash, team/katacoda, refactor. Is this label configured under labels -> additional_labelsorlabels -> restricted_labelsinplugin.yaml`?

In response to this:

/hold /cc @kubernetes/steering-committee /label committee/steering

Explicit hold as all charter changes require steering review

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

k8s-ci-robot avatar Nov 02 '22 21:11 k8s-ci-robot

/committee steering

Let's try that again

cblecker avatar Nov 02 '22 21:11 cblecker

Compared to stated intent in the PR description, I'm not sure I see in the charter change a strong incentivize for that outcome. Pondering if/how we might more strongly....

tpepper avatar Nov 02 '22 23:11 tpepper

@tpepper I don't think we want to stop people from doing so if that's what they desire, but want to make it less of a... "tossing the hat in the ring and then making a decision" (which is fairly disruptive to both of our election processes), but more "committing to staying on the CoCC or running for steering".

endocrimes avatar Nov 02 '22 23:11 endocrimes

To the discussion above, I might suggest updating the PR body re:

I am proposing this charter change to encourage elected members to remain with code of conduct through their committed term.

To also note the positive effect of ensuring a predictable transition period if members do go forward with running anyhow.

Just for anyone coming across this later.

BenTheElder avatar Nov 03 '22 02:11 BenTheElder

/hold /cc @kubernetes/steering-committee /label committee/steering

Explicit hold as all charter changes require steering review

I may want to add this statement to the charter as part of this too then. Currently it states "Any changes to the charter require explicit LGTM or Approve from all committee members. For pull requests, a /hold will be applied until all approvals are present. Any changes merged without consensus will be reverted". I think it makes total sense to have approval from steering as well, I just want to be sure that requirement is documented.

salaxander avatar Nov 03 '22 15:11 salaxander

New changes are detected. LGTM label has been removed.

k8s-ci-robot avatar Nov 03 '22 15:11 k8s-ci-robot

@salaxander Yes, it might make sense to call that out. You could link it to https://github.com/kubernetes/community/tree/master/committee-steering/governance#sig-charter-approval-process which is the relevant section in the governance docs

cblecker avatar Nov 03 '22 16:11 cblecker

Counter-argument: recruiting folks for CoCC at all is very difficult, and the pool of potential CoCC members has like 70% overlap with the pool of potential SC members. Are we sure that we want to position things so that potential candidates say "hmmm, I don't want to run for CoCC because I might want to run for SC next year"?

jberkus avatar Nov 05 '22 19:11 jberkus

Counter-argument: recruiting folks for CoCC at all is very difficult, and the pool of potential CoCC members has like 70% overlap with the pool of potential SC members. Are we sure that we want to position things so that potential candidates say "hmmm, I don't want to run for CoCC because I might want to run for SC next year"?

I think that's a valid concern. Maybe we as the current committee need to do a better job of sourcing candidates for future roles? As it is though, the high turnover and lack of continuity ends up being quite disruptive to the code of conduct committee. I'm open to exploring other options, but we really would like to find a way to ensure that we don't find ourselves in a situation like the current one again (every member of the committee is new)

salaxander avatar Nov 07 '22 16:11 salaxander

The thing I’ve been trying to figure out: how do we incentivize finishing out terms? Admittedly I didn’t on CoCC myself. And while I feel like I need to go back and deliver on that commitment, it wont be easy: to stand for next year’s 2023 CoCC election I’d have to step down early from SC. Or I would need to not stand for 2023 SC reelection and and then wait ten months for 2024 CoCC (re)election. That wouldn’t be helped by first stepping off CoCC per the idea here, and like Josh notes it looses one or both bodies a volunteer. It might only be helped by us individually realizing we need to be finish commitments before aspiring to or taking on next ones. That’s hard to incentivize, especially constructively vs punitively.

Still feels punitive, but would it be more effective to say committee candidacy does not include ones who did not complete a prior committee commitment in the (~1year?) lead up to the committee election?

tpepper avatar Nov 08 '22 18:11 tpepper

Top of head thought: would it make this problem better/worse/no change if we aligned the election cycles between the committees

cblecker avatar Nov 08 '22 19:11 cblecker

Top of head thought: would it make this problem better/worse/no change if we aligned the election cycles between the committees

Currently, the SC elects the CoCC. So that would shift to having the "lame duck" SC elect them. Which isn't necessarily a blocker, but is a shift in who selects what.

jberkus avatar Nov 08 '22 19:11 jberkus

The Kubernetes project currently lacks enough contributors to adequately respond to all PRs.

This bot triages PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Mark this PR as fresh with /remove-lifecycle stale
  • Close this PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar Mar 16 '23 12:03 k8s-triage-robot

/remove-lifecycle stale

salaxander avatar Mar 16 '23 14:03 salaxander

The Kubernetes project currently lacks enough contributors to adequately respond to all PRs.

This bot triages PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Mark this PR as fresh with /remove-lifecycle stale
  • Close this PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar Jan 20 '24 20:01 k8s-triage-robot

@Chiefy0x1: changing LGTM is restricted to collaborators

In response to this:

/LGTM

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

k8s-ci-robot avatar Jan 23 '24 10:01 k8s-ci-robot

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Chiefy0x1, detiber, endocrimes, salaxander

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

k8s-ci-robot avatar Jan 23 '24 10:01 k8s-ci-robot

/lifecycle frozen

endocrimes avatar Jan 23 '24 14:01 endocrimes

@endocrimes: The lifecycle/frozen label cannot be applied to Pull Requests.

In response to this:

/lifecycle frozen

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

k8s-ci-robot avatar Jan 23 '24 14:01 k8s-ci-robot

The Kubernetes project currently lacks enough active contributors to adequately respond to all PRs.

This bot triages PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Mark this PR as fresh with /remove-lifecycle rotten
  • Close this PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle rotten

k8s-triage-robot avatar Feb 22 '24 15:02 k8s-triage-robot

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.

This bot triages PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Reopen this PR with /reopen
  • Mark this PR as fresh with /remove-lifecycle rotten
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/close

k8s-triage-robot avatar Mar 23 '24 15:03 k8s-triage-robot

@k8s-triage-robot: Closed this PR.

In response to this:

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.

This bot triages PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Reopen this PR with /reopen
  • Mark this PR as fresh with /remove-lifecycle rotten
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

k8s-ci-robot avatar Mar 23 '24 15:03 k8s-ci-robot