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)
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)
(deleted),
text/x-python
|
Details | |
(deleted),
patch
|
dustin
:
review+
catlee
:
checked-in+
|
Details | Diff | Splinter Review |
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.
Comment 1•12 years ago
|
||
I thought we were going to try to get rid of tools altogether (which I believe we can).
Reporter | ||
Comment 2•12 years ago
|
||
For tests we can get rid of tools altogether (another bug!), but plenty of build stuff relies on tools.
Reporter | ||
Updated•12 years ago
|
Severity: normal → major
Priority: -- → P2
Reporter | ||
Comment 3•12 years ago
|
||
standalone version of hgtool.py
Reporter | ||
Comment 4•12 years ago
|
||
going to rely on this for mozharness's hgtool support
Blocks: 777530
Reporter | ||
Comment 5•12 years ago
|
||
Attachment #664306 -
Flags: review?
Reporter | ||
Updated•12 years ago
|
Assignee: nobody → catlee
Reporter | ||
Comment 6•12 years ago
|
||
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 7•12 years ago
|
||
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-
Reporter | ||
Comment 8•12 years ago
|
||
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)
Reporter | ||
Comment 9•12 years ago
|
||
(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 10•12 years ago
|
||
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+
Reporter | ||
Updated•12 years ago
|
Attachment #664492 -
Flags: checked-in+
Reporter | ||
Comment 11•11 years ago
|
||
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
Assignee | ||
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
Updated•10 years ago
|
Whiteboard: [hgtool] → [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/2406] [hgtool]
Updated•10 years ago
|
Whiteboard: [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/2406] [hgtool] → [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/2412] [hgtool]
Comment 12•7 years ago
|
||
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
Assignee | ||
Updated•6 years ago
|
Component: Platform Support → Buildduty
Product: Release Engineering → Infrastructure & Operations
Updated•5 years ago
|
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•