Last modified: 2014-03-20 05:04:05 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T60717, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 58717 - Instance 'delete' link doesn't always delete the ldap entry for an instance
Instance 'delete' link doesn't always delete the ldap entry for an instance
Status: RESOLVED FIXED
Product: Wikimedia Labs
Classification: Unclassified
wikitech-interface (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-12-19 21:48 UTC by Andrew Bogott
Modified: 2014-03-20 05:04 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Andrew Bogott 2013-12-19 21:48:40 UTC
The javascript api calls 'deleteInstance' on the instance object but skips the other steps that would otherwise be done in tryDeleteSubmit.  This means we're leaking ldap host records.
Comment 1 Andrew Bogott 2013-12-19 21:50:31 UTC
Probably I should audit all of the API calls for similar problems.
Comment 2 Andrew Bogott 2013-12-20 15:39:52 UTC
Well.. I can't reproduce this today.
Comment 3 Andrew Bogott 2013-12-27 18:53:41 UTC
And now I can again.  It's intermittent, the ldap delete is sometimes failing without explanation.

The good news, we are at least /trying/ to do the ldap delete in all cases.  Well... maybe that's bad news.
Comment 4 Andrew Bogott 2014-03-19 18:36:03 UTC
It is intermittent.   When it fails, the opendj log looks like this:

DELETE RES conn=2907494 op=3 msgID=4 result=50 message="The entry dc=i-0000019f.eqiad.wmflabs,ou=hosts,dc=wikimedia,dc=org cannot be deleted due to insufficient access rights" etime=0

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links