Last modified: 2014-02-26 14:43:08 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 T54258, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 52258 - Obsolete scripts in /usr/local/bin need to be "managed"
Obsolete scripts in /usr/local/bin need to be "managed"
Status: RESOLVED FIXED
Product: Wikimedia Labs
Classification: Unclassified
tools (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: Marc A. Pelletier
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-07-30 01:51 UTC by Tim Landscheidt
Modified: 2014-02-26 14:43 UTC (History)
1 user (show)

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


Attachments

Description Tim Landscheidt 2013-07-30 01:51:32 UTC
The tool afch-updater had hard-coded /usr/*local*/bin/jsub (which was the advertised location, so no blame here); on tools-webserver-01 that was an obsolete version of jsub with bugs.  I fixed that by replacing the grid scripts with symlinks to /usr/bin.

We should manage the compatibility symlinks (I'm fine with removal as well) in /usr/local/bin on the *whole* cluster properly with Puppet, or place small wrappers there that log the caller program's name and user, and then pass the arguments to the "normal" scripts, so that we can track down people using the old location.

On tools-webserver-01, the following files are in /usr/local/bin *and* in /usr/bin:

| check-raid.py
| cpan
| cygdb
| cython
| job
| jstart
| jstop
| jsub
Comment 1 Tim Landscheidt 2013-07-30 18:00:46 UTC
Second thought regarding wrappers: /usr/local/bin has priority in many PATHs, so if we put a wrapper in there, it may be preferred over the "original" in /usr/bin, so the wrapper would have to do a lot of guesswork to determine if a user really called the wrong script.

Proposal: Put symlinks in Puppet, advise users on labs-l to check PATHs and absolute paths by ... end of September (vacation season in the northern hemisphere ATM), then "ensure => absent", then remove Puppet section.
Comment 2 Gerrit Notification Bot 2013-08-02 02:46:37 UTC
Change 77234 had a related patch set uploaded by Tim Landscheidt:
Tools: Manage obsolete files in /usr/local/bin

https://gerrit.wikimedia.org/r/77234
Comment 3 Gerrit Notification Bot 2013-08-23 17:02:58 UTC
Change 77234 merged by coren:
Tools: Manage obsolete files in /usr/local/bin

https://gerrit.wikimedia.org/r/77234
Comment 4 Tim Landscheidt 2014-02-26 13:29:31 UTC
Reopening for "advise users on labs-l to check PATHs and absolute paths by ... end of September (vacation season in the northern hemisphere ATM), then "ensure => absent", then remove Puppet section." with September replaced by ... let's say April.
Comment 5 Gerrit Notification Bot 2014-02-26 13:29:58 UTC
Change 115612 had a related patch set uploaded by Tim Landscheidt:
Tools: Restore local symlinks for jobutils

https://gerrit.wikimedia.org/r/115612
Comment 6 Gerrit Notification Bot 2014-02-26 14:39:39 UTC
Change 115612 abandoned by Tim Landscheidt:
Tools: Restore local symlinks for jobutils

https://gerrit.wikimedia.org/r/115612
Comment 7 Tim Landscheidt 2014-02-26 14:43:08 UTC
Cf. Gerrit change #115612; users advised per http://permalink.gmane.org/gmane.org.wikimedia.labs/2140, "ensure => absent" will be done by data center move.

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


Navigation
Links