1164144 - Commit access level 1 for Ritu Kothari

Status: RESOLVED FIXED

(mozilla.org :: Repository Account Requests, task)

Description

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Commit access level 1 for Ritu Kothari

Comment 1

[Liz Henry (:lizzard)]

Ritu has just joined Mozilla as a release manager and we'd like her to start the process of getting set up with a committer agreement. 

Ritu, you will need to go through the steps described here:
* http://www.mozilla.org/hacking/committer/
* https://www.mozilla.org/en-US/about/governance/policies/commit/access-policy/

Comment 2

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Attachment: id_rsa.pub

public SSH key

Comment 3

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Based on the commit access policy, I will need commit access level 3. 

Here is what I hope to do: Submit a patch for bug 1167410. This would require me to have commit access to mozilla-central and to push the patch on the Try server (not necessarily in that order).

Comment 4

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Marcia, is there anything else that I need to provide? Please let me know.

Comment 5

[Josh Matthews [:jdm]]

Level 3 requests require two module peer/owners as vouchers.

Comment 6

[Lawrence Mandel [:lmandel] (use needinfo)]

I vouch for Ritu.

Comment 7

[Sylvestre Ledru [:Sylvestre]]

I vouch for Ritu too.

Comment 8

[Liz Henry (:lizzard)]

Vouching for Ritu.

Comment 9

[Marcia Knous [:marcia]]

One thing missing from this request - Ritu needs to agree to the Commit access policy in this bug - https://www.mozilla.org/en-US/about/governance/policies/commit/access-policy/.

Second, regarding L3 access and tryserver - for tryserver, only Level 1 is required and that seems to be what is requested in the bug title. But in Comment 3 she also wants to commit code to Mozilla central. If so, I would like to see more of a history of patches and checkins since she is new to the project - we don't typically give out L3 access without some kind of history of patches being submitted.

Comment 10

[Lawrence Mandel [:lmandel] (use needinfo)]

(In reply to Marcia Knous [:marcia - use ni] from comment #9)
> One thing missing from this request - Ritu needs to agree to the Commit
> access policy in this bug -
> https://www.mozilla.org/en-US/about/governance/policies/commit/access-policy/

Ritu - For your review.

> Second, regarding L3 access and tryserver - for tryserver, only Level 1 is
> required and that seems to be what is requested in the bug title. But in
> Comment 3 she also wants to commit code to Mozilla central. If so, I would
> like to see more of a history of patches and checkins since she is new to
> the project - we don't typically give out L3 access without some kind of
> history of patches being submitted.

I think that's reasonable for m-c although I'm not sure is applied consistently. However, the immediate need is for Ritu to have commit access to the product-details SVN repo.

Comment 11

[Marcia Knous [:marcia]]

(In reply to Lawrence Mandel [:lmandel] (use needinfo) from comment #10)
> (In reply to Marcia Knous [:marcia - use ni] from comment #9)
> > One thing missing from this request - Ritu needs to agree to the Commit
> > access policy in this bug -
> > https://www.mozilla.org/en-US/about/governance/policies/commit/access-policy/
> 
> Ritu - For your review.
> 
> > Second, regarding L3 access and tryserver - for tryserver, only Level 1 is
> > required and that seems to be what is requested in the bug title. But in
> > Comment 3 she also wants to commit code to Mozilla central. If so, I would
> > like to see more of a history of patches and checkins since she is new to
> > the project - we don't typically give out L3 access without some kind of
> > history of patches being submitted.
> 
> I think that's reasonable for m-c although I'm not sure is applied
> consistently. However, the immediate need is for Ritu to have commit access
> to the product-details SVN repo.


Lawrence - Just to clarify - It is applied consistently for L3 - I have been doing these requests for a long time and we have kept the same standard for L3 commit access. 

Bug 1164158 seems to be on file for the other request - as soon as she approves the Commit form I can move that one forward.

Comment 12

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

(In reply to Marcia Knous [:marcia - use ni] from comment #9)
> One thing missing from this request - Ritu needs to agree to the Commit
> access policy in this bug -
> https://www.mozilla.org/en-US/about/governance/policies/commit/access-policy/
> .
> 
> Second, regarding L3 access and tryserver - for tryserver, only Level 1 is
> required and that seems to be what is requested in the bug title. But in
> Comment 3 she also wants to commit code to Mozilla central. If so, I would
> like to see more of a history of patches and checkins since she is new to
> the project - we don't typically give out L3 access without some kind of
> history of patches being submitted.

Marcia, is there something on the commit access policy that I need to "I agree" on? I have read that page and comply with the access policy. Please let me know if I need to email/"submit" a form to indicate compliance.

Comment 13

[Marcia Knous [:marcia]]

Level 1 access granted. When you are ready for Level 3 please open a new bug.

Comment 14

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Thanks! I will.

Comment 15

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Marcia, as part of managing Beta41, I need to update early_beta flag in mozilla-beta repository build\defines.sh file. Can I get access level3 access?

Comment 16

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Filed bug 1199741 as per comment 13.