Closed
Bug 1279040
Opened 8 years ago
Closed 8 years ago
Move the interactive wizard to testing/taskcluster/scripts and set it up at runtime
Categories
(Taskcluster Graveyard :: Docker Images, defect)
Taskcluster Graveyard
Docker Images
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ahal, Assigned: ahal)
References
Details
Attachments
(1 file)
In bug 1250904 I baked the run-wizard command into the desktop-test image. Dustin points out I should have downloaded it instead rather than baking it in, oops.
Assignee | ||
Comment 1•8 years ago
|
||
The run-wizard binary (used by interactive workers) will likely need to change relatively frequently. Therefore, it should be baked directly into the docker image. This patch instead downloads it from the appropriate commit on hg.mozilla.org, only when needed. Review commit: https://reviewboard.mozilla.org/r/60834/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/60834/
Attachment #8765488 -
Flags: review?(dustin)
Comment 2•8 years ago
|
||
Comment on attachment 8765488 [details] Bug 1279040 - Move 'run-wizard' binary out of docker image, https://reviewboard.mozilla.org/r/60834/#review57666 I assume the download is necessary because this can work even in a case where there is no gecko checkout (e.g., test runs)?
Attachment #8765488 -
Flags: review?(dustin) → review+
Assignee | ||
Comment 3•8 years ago
|
||
Yes, exactly. All this is only for things that use the desktop-test image.
Pushed by ahalberstadt@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/53a8197a49eb Move 'run-wizard' binary out of docker image, r=dustin
Comment 5•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/53a8197a49eb
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Taskcluster → Taskcluster Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•