New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 727227 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Access request to webrtc.DEPS repo

Project Member Reported by mbonadei@chromium.org, May 29 2017

Issue description

Hi, 

I work on WebRTC and this morning I was not able to land this CL: 
https://codereview.chromium.org/2913583002/.

Is it possible to request access to all repos in:
https://chromium.googlesource.com/chromium/deps/webrtc/ 

Thanks,

Mirko
 
Cc: kjellander@chromium.org
Cc: ehmaldonado@chromium.org oprypin@chromium.org charujain@chromium.org
Is it also possible to grant the same permissions to:

- charujain@
- ehmaldonado@
- oprypin@

Thanks

Comment 3 by aga...@chromium.org, May 30 2017

Two questions:
1) You say "all repos in...", but the link only lists a single repo. Do you only want access to webrtc.DEPS, or are there other repos as well?
2) Who is approving this ACL change? None of the requested people are in https://chrome-infra-auth.appspot.com/auth/groups/project-webrtc-admins, so I'll need approval from someone who is.
1) Yes, there is only one repo. The request is misspelled.
2) I think that kjellander@ can approve the ACL change.

Thanks.
Owner: kjellander@chromium.org
Status: Assigned (was: Untriaged)
Summary: Access request to webrtc.DEPS repo (was: Access request to all the webrtc repos)
kjellander: please approve this change before a git admin makes it
Owner: ----
Status: Untriaged (was: Assigned)
Somehow I missed this notification. I approve this, yes. They're all a part of the WebRTC EngProd team.

We're going to try to phase out webrtc.DEPS, but we need access to do so (I don't want to be the bottleneck).

Comment 7 by emso@chromium.org, Jun 9 2017

Owner: emso@chromium.org
Status: Started (was: Untriaged)

Comment 8 by emso@chromium.org, Jun 9 2017

From discussing with kjellander@ offline, all four users are members of webrtc-committers. The current configuration gives Push rights for chromium-committers by inheriting from chromium/deps. I've explicitly added chromium-committers together with webrtc-committers.


Status: Verified (was: Started)
https://codereview.chromium.org/2904413002/ was submitted successfully.

Sign in to add a comment