My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 2388: Make Gerrit UI and git push consistant on assignment of groups
2 people starred this issue and may be notified of changes. Back to list
Status:  Submitted
Owner:  zaro0508
Closed:  Nov 2014


Sign in to add a comment
 
Project Member Reported by zaro0508, Jan 9, 2014
Assigning groups to permissions works inconsistently between Gerrit UI and git push.

Affected Version: 2.8

What steps will reproduce the problem?
1. create a new gerrit site
2. clone All-Projects
*Note - the groups file does not contain an entry for "Change Owner"
3. edit the project.config and add a group (which does not exist in the groups file) to a permission.  

Example: 
  [access "refs/heads/*"]
        label-Code-Review = -1..+1 group Change Owner

5. push to All-Projects (git push origin HEAD:refs/meta/config)

What is the expected output? What do you see instead?
You can do this exact same thing using gerrit UI and it works just fine.  However when using command line route it's not allowed because the group isn't in the groups file.  I get this error..

Writing objects: 100% (3/3), 487 bytes | 0 bytes/s, done.
Total 3 (delta 1), reused 0 (delta 0)
remote: Resolving deltas: 100% (1/1)
remote: Processing changes: refs: 1, done
remote: error: Invalid project configuration:
remote: error:   project.config: group "Change Owner" not in groups
To http://localhost:8080/All-Projects
 ! [remote rejected] HEAD -> refs/meta/config (invalid project configuration)
error: failed to push some refs to 'http://localhost:8080/All-Projects'

Jan 9, 2014
Project Member #1 zaro0508
I meant to enter this as an enhancement.
Jan 9, 2014
Project Member #2 david.pu...@sonymobile.com
(No comment was entered for this change.)
Labels: -Type-Bug Type-Feature
Aug 24, 2014
Project Member #3 zaro0508
https://gerrit-review.googlesource.com/#/c/59570
Status: ChangeUnderReview
Owner: zaro0508
Sep 5, 2014
Project Member #4 zaro0508
abandoned change 59570, replaced with  https://gerrit-review.googlesource.com/#/c/59850
Nov 15, 2014
Project Member #5 zaro0508
(No comment was entered for this change.)
Status: Submitted
Sign in to add a comment

Powered by Google Project Hosting