|
= Committer Policy =
|
|
# Committer Policy
|
|
|
|
|
|
|
|
|
|
[[TOC]]
|
|
|
|
|
|
|
|
This policy was provisionally accepted by the network team on 7 November 2019, by voting and agreement in our team meeting. It became non-provisional on 7 February 2020.
|
|
This policy was provisionally accepted by the network team on 7 November 2019, by voting and agreement in our team meeting. It became non-provisional on 7 February 2020.
|
|
|
|
|
|
For information on:
|
|
For information on:
|
|
* '''when and what''' to merge, see the [[org/teams/NetworkTeam/MergePolicy|MergePolicy]] document.
|
|
* **when and what** to merge, see the [[org/teams/NetworkTeam/MergePolicy|MergePolicy]] document.
|
|
* '''how''' to merge, see the [[org/teams/NetworkTeam/MergeProcess|MergeProcess]] document.
|
|
* **how** to merge, see the [[org/teams/NetworkTeam/MergeProcess|MergeProcess]] document.
|
|
|
|
|
|
== Scope ==
|
|
## Scope
|
|
|
|
|
|
This policy gives Network Team staff the commit bit. Staff should
|
|
This policy gives Network Team staff the commit bit. Staff should
|
|
get the commit bit if they are regularly doing Network Team roles
|
|
get the commit bit if they are regularly doing Network Team roles
|
... | @@ -19,7 +19,7 @@ but a similar "get permissions if needed" policy should also apply to: |
... | @@ -19,7 +19,7 @@ but a similar "get permissions if needed" policy should also apply to: |
|
* other Network Team repositories on git.torproject.org,
|
|
* other Network Team repositories on git.torproject.org,
|
|
* permissions needed on other platforms, like GitHub or GitLab.
|
|
* permissions needed on other platforms, like GitHub or GitLab.
|
|
|
|
|
|
== Rationale ==
|
|
## Rationale
|
|
|
|
|
|
There are two network team roles that automatically get the commit bit.
|
|
There are two network team roles that automatically get the commit bit.
|
|
The people who are currently in those roles are:
|
|
The people who are currently in those roles are:
|
... | @@ -48,9 +48,9 @@ For details see: |
... | @@ -48,9 +48,9 @@ For details see: |
|
Merge Policy:
|
|
Merge Policy:
|
|
* https://trac.torproject.org/projects/tor/wiki/org/teams/NetworkTeam/MergePolicy
|
|
* https://trac.torproject.org/projects/tor/wiki/org/teams/NetworkTeam/MergePolicy
|
|
|
|
|
|
== Process ==
|
|
## Process
|
|
|
|
|
|
=== 0. Learn How We Merge ===
|
|
### 0. Learn How We Merge
|
|
|
|
|
|
If you want to do master merges, read the Merge Policy
|
|
If you want to do master merges, read the Merge Policy
|
|
and Merge Process:
|
|
and Merge Process:
|
... | @@ -60,27 +60,27 @@ and Merge Process: |
... | @@ -60,27 +60,27 @@ and Merge Process: |
|
If you want to do backport merges, also read the Backport Process:
|
|
If you want to do backport merges, also read the Backport Process:
|
|
* https://trac.torproject.org/projects/tor/wiki/org/teams/NetworkTeam/Backports
|
|
* https://trac.torproject.org/projects/tor/wiki/org/teams/NetworkTeam/Backports
|
|
|
|
|
|
=== 1. Set Up Your Git Upstream Environment ===
|
|
### 1. Set Up Your Git Upstream Environment
|
|
|
|
|
|
Follow the latest Merge Policy setup instructions at:
|
|
Follow the latest Merge Policy setup instructions at:
|
|
* https://trac.torproject.org/projects/tor/wiki/org/teams/NetworkTeam/MergeProcess#ConfiguringyourEnvironment
|
|
* https://trac.torproject.org/projects/tor/wiki/org/teams/NetworkTeam/MergeProcess#ConfiguringyourEnvironment
|
|
|
|
|
|
=== 2. Check If You Already Have Access ===
|
|
### 2. Check If You Already Have Access
|
|
|
|
|
|
Make sure you have an LDAP account with working SSH keys:
|
|
Make sure you have an LDAP account with working SSH keys:
|
|
* https://db.torproject.org/doc-mail.html
|
|
* https://db.torproject.org/doc-mail.html
|
|
|
|
|
|
Run the command:
|
|
Run the command:
|
|
{{{
|
|
```
|
|
$ ssh git@git-rw.torproject.org | grep W
|
|
$ ssh git@git-rw.torproject.org | grep W
|
|
}}}
|
|
```
|
|
|
|
|
|
=== 3. Get Access ===
|
|
### 3. Get Access
|
|
|
|
|
|
A current merger gets you access, using the git.torproject.org process at:
|
|
A current merger gets you access, using the git.torproject.org process at:
|
|
* https://trac.torproject.org/projects/tor/wiki/org/operations/services/git.torproject.org#Addingdeveloperstoarepository
|
|
* https://trac.torproject.org/projects/tor/wiki/org/operations/services/git.torproject.org#Addingdeveloperstoarepository
|
|
|
|
|
|
=== 4. First Time ===
|
|
### 4. First Time
|
|
|
|
|
|
The first time you do a master merge, ask another merger to check
|
|
The first time you do a master merge, ask another merger to check
|
|
your work before you push to upstream.
|
|
your work before you push to upstream.
|
... | @@ -88,7 +88,7 @@ your work before you push to upstream. |
... | @@ -88,7 +88,7 @@ your work before you push to upstream. |
|
The first time you do a backport merge, ask a backport merger to
|
|
The first time you do a backport merge, ask a backport merger to
|
|
check your work before you push to upstream.
|
|
check your work before you push to upstream.
|
|
|
|
|
|
=== 5. Removing Access ===
|
|
### 5. Removing Access
|
|
|
|
|
|
Anyone may request to drop the commit bit. They do not need to
|
|
Anyone may request to drop the commit bit. They do not need to
|
|
provide any reasons.
|
|
provide any reasons.
|
... | @@ -102,7 +102,7 @@ We should review the list of people who have the commit bit every |
... | @@ -102,7 +102,7 @@ We should review the list of people who have the commit bit every |
|
When a staff member leaves the network team, we will remove their
|
|
When a staff member leaves the network team, we will remove their
|
|
commit bit, unless they still need to use it regularly.
|
|
commit bit, unless they still need to use it regularly.
|
|
|
|
|
|
=== 6. Who Can Use This New Process? ===
|
|
### 6. Who Can Use This New Process?
|
|
|
|
|
|
Network Team staff members who are regularly doing the CI, Coder, or
|
|
Network Team staff members who are regularly doing the CI, Coder, or
|
|
Reviewer roles can get the commit bit using this process.
|
|
Reviewer roles can get the commit bit using this process.
|
... | | ... | |