Skip to content

Conversation

@estroz
Copy link
Member

@estroz estroz commented Apr 29, 2020

  • code-of-conduct.md: link to CNCF code of conduct add contact instructions

@openshift-ci-robot openshift-ci-robot added the do-not-merge/invalid-owners-file Indicates that a PR should not merge because it has an invalid OWNERS file in it. label Apr 29, 2020
@jberkhahn
Copy link
Contributor

jberkhahn commented May 7, 2020

couple of questions:

@estroz
Copy link
Member Author

estroz commented May 7, 2020

@jberkhahn "community chair" is a governance designation, while "subproject owner" is a project-specific role. The election process for each is different.

I assume you mean a Github team construct. Is that typical for reviewers? If so, then probably.

@jberkhahn
Copy link
Contributor

Yeah, I meant a Github team.

Is the co-chair position you, Ken Sipe, and myself do getting folded into the community chair position, the subproject owner position, or is it remaining it's own thing? If so, it should be documented somewhere.

@estroz
Copy link
Member Author

estroz commented May 7, 2020

@jberkhahn community co-chairs should be documented somewhere, and probably have an owners alias. Will add this as a follow up to #13

@jberkhahn
Copy link
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label May 7, 2020
Copy link
Member

@jmrodri jmrodri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 15, 2020
@estroz estroz changed the title add code of conduct, organize OWNERS add code of conduct, update OWNERS May 19, 2020
@estroz estroz force-pushed the chore/code-of-conduct branch from 69779a6 to 824c43a Compare May 19, 2020 00:59
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label May 19, 2020
@openshift-ci-robot
Copy link

New changes are detected. LGTM label has been removed.

@openshift-ci-robot openshift-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 19, 2020
@openshift-ci-robot
Copy link

The following users are mentioned in OWNERS file(s) but are not members of the operator-framework org.

Once all users have been added as members of the org, you can trigger verification by writing /verify-owners in a comment.

  • todo
    • OWNERS_ALIASES

Copy link
Member

@kensipe kensipe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

left some comments / questions... but generally
lgtm

@estroz estroz force-pushed the chore/code-of-conduct branch from 824c43a to cc435de Compare July 1, 2020 16:00
@estroz estroz changed the title add code of conduct, update OWNERS add code of conduct Jul 1, 2020
* Figure out how to keep and then keep a record of CoC reports and outcomes for precedent _This is something we are still actively discussing on how to do and is not fully hardened._

## Formation of the Code of Conduct Committee (CoCC):
* The CoCC consists of 5 members. In the first election, the top 3 voted people will be appointed 2 year terms and the other 2 members will be appointed for a 1 year term.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two years seems like a long time. Perhaps everyone gets a 1 year term but every year only half of the folks are up for re-election. So after you get the initial 5 members, 3 of the folks up for election next year and then the other two up for election the year after that. That way you always have the opportunity for new folks on the committee but you also have continuity since not everyone is up for election at the same time.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your wording is confusing.

not everyone is up for election at the same time.

You can't stagger election years if everyone has a 1 year term. Are you suggesting that everyone has a 1 year term, then at the end of that year 3 members can be re-elected but 2 people cannot until the following year?

* The members of the Code of Conduct Committee (CoCC) will be public so the people who report CoC issues know exactly who they are working with.

## Upon appointment, the CoCC will:
* Review the current Code of Conduct and propose revisions or addendums if necessary.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the process for approving revisions or addendums? Majority vote? Two thirds vote?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From charter.md:

Meetings are considered at quorum when a simple majority of the members are
present. Where there are 4 or fewer members available due vacant seats or
recusal, quorum is 2.

@openshift-ci-robot
Copy link

The following users are mentioned in OWNERS file(s) but are untrusted for the following reasons. One way to make the user trusted is to add them as members of the operator-framework org. You can then trigger verification by writing /verify-owners in a comment.

  • todo
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.

@estroz estroz requested a review from kensipe September 9, 2020 17:25
@estroz estroz force-pushed the chore/code-of-conduct branch from e9f0141 to 07c2f48 Compare September 22, 2020 18:45
@estroz estroz force-pushed the chore/code-of-conduct branch from 07c2f48 to b250281 Compare September 22, 2020 18:47
@estroz estroz force-pushed the chore/code-of-conduct branch from b250281 to 6d398c9 Compare September 22, 2020 18:48
@estroz
Copy link
Member Author

estroz commented Sep 22, 2020

On further thought, having a committee for operator-framework may be superfluous since the CNCF already has a mediator. If we find we need a full CoC committee, we can revisit the documents previously proposed here.

@estroz estroz changed the title add code of conduct code-of-conduct.md: link to CNCF code of conduct and add contact info Sep 22, 2020
@estroz estroz merged commit 30095f9 into operator-framework:master Sep 22, 2020
@estroz estroz deleted the chore/code-of-conduct branch September 22, 2020 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/invalid-owners-file Indicates that a PR should not merge because it has an invalid OWNERS file in it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants