Skip to content
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

governance role invitations #10

Closed
27 of 30 tasks
jacobweinstock opened this issue Apr 1, 2022 · 39 comments
Closed
27 of 30 tasks

governance role invitations #10

jacobweinstock opened this issue Apr 1, 2022 · 39 comments
Labels
priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release.

Comments

@jacobweinstock
Copy link
Member

jacobweinstock commented Apr 1, 2022

Tinkerbell Role Bootstrapping

The following is the proposal for role bootstrapping. See the governance (here) for definitions, details, responsibilities, etc on each role.

Response Requested

To accept or decline a role, please comment in this issue with the repository and role name, and whether you accept this role or not.

Repository Maintainers Approvers Reviewers
tink @mmlb,@micahhausler @jacobweinstock, @tstromberg, @displague, @nshalman @tobert, @stephen-fox, @chrisdoherty4, @detiber
boots @mmlb,@jacobweinstock @tobert, @nshalman, @tstromberg, @rgl @mikemrm, @ScottGarman, @displague, @dlaube
hegel @nshalman, @mmlb, @chrisdoherty4 @tstromberg, @displague @jacobweinstock, @splaspood
pbnj @jacobweinstock @nshalman, @joelrebel @tstromberg, @mmlb, @abhay-krishna, @Raj-Dharwadkar, @nhorman
sandbox @jacobweinstock, @mmlb @displague, @detiber @ProvenGuilty, @nshalman
ipxedust @jacobweinstock @mmlb @stephen-fox, @nshalman, @Raj-Dharwadkar, @rgl
dhcp @jacobweinstock @nshalman @displague, @tobert
lint-install @tstromberg @mmlb, @stephen-fox @jacobweinstock, @displague, @Raj-Dharwadkar
hook @thebsdbox @tstromberg, @mmlb @jmpolom, @ScottGarman, @jacobweinstock, @Raj-Dharwadkar
hub @thebsdbox, @nshalman @mmlb, @displague @alienninja, @jacobweinstock, @stephen-fox
cluster-api-provider-tinkerbell @detiber, @displague, @micahhausler @tstromberg @mmlb
crocodile @thebsdbox, @mmlb @tstromberg, @dlaube @jeremytanner, @rgl
terraform-provider-tinkerbell @jeremytanner @mmlb, @displague @jacobweinstock
tinkerbell-docs @DailyAlice, @displague @mmlb, @tstromberg @rgl, @cprivitere, @felixwidjaja, @jacobweinstock
k8s-sandbox @detiber @tstromberg @sheepcat, @mmlb
infrastructure @mmlb @tstromberg @rawkode

Implementation Details

Each person was selected for a role based on their contributions and history in the repo. Load in other areas of the Tinkerbell project was also taken into account. Contributions were pulled from the CNCF's devstats tool. The script found here was used to get the data.

FAQ

What if I don't want to have a role?

No worries. There is no obligation. This is an invitation based on your recent history of activity and contribution to the Tinkerbell community and project.

What if I wanted to have a specific role but I'm not?

Your desire is awesome! This is just a one-time bootstrap process. Please follow the existing governance on how to become the role for your desired repository.

What if I'm not even a member of the Tinkerbell org?

First off, thank you for contributing! You rock. As part of this bootstrapping process, and if you accept, you will be added to the Tinkerbell org and the proposed role.

What should be done if a role or roles in a repo are "short-staffed"?

If a repo is "short-staffed" for roles, the repo's maintainer(s) can propose others to be an Approver or Reviewer. If there are no objections from any other maintainer(s) across the Tinkerbell org then the individual should be added to the role. This will be a one-time bootstrapping activity. After the bootstrapping process is complete individuals should follow the existing governance.

Response Tracking

List of all people invited to a role. A checked box indicates the individual has responded to the invitation.

@jacobweinstock
Copy link
Member Author

jacobweinstock commented Apr 1, 2022

Hello, I accept all role invitations. Thank you.

repo role accepted
tink approver yes
boots maintainer yes
hegel reviewer yes
pbnj maintainer yes
sandbox maintainer yes
ipxedust maintainer yes
dhcp maintainer yes
lint-install reviewer yes
hook reviewer yes
hub reviewer yes
terraform-provider-tinkerbell reviewer yes
tinkerbell-docs reviewer yes

@displague
Copy link
Member

Thanks for preparing this @jacobweinstock! I accept all role invitations. Thanks.

@rawkode
Copy link

rawkode commented Apr 1, 2022

Happy to review Infrastructure 💯

@micahhausler
Copy link
Contributor

I the invitation to be a maintainer of Tink! If we're short on folks for any role in cluster-api, I'm happy to help there too

@cprivitere
Copy link

I accept the reviewer nomination for tinkerbell-docs.

@Raj-Dharwadkar
Copy link

I accept reviewer invitations for pbnj, ipxedust, hook and lint-install. Thank you!

@jmpolom
Copy link

jmpolom commented Apr 2, 2022

I accept being a reviewer for hook.

@jacobweinstock
Copy link
Member Author

I the invitation to be a maintainer of Tink! If we're short on folks for any role in cluster-api, I'm happy to help there too

awesome, thank you.

@displague, you are a maintainer for CAPT, up to you if you would like to nominate. No objections from me. See the FAQ above on "What should be done if a role or roles in a repo are "short-staffed"?"

@displague
Copy link
Member

Definitely appreciated and accepted, @micahhausler

@chrisdoherty4
Copy link
Member

I accept being a reviewer for Tink

@jacobweinstock
Copy link
Member Author

Definitely appreciated and accepted, @micahhausler

Hey. @displague. To which role in CAPT would you like me to add Micah?

@DailyAlice
Copy link

I can definitely help out with docs! I accept being a maintainer for tinkerbell-docs.

@mmlb
Copy link
Contributor

mmlb commented Apr 4, 2022

I accept my roles.

@felixwidjaja
Copy link

I accept being reviewer role at tinkerbell-docs.
Thanks 🙏

@nshalman
Copy link
Member

nshalman commented Apr 5, 2022

I accept the following role invitations:

Repo Role
tink approver
boots approver
hegel maintainer
pbnj approver
sandbox reviewer
ipxedust reviewer
dhcp approver
hub maintainer

@jeremytanner
Copy link
Member

I accept review crocodile / maintain terraform-provider-tinkerbell

@detiber
Copy link

detiber commented Apr 5, 2022

I accept review tink, approver sandbox, maintainer cluster-api-provider-tinkerbell.

For k8s-sandbox, I'd like to suggest that we archive it in favor of tracking work to update sandbox to support kubernetes.

@joelrebel
Copy link

I accept the approver role for pbnj.

@displague
Copy link
Member

@jacobweinstock

@micahhausler's interest in maintainership of capt matches the commitment level on tink - it's great to see! (sgtm)

@thebsdbox
Copy link

I formally accepted my roles !

@chrisdoherty4
Copy link
Member

I'd like to tender myself for consideration as a maintainer of Hegel. I recognize its slightly premature but figured I would throw it out there given the refactoring I'm doing (lots more to come).

@nshalman @mmlb @jacobweinstock @displague

@jacobweinstock
Copy link
Member Author

jacobweinstock commented Apr 21, 2022

I'd like to tender myself for consideration as a maintainer of Hegel. I recognize its slightly premature but figured I would throw it out there given the refactoring I'm doing (lots more to come).

@nshalman @mmlb @jacobweinstock @displague

I'm all for it! The process during this one time bootstrap event calls for the maintainers (@nshalman and @mmlb) to make this type of call though.

@ScottGarman
Copy link

I accept my role invitations for boots and hook.

@tobert
Copy link

tobert commented Apr 21, 2022

I accept for boots & dhcp.

@splaspood
Copy link

I accept the reviewer nomination for hegel. :)

@mikemrm
Copy link

mikemrm commented Apr 21, 2022

I accept my role invitations

@dlaube
Copy link

dlaube commented Apr 21, 2022

I accept my role invitations.

@stephen-fox
Copy link

Hi Jacob - I accept my reviewer role for tink, ipxe-dust, and hub. I also accept my approver role for lint-install.

@rgl
Copy link

rgl commented Apr 22, 2022

I accept my role invitations.

role repositories
approver boots
reviewer ipxedust, crocodile, tinkerbell-docs

@alienninja
Copy link

I accept my role nomination for hub reviewer. Thanks!

jacobweinstock added a commit to jacobweinstock/hook that referenced this issue Apr 29, 2022
This is needed to enable the repo roles defined by
the community here: tinkerbell/org#10

Signed-off-by: Jacob Weinstock <[email protected]>
jacobweinstock added a commit to jacobweinstock/actions that referenced this issue Apr 29, 2022
This is needed to enable the repo roles defined by
the community here: tinkerbell/org#10

Signed-off-by: Jacob Weinstock <[email protected]>
jacobweinstock added a commit to jacobweinstock/crocodile that referenced this issue Apr 29, 2022
This is needed to enable the repo roles defined by
the community here: tinkerbell/org#10

Signed-off-by: Jacob Weinstock <[email protected]>
jacobweinstock added a commit to jacobweinstock/terraform-provider-tinkerbell that referenced this issue Apr 29, 2022
This is needed to enable the repo roles defined by
the community here: tinkerbell/org#10

Signed-off-by: Jacob Weinstock <[email protected]>
jacobweinstock added a commit to jacobweinstock/tinkerbell-docs that referenced this issue Apr 29, 2022
This is needed to enable the repo roles defined by
the community here: tinkerbell/org#10

Signed-off-by: Jacob Weinstock <[email protected]>
jacobweinstock added a commit to jacobweinstock/infrastructure that referenced this issue Apr 29, 2022
This is needed to enable the repo roles defined by
the community here: tinkerbell/org#10

Signed-off-by: Jacob Weinstock <[email protected]>
mergify bot added a commit to tinkerbell/ipxedust that referenced this issue Apr 29, 2022
## Description


This is needed to enable the repo roles defined by the community here: tinkerbell/org#10

## Why is this needed



Fixes: #

## How Has This Been Tested?





## How are existing users impacted? What migration steps/scripts do we need?





## Checklist:

I have:

- [ ] updated the documentation and/or roadmap (if required)
- [ ] added unit or e2e tests
- [ ] provided instructions on how to upgrade
mergify bot added a commit to tinkerbell/lint-install that referenced this issue Apr 29, 2022
## Description


This is needed to enable the repo roles defined by the community here: tinkerbell/org#10

## Why is this needed



Fixes: #

## How Has This Been Tested?





## How are existing users impacted? What migration steps/scripts do we need?





## Checklist:

I have:

- [ ] updated the documentation and/or roadmap (if required)
- [ ] added unit or e2e tests
- [ ] provided instructions on how to upgrade
mergify bot added a commit to tinkerbell/hook that referenced this issue Apr 29, 2022
## Description


This is needed to enable the repo roles defined by the community here: tinkerbell/org#10

## Why is this needed



Fixes: #

## How Has This Been Tested?





## How are existing users impacted? What migration steps/scripts do we need?





## Checklist:

I have:

- [ ] updated the documentation and/or roadmap (if required)
- [ ] added unit or e2e tests
- [ ] provided instructions on how to upgrade
mergify bot added a commit to tinkerbell/crocodile that referenced this issue Apr 29, 2022
## Description


This is needed to enable the repo roles defined by the community here: tinkerbell/org#10

## Why is this needed



Fixes: #

## How Has This Been Tested?





## How are existing users impacted? What migration steps/scripts do we need?





## Checklist:

I have:

- [ ] updated the documentation and/or roadmap (if required)
- [ ] added unit or e2e tests
- [ ] provided instructions on how to upgrade
mergify bot added a commit to tinkerbell/actions that referenced this issue Apr 29, 2022
## Description


This is needed to enable the repo roles defined by the community here: tinkerbell/org#10

## Why is this needed



Fixes: #

## How Has This Been Tested?





## How are existing users impacted? What migration steps/scripts do we need?





## Checklist:

I have:

- [ ] updated the documentation and/or roadmap (if required)
- [ ] added unit or e2e tests
- [ ] provided instructions on how to upgrade
mergify bot added a commit to tinkerbell/infrastructure that referenced this issue Apr 29, 2022
## Description


This is needed to enable the repo roles defined by the community here: tinkerbell/org#10

## Why is this needed



Fixes: #

## How Has This Been Tested?





## How are existing users impacted? What migration steps/scripts do we need?





## Checklist:

I have:

- [ ] updated the documentation and/or roadmap (if required)
- [ ] added unit or e2e tests
- [ ] provided instructions on how to upgrade
jacobweinstock added a commit to jacobweinstock/tinkerbell-docs that referenced this issue Apr 29, 2022
This is needed to enable the repo roles defined by
the community here: tinkerbell/org#10

Signed-off-by: Jacob Weinstock <[email protected]>
jacobweinstock added a commit to jacobweinstock/tinkerbell-docs that referenced this issue Apr 29, 2022
This is needed to enable the repo roles defined by
the community here: tinkerbell/org#10

Signed-off-by: Jacob Weinstock <[email protected]>
mergify bot added a commit to tinkerbell/dhcp that referenced this issue Apr 29, 2022
## Description


This is needed to enable the repo roles defined by the community here: tinkerbell/org#10

## Why is this needed



Fixes: #

## How Has This Been Tested?





## How are existing users impacted? What migration steps/scripts do we need?





## Checklist:

I have:

- [ ] updated the documentation and/or roadmap (if required)
- [ ] added unit or e2e tests
- [ ] provided instructions on how to upgrade
mergify bot added a commit to tinkerbell/tinkerbell-docs that referenced this issue Apr 29, 2022
## Description


This is needed to enable the repo roles defined by the community here: tinkerbell/org#10

## Why is this needed



Fixes: #

## How Has This Been Tested?





## How are existing users impacted? What migration steps/scripts do we need?





## Checklist:

I have:

- [ ] updated the documentation and/or roadmap (if required)
- [ ] added unit or e2e tests
- [ ] provided instructions on how to upgrade
mmlb added a commit to tinkerbell/terraform-provider-tinkerbell that referenced this issue Apr 29, 2022
## Description

<!--- Please describe what this PR is going to change -->
This is needed to enable the repo roles defined by the community here: tinkerbell/org#10


## Why is this needed

<!--- Link to issue you have raised -->

Fixes: #

## How Has This Been Tested?
<!--- Please describe in detail how you tested your changes. -->
<!--- Include details of your testing environment, and the tests you ran to -->
<!--- see how your change affects other areas of the code, etc. -->


## How are existing users impacted? What migration steps/scripts do we need?

<!--- Fixes a bug, unblocks installation, removes a component of the stack etc -->
<!--- Requires a DB migration script, etc. -->


## Checklist:

I have:

- [ ] updated the documentation and/or roadmap (if required)
- [ ] added unit or e2e tests
- [ ] provided instructions on how to upgrade
mmlb added a commit to tinkerbell/cluster-api-provider-tinkerbell that referenced this issue Apr 29, 2022
Signed-off-by: Micah Hausler <[email protected]>

## Description

Add CODEOWNERS based on tinkerbell/org#10

## Why is this needed

Allows additional folks to review and approve changes
@chrisdoherty4
Copy link
Member

@jacobweinstock What's left to do so we can close?

@displague
Copy link
Member

The previously maintained (now outdated) GitHub Teams for admins, writers, triage, and others have been removed.

The probot monitored .github/settings.yml lists are now authoritative.

@displague
Copy link
Member

We'll want to add https://github.com/tinkerbell/tinkerbell.org to the list of projects under governance.

ttwd80 pushed a commit to ttwd80/tinkerbell-playground that referenced this issue Sep 7, 2024
This is needed to enable the repo roles defined by
the community here: tinkerbell/org#10

Signed-off-by: Jacob Weinstock <[email protected]>
ttwd80 pushed a commit to ttwd80/tinkerbell-playground that referenced this issue Sep 7, 2024
## Description


This is needed to enable the repo roles defined by the community here: tinkerbell/org#10

## Why is this needed



Fixes: #

## How Has This Been Tested?





## How are existing users impacted? What migration steps/scripts do we need?





## Checklist:

I have:

- [ ] updated the documentation and/or roadmap (if required)
- [ ] added unit or e2e tests
- [ ] provided instructions on how to upgrade
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release.
Projects
None yet
Development

No branches or pull requests