Migration.php strands users on db connection fail

RESOLVED FIXED

Status

Cloud Services
Server: Other
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: zandr, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [qa-])

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
As it stands now, if migration.php can't connect to the database server, it simply errors out, leaving the user being migrated stranded on the bogus/migration node. It should clean up and put them back on their original node.
Created attachment 479068 [details] [diff] [review]
Put back the old node value in case the SQL DB was not reachable

I created a patch with the error Zandr gave us on IRC:

 moving XXX (to XXX.services.mozilla.com)<p>unable to connect to source db: SQLSTATE[00000] [1040] Too many connections</p>

This patch will put back in LDAP the old node in case mysql_move_user() fails
Attachment #479068 - Flags: review?(telliott)
Attachment #479068 - Flags: review?(telliott) → review+
(Reporter)

Comment 2

8 years ago
Telliott pushed http://hg.mozilla.org/services/admin-scripts/rev/9455cf3f257b

In use now, does what it says on the tin.
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED

Updated

8 years ago
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.