Closed Bug 764077 Opened 12 years ago Closed 7 years ago

Install hgtool as separate package on build/test slaves

Categories

(Infrastructure & Operations Graveyard :: CIDuty, task, P2)

x86_64
Linux

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: catlee, Unassigned)

References

Details

(Whiteboard: [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/2412] [hgtool])

Attachments

(2 files, 1 obsolete file)

Let's deploy hgtool (and its dependencies) on the build/test slaves in /usr/local/bin or equivalent. Then we can use it to make sure our tools checkouts are up to date withtout the expensive clobber/re-clone we do right now.
I thought we were going to try to get rid of tools altogether (which I believe we can).
For tests we can get rid of tools altogether (another bug!), but plenty of build stuff relies on tools.
Blocks: 758216
Severity: normal → major
Priority: -- → P2
Attached file standalone hgtool.py (deleted) —
standalone version of hgtool.py
going to rely on this for mozharness's hgtool support
Blocks: 777530
Attachment #664306 - Flags: review?
Assignee: nobody → catlee
Comment on attachment 664306 [details] [diff] [review] puppet manifests for deploying standalone hgtool as per IRC, not sure about naming here...maybe tools::hgtool or utils::hgtool is a better module name?
Attachment #664306 - Flags: review? → review?(dustin)
Comment on attachment 664306 [details] [diff] [review] puppet manifests for deploying standalone hgtool Review of attachment 664306 [details] [diff] [review]: ----------------------------------------------------------------- The comment below actually sums it up - if this is the authoritative place for hgtool, then it deserves a top-level module and uncompressed sources so we can see the diffs. If not, it should be considered a package, preferably as an RPM, but at least under `packages::mozilla::hgtool`. I think that the same applies to tooltool, and that build/puppet is not the authoritative source for tooltool, but if that needs to be moved too, we should do so. ::: modules/hgtool/files/hgtool.py @@ +1,2 @@ > +#!/usr/bin/python > +### Compressed module sources ### I don't think this is a great idea. If hgtool must be distributed like this, then let's build an RPM for it. Otherwise, these files should be included in plaintext in puppet so that we'll see reasonable diffs if/when they change. That really depends on whether this is the "authoritative" copy of hgtool (in which case it should be diffable) or just a derivative copy of something in tools/ (in which case it should be packaged).
Attachment #664306 - Flags: review?(dustin) → review-
I think creating an rpm (/dmg) is overkill for this. The scripts and instructions for generating the packed version from the authoritative copy in build/tools will be added to build/tools.
Attachment #664306 - Attachment is obsolete: true
Attachment #664492 - Flags: review?(dustin)
(In reply to Chris AtLee [:catlee] from comment #8) > I think creating an rpm (/dmg) is overkill for this. The scripts and > instructions for generating the packed version from the authoritative copy > in build/tools will be added to build/tools. I take this back. Having an rpm will make it easier to get hgtool running _inside_ mock. I don't know if that's something we need right away or not.
Comment on attachment 664492 [details] [diff] [review] move into packages::mozilla::hgtool This looks fine to commit now; it will be an easy change to replace this with an RPM.
Attachment #664492 - Flags: review?(dustin) → review+
Attachment #664492 - Flags: checked-in+
Works on all platforms that are on puppetagain. We still need it on Windows.
Assignee: catlee → nobody
Component: Release Engineering: Automation (General) → Release Engineering: Platform Support
QA Contact: catlee → coop
Product: mozilla.org → Release Engineering
Whiteboard: [hgtool] → [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/2406] [hgtool]
Whiteboard: [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/2406] [hgtool] → [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/2412] [hgtool]
Probably not going to do this since all of machines we manage are dead or dying?
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Component: Platform Support → Buildduty
Product: Release Engineering → Infrastructure & Operations
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: