Problem with Abhishek Paliwal's Gerrit account(s)

Bug #949100 reported by Paul Sokolovsky
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Linaro Android Infrastructure
Fix Released
High
Paul Sokolovsky

Bug Description

It turned out that Abhishek Paliwal got 2 Gerrit accounts with different IDs, backed by different OpenIDs. Account with non-Launchpad OpenID is valid, but thus untrusted, it can be added to some group (Administrators requested), but membership is not really active. And Launchpad-backed account is not completely active as Gerrit thinks, trying to add it to a group leads to an error:

Code Review - Unknown User
Abhishek Paliwal <email address hidden> is not a registered user.

Suspected reason was unconfirmed email, but confirming it, the error persists.

Data about accounts:

Launchpad-backed account:

<pabhishek> If I login using Launchpad OpenID(https://launchpad.net/~abhishek-paliwal) i see following info:
<pabhishek> Username: pabhishek
<pabhishek> Full Name: Abhishek Paliwal
<pabhishek> Email Address: <email address hidden>
<pabhishek> Registered: Sep 8, 2011 2:29 PM
<pabhishek> Account ID: 1000028

Another (gmail-backed) account:

<pabhishek> username: pabhisek
<pabhishek> Full Name Abhishek Paliwal
<pabhishek> Email Address <email address hidden>
<pabhishek> Registered Jan 17, 2012 8:40 PM
<pabhishek> 1000051

Revision history for this message
Paul Sokolovsky (pfalcon) wrote :

Course of action is to lookup in the source code what causes Gerrit to report that error, then look in the DB for "bad" data, then decide about approach to fix it.

Changed in linaro-android-infrastructure:
importance: Undecided → High
status: New → Confirmed
assignee: nobody → Paul Sokolovsky (pfalcon)
Revision history for this message
Paul Sokolovsky (pfalcon) wrote :

Issue 1061: Add ability to remove user account OR ask for confirmation when creating one http://code.google.com/p/gerrit/issues/detail?id=1061

Issue 567: Merging Gerrit User Accounts http://code.google.com/p/gerrit/issues/detail?id=567

Changed in linaro-android-infrastructure:
status: Confirmed → In Progress
Revision history for this message
Paul Sokolovsky (pfalcon) wrote :

Turned out Gerrit uses the same (misguiding) error message when there're no user account matching, or more than 1 account matching full name/email. Fortunately, it allows to type SSH username or user id directly. This is documented in https://wiki.linaro.org/Platform/Android/Gerrit#Known_Issues

Abhishek's correct account was added to Administrators.

TODO: Prepare SQL to have the old account renamed out of the way.

Changed in linaro-android-infrastructure:
milestone: none → 2012.03
Revision history for this message
Paul Sokolovsky (pfalcon) wrote :

Ok, in the end, the culprit was indeed duplicate associated mails - they have precedence when looking up.

Performed:

ssh -p 29418 android.git.linaro.org gerrit flush-caches --all

update accounts set inactive='Y' where account_id=1000051;
update accounts set inactive='Y' where account_id=1000064;
update accounts set full_name='??Abhishek Paliwal' where account_id in (1000051, 1000064);
delete from account_external_ids where account_id=1000051;
delete from account_external_ids where account_id=1000064;

ssh -p 29418 android.git.linaro.org gerrit flush-caches --all

Changed in linaro-android-infrastructure:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.