opendev
/
jeepyb
Code Issues Proposed changes

Username is on the connection objet 

We're manipulating a Gerrit object not a GerritConnection object.
The username property is found on the Gerrit object.

Change-Id: I9b4c4cc0dacf456c1cb71c3798141ac726b1db71
This commit is contained in:
Monty Taylor 2020-03-30 08:03:44 -05:00
parent ce25e06d8f
commit 77bed1ae7a
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
jeepyb/cmd/manage_projects.py
View File

@ -219,9 +219,9 @@ def get_group_uuid(gerrit, group):
return GERRIT_SYSTEM_GROUPS[group] return GERRIT_SYSTEM_GROUPS[group]
gerrit.createGroup(group) gerrit.createGroup(group)
for user in list(gerrit.listMembers(group)): for user in list(gerrit.listMembers(group)):
if gerrit.username == user['username']: if gerrit.connection.username == user['username']:
# Gerrit now adds creating user to groups. We don't want that. # Gerrit now adds creating user to groups. We don't want that.
gerrit.removeMember(group, gerrit.username) gerrit.removeMember(group, gerrit.connection.username)
break break
uuid = _get_group_uuid(gerrit, group) uuid = _get_group_uuid(gerrit, group)
if uuid: if uuid: