remove the build-tools package from opsi/puppet and blow away dir

RESOLVED FIXED

Status

Release Engineering
General
RESOLVED FIXED
7 years ago
4 years ago

People

(Reporter: dustin, Assigned: dustin)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

7 years ago
buildbot-tac.py is the only thing that is used in /tools/build-tools, which is installed by this package.  This makes it pretty fiddly to upgrade the script, and there's no reason to build a package for a simple python script.

Bug 611149 will remove the need for this package, at which point we should remove the package and delete the directory it installs into.
(Assignee)

Comment 1

7 years ago
Changing to depend on the dynamic slave alloc bug, in hopes that it will involve removing buildbot-tac.py.
Depends on: 605278
No longer depends on: 611149
(Assignee)

Updated

7 years ago
Depends on: 616003
(Assignee)

Updated

7 years ago
Depends on: 616351, 616352
No longer depends on: 605278
(Assignee)

Updated

7 years ago
Depends on: 629690
(Assignee)

Updated

7 years ago
Depends on: 629692
(Assignee)

Updated

7 years ago
No longer depends on: 616352
(Assignee)

Comment 2

7 years ago
I think this is ready to do now - win64 isn't in OPSI, so it doesn't block this bug.
No longer depends on: 629690
(Assignee)

Comment 3

7 years ago
Created attachment 542350 [details] [diff] [review]
m615301-puppet-manifests-p1-r1.patch

Ben, how confident are you that we don't use things under /tools/build-tools for other purposes?  A check of the build-central MXR doesn't reveal anything obvious:
  http://mxr.mozilla.org/build-central/search?string=build-tools
Attachment #542350 - Flags: review?(bhearsum)
(Assignee)

Comment 4

7 years ago
Huh, I'm not sure *where* this is installed with OPSI.  I guess we can just let it die.
(Assignee)

Comment 5

7 years ago
Created attachment 542362 [details] [diff] [review]
m615301-rpm-sources-p1-r1.patch
Attachment #542362 - Flags: review?(bhearsum)
(In reply to comment #4)
> Huh, I'm not sure *where* this is installed with OPSI.  I guess we can just
> let it die.

It's installed on the Windows build machines, to d:\tools:
D:\>dir tools
 Volume in drive D is tools
 Volume Serial Number is 048A-CE48

 Directory of D:\tools

02/23/2010  03:12 PM    <DIR>          .
02/23/2010  03:12 PM    <DIR>          ..
02/23/2010  03:12 PM    <DIR>          .hg
02/23/2010  03:12 PM             3,633 .hgtags
02/23/2010  03:12 PM    <DIR>          breakpad
02/23/2010  03:12 PM    <DIR>          buildbot-helpers
02/23/2010  03:12 PM    <DIR>          buildfarm
02/23/2010  03:12 PM    <DIR>          cdmaker
02/23/2010  03:12 PM    <DIR>          clobberer
02/23/2010  03:12 PM    <DIR>          release
02/23/2010  03:12 PM    <DIR>          stage
               1 File(s)          3,633 bytes
              10 Dir(s)  18,198,835,200 bytes free


(In reply to comment #3)
> Created attachment 542350 [details] [diff] [review] [review]
> m615301-puppet-manifests-p1-r1.patch
> 
> Ben, how confident are you that we don't use things under /tools/build-tools
> for other purposes?  A check of the build-central MXR doesn't reveal
> anything obvious:
>   http://mxr.mozilla.org/build-central/search?string=build-tools

I don't know of anything else on POSIX machines that uses /tools/build-tools. On Windows, the OPSI host key generator (http://hg.mozilla.org/build/opsi-package-sources/file/fcce4578d03f/hostkey-generator/CLIENT_DATA/hostkey.ins) uses d:\tools to reset the host key in fresh clones.
Attachment #542350 - Flags: review?(bhearsum) → review+
Attachment #542362 - Flags: review?(bhearsum) → review+
(Assignee)

Comment 7

7 years ago
Both are landed, and puppet servers restarted.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.