Install run-task and hg robustcheckout on desktop-test image

RESOLVED FIXED

Status

RESOLVED FIXED
2 years ago
9 months ago

People

(Reporter: gps, Assigned: gps)

Tracking

(Blocks: 2 bugs)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(10 attachments)

58 bytes, text/x-review-board-request
dustin
: review+
Details
58 bytes, text/x-review-board-request
dustin
: review+
Details
58 bytes, text/x-review-board-request
dustin
: review+
Details
58 bytes, text/x-review-board-request
dustin
: review+
Details
58 bytes, text/x-review-board-request
dustin
: review+
Details
58 bytes, text/x-review-board-request
dustin
: review+
Details
58 bytes, text/x-review-board-request
dustin
: review+
Details
58 bytes, text/x-review-board-request
dustin
: review+
Details
58 bytes, text/x-review-board-request
dustin
: review+
Details
58 bytes, text/x-review-board-request
dustin
: review+
Details
(Assignee)

Description

2 years ago
Let's get run-task and `hg robustcheckout` installed on the desktop-test Docker image.
(Assignee)

Comment 1

2 years ago
Created attachment 8778318 [details]
Bug 1291365 - Refactor install-mercurial.sh to detect OS version;

Currently, install-mercurial.sh only supports Ubuntu 16.04. In
preparation for supporting other distros, we add code for performing
OS/distro detection.

Review commit: https://reviewboard.mozilla.org/r/69624/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/69624/
Attachment #8778318 - Flags: review?(dustin)
Attachment #8778319 - Flags: review?(dustin)
Attachment #8778320 - Flags: review?(dustin)
Attachment #8778321 - Flags: review?(dustin)
(Assignee)

Comment 2

2 years ago
Created attachment 8778319 [details]
Bug 1291365 - Support installing Mercurial on Ubuntu 12.04;

I produced Mercurial .deb packages for Ubuntu 12.04 and uploaded them to
tooltool. I have updated the Mercurial installation script to detect
Ubuntu 12.04 and install the appropriate Mercurial packages.

Review commit: https://reviewboard.mozilla.org/r/69626/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/69626/
(Assignee)

Comment 3

2 years ago
Created attachment 8778320 [details]
Bug 1291365 - Install tooltool securely in ubuntu1204-test image;

More determinism. More security. More reliable. Vendoring.

Review commit: https://reviewboard.mozilla.org/r/69628/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/69628/
(Assignee)

Comment 4

2 years ago
Created attachment 8778321 [details]
Bug 1291365 - Install Mercurial via install-mercurial.sh in ubuntu1204-test image;

This also bumps the Mercurial version from 3.7.3 to 3.8.4.

Review commit: https://reviewboard.mozilla.org/r/69630/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/69630/
Blocks: 1292561
(Assignee)

Comment 5

2 years ago
Created attachment 8778334 [details]
Bug 1291365 - Add run-task to ubuntu1204-test image;

Review commit: https://reviewboard.mozilla.org/r/69638/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/69638/
Attachment #8778334 - Flags: review?(dustin)
Attachment #8778335 - Flags: review?(dustin)
Attachment #8778336 - Flags: review?(dustin)
Attachment #8778337 - Flags: review?(dustin)
Attachment #8778338 - Flags: review?(dustin)
(Assignee)

Comment 6

2 years ago
Created attachment 8778335 [details]
Bug 1291365 - Specify Ubuntu version in ubuntu1604-test image;

Without this, we'll use the latest Ubuntu release at image build time!

Review commit: https://reviewboard.mozilla.org/r/69640/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/69640/
(Assignee)

Comment 7

2 years ago
Created attachment 8778336 [details]
Bug 1291365 - Install tooltool securely in ubuntu1604-test image;

And make it deterministic since we are using a vendored file.

Review commit: https://reviewboard.mozilla.org/r/69642/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/69642/
(Assignee)

Comment 8

2 years ago
Created attachment 8778337 [details]
Bug 1291365 - Use install-mercurial.sh in ubuntu1604-test;

Review commit: https://reviewboard.mozilla.org/r/69644/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/69644/
(Assignee)

Comment 9

2 years ago
Created attachment 8778338 [details]
Bug 1291365 - Add run-task to ubuntu1604-test image;

Review commit: https://reviewboard.mozilla.org/r/69646/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/69646/
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 20

2 years ago
mozreview-review
Comment on attachment 8778318 [details]
Bug 1291365 - Refactor install-mercurial.sh to detect OS version;

https://reviewboard.mozilla.org/r/69624/#review67252
Attachment #8778318 - Flags: review?(dustin) → review+

Comment 21

2 years ago
mozreview-review
Comment on attachment 8778319 [details]
Bug 1291365 - Support installing Mercurial on Ubuntu 12.04;

https://reviewboard.mozilla.org/r/69626/#review67254
Attachment #8778319 - Flags: review?(dustin) → review+

Comment 22

2 years ago
mozreview-review
Comment on attachment 8778320 [details]
Bug 1291365 - Install tooltool securely in ubuntu1204-test image;

https://reviewboard.mozilla.org/r/69628/#review67256

::: testing/docker/ubuntu1204-test/Dockerfile:8
(Diff revision 2)
>  
>  RUN useradd -d /home/worker -s /bin/bash -m worker
>  WORKDIR /home/worker
>  
> +# %include testing/docker/recipes/tooltool.py
> +ADD topsrcdir/testing/docker/recipes/tooltool.py /tmp/tooltool.py

I fear someone is going to notice "Hey, tooltool's installed in `/tmp` so let's just use that!".  Could this end up under `/setup` so it gets removed at the end of `system-setup.sh`?  It's a minor issue, so feel free to not change this.
Attachment #8778320 - Flags: review?(dustin) → review+

Comment 23

2 years ago
mozreview-review
Comment on attachment 8778321 [details]
Bug 1291365 - Install Mercurial via install-mercurial.sh in ubuntu1204-test image;

https://reviewboard.mozilla.org/r/69630/#review67258
Attachment #8778321 - Flags: review?(dustin) → review+

Comment 24

2 years ago
mozreview-review
Comment on attachment 8778334 [details]
Bug 1291365 - Add run-task to ubuntu1204-test image;

https://reviewboard.mozilla.org/r/69638/#review67260
Attachment #8778334 - Flags: review?(dustin) → review+

Comment 25

2 years ago
mozreview-review
Comment on attachment 8778335 [details]
Bug 1291365 - Specify Ubuntu version in ubuntu1604-test image;

https://reviewboard.mozilla.org/r/69640/#review67262
Attachment #8778335 - Flags: review?(dustin) → review+

Comment 26

2 years ago
mozreview-review
Comment on attachment 8778336 [details]
Bug 1291365 - Install tooltool securely in ubuntu1604-test image;

https://reviewboard.mozilla.org/r/69642/#review67264
Attachment #8778336 - Flags: review?(dustin) → review+

Comment 27

2 years ago
mozreview-review
Comment on attachment 8778337 [details]
Bug 1291365 - Use install-mercurial.sh in ubuntu1604-test;

https://reviewboard.mozilla.org/r/69644/#review67266
Attachment #8778337 - Flags: review?(dustin) → review+

Comment 28

2 years ago
mozreview-review
Comment on attachment 8778338 [details]
Bug 1291365 - Add run-task to ubuntu1604-test image;

https://reviewboard.mozilla.org/r/69646/#review67268
Attachment #8778338 - Flags: review?(dustin) → review+

Comment 29

2 years ago
mozreview-review
Comment on attachment 8778968 [details]
Bug 1291365 - Remove hgrc file from desktop1604-test image;

https://reviewboard.mozilla.org/r/70046/#review67270
Attachment #8778968 - Flags: review?(dustin) → review+
(Assignee)

Comment 30

2 years ago
mozreview-review-reply
Comment on attachment 8778320 [details]
Bug 1291365 - Install tooltool securely in ubuntu1204-test image;

https://reviewboard.mozilla.org/r/69628/#review67256

> I fear someone is going to notice "Hey, tooltool's installed in `/tmp` so let's just use that!".  Could this end up under `/setup` so it gets removed at the end of `system-setup.sh`?  It's a minor issue, so feel free to not change this.

Makes sense. Will fix.
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 41

2 years ago
Pushed by gszorc@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/62bd3cf6661d
Refactor install-mercurial.sh to detect OS version; r=dustin
https://hg.mozilla.org/integration/autoland/rev/df3c13ea7d3e
Support installing Mercurial on Ubuntu 12.04; r=dustin
https://hg.mozilla.org/integration/autoland/rev/dba9f54f0512
Install tooltool securely in ubuntu1204-test image; r=dustin
https://hg.mozilla.org/integration/autoland/rev/51ac9d25dbe9
Install Mercurial via install-mercurial.sh in ubuntu1204-test image; r=dustin
https://hg.mozilla.org/integration/autoland/rev/dca57fa9b9e2
Add run-task to ubuntu1204-test image; r=dustin
https://hg.mozilla.org/integration/autoland/rev/fe0fc504b4b4
Specify Ubuntu version in ubuntu1604-test image; r=dustin
https://hg.mozilla.org/integration/autoland/rev/5111fd8cb91d
Install tooltool securely in ubuntu1604-test image; r=dustin
https://hg.mozilla.org/integration/autoland/rev/3cf365cf9b90
Use install-mercurial.sh in ubuntu1604-test; r=dustin
https://hg.mozilla.org/integration/autoland/rev/fdc5c16b3f8e
Add run-task to ubuntu1604-test image; r=dustin
https://hg.mozilla.org/integration/autoland/rev/470f72a4827c
Remove hgrc file from desktop1604-test image; r=dustin

Updated

9 months ago
Product: TaskCluster → Firefox Build System
You need to log in before you can comment on or make changes to this bug.