convert all uses of tooltool in automation to use the new relengapi tooltool

RESOLVED FIXED

Status

Release Engineering
General Automation
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: dustin, Assigned: dustin)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(12 attachments, 6 obsolete attachments)

1.45 KB, patch
coop
: review+
Details | Diff | Splinter Review
8.23 KB, patch
jgriffin
: review+
dustin
: checked-in+
Details | Diff | Splinter Review
51.27 KB, patch
mshal
: review+
dustin
: checked-in+
Details | Diff | Splinter Review
2.82 KB, patch
markco
: review+
dustin
: checked-in+
Details | Diff | Splinter Review
59.77 KB, patch
dustin
: review+
dustin
: checked-in+
Details | Diff | Splinter Review
2.70 KB, patch
jlund
: review+
dustin
: checked-in+
Details | Diff | Splinter Review
1.75 KB, patch
markco
: review+
dustin
: checked-in+
Details | Diff | Splinter Review
2.14 KB, patch
markco
: review+
dustin
: checked-in+
Details | Diff | Splinter Review
39 bytes, text/x-review-board-request
mshal
: review+
Details | Review
39 bytes, text/x-review-board-request
mshal
: review+
Details | Review
39 bytes, text/x-review-board-request
mshal
: review+
Details | Review
39 bytes, text/x-review-board-request
mshal
: review+
Details | Review
(Assignee)

Description

2 years ago
The will require changing URLs and using a relengapi token with permission to download public and internal files from tooltool.  The easy bit should be Mozharness.  Slightly harder are buildbot steps and shell scripts running tooltool.  Harder still will be URLs referring directly to files on tooltool.  Those should be converted to use tooltool with a cache, rather than fetching from S3 on every run.
(Assignee)

Updated

2 years ago
Assignee: nobody → dustin
(Assignee)

Comment 1

2 years ago
I have confirmed that the new tooltool.py can download from the old URLs, so the first step will be to deploy the new script everywhere.
(Assignee)

Comment 2

2 years ago
Created attachment 8593445 [details] [diff] [review]
bug1155238.patch

This is just copied from the tooltool repo.
Attachment #8593445 - Flags: review?(mshal)
(Assignee)

Comment 3

2 years ago
1155256
Depends on: 1155256
(Assignee)

Comment 4

2 years ago
Created attachment 8593480 [details] [diff] [review]
bug1155238-mozharness.patch

This is a basic mozharness patch to update just the URLs.  I'll need to figure out how to test this in try, once the new tooltool.py is in place.
Attachment #8593480 - Flags: review?(mshal)
(Assignee)

Comment 5

2 years ago
Created attachment 8593486 [details] [diff] [review]
bug1155238-mozilla-central.patch

I'm not sure whether this would have to ride the trains -- is this used in automation, or just by devs?
Attachment #8593486 - Flags: review?(coop)
(Assignee)

Comment 6

2 years ago
The explicitly-linked files have sha512's

372c89f9dccaf5ee3b9d35fd1cfeb089e1e5db3ff1c04e35aa3adc8800bc61a2ae10e321f37ae7bab20b56e60941f91bb003bcb22035902a73
0748e900821820f1a42e2f1f3fa4d9002ef257c351b9e6b78e7de0ddd0202eace351f440372fbb1ae0b7e69e8361b036f6bd3362df99e67fc585082a311fc0df
4d8d7a37d90c34a2a2fda3066a8fe85c189b183d05389cb957fc6fed31f10a6924e50c1b84488ff61c015293803f58a3aed5d4819374d04c8e0ee2b9e3997278
dc9503b21c87b5a469118746f99e4f41d73888972ce735fa10a80f6d218086da0e3da525d9a4cd8e4ea497ec199fef720e4a525873d77a1af304ac505e076462

and all are public.
(Assignee)

Comment 7

2 years ago
Created attachment 8593502 [details] [diff] [review]
bug1155238-direct-links.patch

This takes the cheap way out and downloads these files directly from S3 on every run, without any caching.  That's slow and expensive, though.  Jonathan, can you help me figure out how to test the scripts that use these variables, so I can make the necessary changes?

mozharness/mozilla/gaia.py:    def extract_xre(self, xre_url, xre_path=None, parent_dir=None):
mozharness/mozilla/gaia.py:        filename = self.download_file(xre_url, parent_dir=parent_dir)
mozharness/mozilla/gaia.py:                  xre_url=None, build_config_path=None):
mozharness/mozilla/gaia.py:        if xre_url:
mozharness/mozilla/gaia.py:            self.extract_xre(xre_url, xre_path=xre_dir, parent_dir=gaia_dir)
mozharness/mozilla/testing/gaia_test.py:         "dest": "xre_url",
scripts/android_emulator_unittest.py:         "dest": "xre_url",
scripts/b2g_emulator_unittest.py:         "dest": "xre_url",
scripts/b2g_emulator_unittest.py:        self._download_unzip(self.config['xre_url'],
scripts/gaia_unit.py:                       xre_url=self.config.get('xre_url'),
scripts/luciddream_unittest.py:         "dest": "xre_url",
scripts/luciddream_unittest.py:                       xre_url=self.config.get('xre_url'),
scripts/marionette.py:         "dest": "xre_url",
scripts/marionette.py:                           xre_url=self.config.get('xre_url'),

scripts/b2g_emulator_unittest.py:         "dest": "busybox_url",
scripts/b2g_emulator_unittest.py:        if self.config.get('busybox_url'):
scripts/b2g_emulator_unittest.py:            self.download_file(self.config['busybox_url'],
Attachment #8593502 - Flags: review?(jgriffin)
(Assignee)

Updated

2 years ago
Duplicate of this bug: 1151527
Comment on attachment 8593445 [details] [diff] [review]
bug1155238.patch

-#! <%= @python %>
+#!/usr/bin/env python

I thought the <%= @python %> thing was needed to get the python path correct on all platforms, but it's been a while since I've looked. Has that changed? If you're sure it's fine, I can switch to r+.
Attachment #8593445 - Flags: review?(mshal) → feedback+
(Assignee)

Comment 10

2 years ago
Oh, thanks for spotting that -- the hashbang line should remain unchanged.
(Assignee)

Comment 11

2 years ago
Created attachment 8593511 [details] [diff] [review]
bug1155238-p2.patch
Attachment #8593445 - Attachment is obsolete: true
Attachment #8593511 - Flags: review?(mshal)
Comment on attachment 8593480 [details] [diff] [review]
bug1155238-mozharness.patch

If there's only a single tooltool URL now, I think they can probably be removed from configs/* and just use the defaults in scripts/*, but no need to redo the patch.

If you don't feel like setting up a mozharness user repo to test on try, feel free to ping me & I can help test it.
Attachment #8593480 - Flags: review?(mshal) → review+

Updated

2 years ago
Attachment #8593511 - Flags: review?(mshal) → review+
Comment on attachment 8593486 [details] [diff] [review]
bug1155238-mozilla-central.patch

Review of attachment 8593486 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/devtools/automation/winbuildenv.sh
@@ +33,5 @@
>  # newer mozilla-build or tooltool.
>  if ! which mozmake 2>/dev/null; then
>      export PATH="$PATH:$SOURCE/.."
>      if ! which mozmake 2>/dev/null; then
> +	TT_SERVER=${TT_SERVER:-https://api.pub.build.mozilla.org/tooltool/}

Only concern is the trailing slash. Previous version didn't have it.
Attachment #8593486 - Flags: review?(coop) → review+
(Assignee)

Comment 14

2 years ago
The trailing slash is necessary now.

I'll try the defaults-in-scripts idea.  There's always been only a single tooltool URL, so I'm not sure why that wasn't done to begin with!
(Assignee)

Comment 15

2 years ago
Comment on attachment 8593511 [details] [diff] [review]
bug1155238-p2.patch

remote:   https://hg.mozilla.org/build/puppet/rev/0cdc5066f976
remote:   https://hg.mozilla.org/build/puppet/rev/34c98b742ec0

Note that this still has to be deployed on Windows before I can land any of the other patches.
Attachment #8593511 - Flags: checked-in+
(Assignee)

Comment 16

2 years ago
Created attachment 8593555 [details] [diff] [review]
bug1155238-mozharness-r2.patch

This removes all tooltool_servers config, but leaves the old-old-old tooltool_url config in place.  This is a little more than a search-and-replace, but hopefully testing it on a few builders will be enough to indicate it's correct.
Attachment #8593555 - Flags: review?(mshal)
Comment on attachment 8593555 [details] [diff] [review]
bug1155238-mozharness-r2.patch

>-    "busybox_url": "http://tooltool.pvt.build.mozilla.org/build/sha512/0748e900821820f1a42e2f1f3fa4d9002ef257c351b9e6b78e7de0ddd0202eace351f440372fbb1ae0b7e69e8361b036f6bd3362df99e67fc585082a311fc0df",
>-    "xre_url": "http://tooltool.pvt.build.mozilla.org/build/sha512/dc9503b21c87b5a469118746f99e4f41d73888972ce735fa10a80f6d218086da0e3da525d9a4cd8e4ea497ec199fef720e4a525873d77a1af304ac505e076462",
>-    "tooltool_servers": ["https://api.pub.build.mozilla.org/tooltool/"],
>+    "busybox_url": "https://api.pub.build.mozilla.org/tooltool/sha512/0748e900821820f1a42e2f1f3fa4d9002ef257c351b9e6b78e7de0ddd0202eace351f440372fbb1ae0b7e69e8361b036f6bd3362df99e67fc585082a311fc0df",
>+    "xre_url": "https://api.pub.build.mozilla.org/tooltool/sha512/dc9503b21c87b5a469118746f99e4f41d73888972ce735fa10a80f6d218086da0e3da525d9a4cd8e4ea497ec199fef720e4a525873d77a1af304ac505e076462",

It looks like you combined this with the direct-links patch? Not sure if that was intended.

In any case, this looks good to me, though we definitely need to test a bunch of builds.
Attachment #8593555 - Flags: review?(mshal) → review+
(Assignee)

Updated

2 years ago
No longer depends on: 1155256
(Assignee)

Updated

2 years ago
Depends on: 1155257
(Assignee)

Comment 18

2 years ago
Created attachment 8593999 [details] [diff] [review]
bug1155238-puppet-token.patch
Attachment #8593999 - Flags: review?(mcornmesser)
Attachment #8593999 - Flags: review?(mcornmesser) → review+
(Assignee)

Comment 19

2 years ago
Created attachment 8594017 [details] [diff] [review]
bug1155238-mozharness-r3.patch

Corrected patch, carrying forward review from attachment 8593555 [details] [diff] [review].
Attachment #8593555 - Attachment is obsolete: true
Attachment #8594017 - Flags: review+
(Assignee)

Comment 20

2 years ago
Created attachment 8594039 [details] [diff] [review]
bug1155238-auth-file.patch
Attachment #8594039 - Flags: review?(jlund)
Comment on attachment 8594039 [details] [diff] [review]
bug1155238-auth-file.patch

Review of attachment 8594039 [details] [diff] [review]:
-----------------------------------------------------------------

::: mozharness/mozilla/tooltool.py
@@ +21,5 @@
>  ]
>  
> +# set the default authentication file based on platform; this corresponds to where
> +# puppet puts the token
> +if sys.platform == 'win32':

can we define this within the TooltoolMixin class so we can avail of things like _is_windows():
http://mxr.mozilla.org/build/source/mozharness/mozharness/base/script.py#122
(Assignee)

Comment 22

2 years ago
Created attachment 8594770 [details] [diff] [review]
bug1155238-auth-file-r2.patch
Attachment #8594770 - Flags: review?(jlund)
(Assignee)

Updated

2 years ago
Attachment #8594039 - Attachment is obsolete: true
Attachment #8594039 - Flags: review?(jlund)
(Assignee)

Comment 23

2 years ago
Comment on attachment 8593999 [details] [diff] [review]
bug1155238-puppet-token.patch

remote:   https://hg.mozilla.org/build/puppet/rev/d613d86bc91d
remote:   https://hg.mozilla.org/build/puppet/rev/6d5f6e258698
Attachment #8593999 - Flags: checked-in+
(Assignee)

Updated

2 years ago
Attachment #8593480 - Attachment is obsolete: true
Attachment #8593502 - Flags: review?(jgriffin) → review+
(Assignee)

Comment 24

2 years ago
All MH patches are being tested in https://treeherder.mozilla.org/#/jobs?repo=try&revision=dab022900494

If that goes well, I'll merge the m-c patch (attachment 8593486 [details] [diff] [review]) and do a full try run.
(In reply to Dustin J. Mitchell [:dustin] from comment #7)
> Created attachment 8593502 [details] [diff] [review]
> bug1155238-direct-links.patch
> 
> This takes the cheap way out and downloads these files directly from S3 on
> every run, without any caching.  That's slow and expensive, though. 
> Jonathan, can you help me figure out how to test the scripts that use these
> variables, so I can make the necessary changes?
> 

The easiest way would be to land these changes on mozharness default and then update the mozharness pointer on cedar, and see what happens.
Or, try works too.  :)
(Assignee)

Comment 27

2 years ago
I added attachment 8593486 [details] [diff] [review] and pushed a full run to try:
  https://treeherder.mozilla.org/#/jobs?repo=try&revision=c5a3d30c4152
(Assignee)

Comment 28

2 years ago
Created attachment 8596182 [details] [diff] [review]
bug1155238-puppetagain-all.patch

My try run failed a bunch of test jobs because the token was only installed on builders.  Oops :)
Attachment #8596182 - Flags: review?(mcornmesser)
Attachment #8596182 - Flags: review?(mcornmesser) → review+
(Assignee)

Updated

2 years ago
Attachment #8596182 - Flags: checked-in+
(Assignee)

Comment 29

2 years ago
Just waiting on deployment on Windows before I can test this again.
Comment on attachment 8594770 [details] [diff] [review]
bug1155238-auth-file-r2.patch

Review of attachment 8594770 [details] [diff] [review]:
-----------------------------------------------------------------

I've been at a work-week the last few days and thus slow to clear bug review queue. Apologies for the delay!
Attachment #8594770 - Flags: review?(jlund) → review+
(Assignee)

Comment 31

2 years ago
I'm re-triggering a fwe jobs on my last try push to see if things are working now.  If so, I'll push a fresh commit to try, since this one may be a bit rotten.
(Assignee)

Comment 32

2 years ago
Created attachment 8598841 [details] [diff] [review]
bug1155238-config.patch
Attachment #8598841 - Flags: review?(mcornmesser)
(Assignee)

Comment 33

2 years ago
Comment on attachment 8598841 [details] [diff] [review]
bug1155238-config.patch

wrong patch
Attachment #8598841 - Attachment is obsolete: true
Attachment #8598841 - Flags: review?(mcornmesser)
(Assignee)

Comment 34

2 years ago
Created attachment 8598851 [details] [diff] [review]
bug1155238-puppetagain-config.patch
Attachment #8598851 - Flags: review?(mcornmesser)
Attachment #8598851 - Flags: review?(mcornmesser) → review+
(Assignee)

Comment 35

2 years ago
Comment on attachment 8598851 [details] [diff] [review]
bug1155238-puppetagain-config.patch

remote:   https://hg.mozilla.org/build/puppet/rev/212a8079cac5
remote:   https://hg.mozilla.org/build/puppet/rev/d6e2251b93c1

and again we wait for AMIs
Attachment #8598851 - Flags: checked-in+
(Assignee)

Comment 36

2 years ago
New, rebased try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=7329bf847055
(Assignee)

Comment 37

2 years ago
That's looking green, and I'm seeing tooltool hits at
  https://papertrailapp.com/groups/1141234/events?q=api+%2Ftooltool%2F
so I'm going to push a full try job.
(Assignee)

Comment 38

2 years ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=3315413ccd0a
(Assignee)

Comment 39

2 years ago
Hm, tooltool_wrapper.sh appears to not be passing any authentication info:

11:09:51     INFO - ['sh', '/builds/slave/try-lx-00000000000000000000000/build/tools/scripts/tooltool/tooltool_wrapper.sh', '/builds/slave/try-lx-00000000000000000000000/build/src/browser/config/tooltool-manifests/linux32/releng.manifest', 'https://api.pub.build.mozilla.org/tooltool/', 'setup.sh', '/builds/tooltool.py']
11:09:51     INFO - Running command: ['sh', '/builds/slave/try-lx-00000000000000000000000/build/tools/scripts/tooltool/tooltool_wrapper.sh', '/builds/slave/try-lx-00000000000000000000000/build/src/browser/config/tooltool-manifests/linux32/releng.manifest', 'https://api.pub.build.mozilla.org/tooltool/', 'setup.sh', '/builds/tooltool.py'] in /builds/slave/try-lx-00000000000000000000000/build/src
11:09:51     INFO - Copy/paste: sh /builds/slave/try-lx-00000000000000000000000/build/tools/scripts/tooltool/tooltool_wrapper.sh /builds/slave/try-lx-00000000000000000000000/build/src/browser/config/tooltool-manifests/linux32/releng.manifest https://api.pub.build.mozilla.org/tooltool/ setup.sh /builds/tooltool.py
11:09:51     INFO -  /builds/slave/try-lx-00000000000000000000000/build/src/browser/config/tooltool-manifests/linux32/releng.manifest content
11:09:51     INFO -  =======================================================
11:09:51     INFO -  [
11:09:51     INFO -  {
11:09:51     INFO -  "size": 82,
11:09:51     INFO -  "digest": "70a6126249e40aa1da32248bf6bfe45e0d8c87334579ec0cf69403e61b635e27c766d9bf08d530978286552f158ee24c74b0168a57cc6b734dcfed4fc5e09cff",
11:09:51     INFO -  "algorithm": "sha512",
11:09:51     INFO -  "filename": "setup.sh"
11:09:51     INFO -  },
11:09:51     INFO -  {
11:09:51     INFO -  "size": 80458572,
11:09:51     INFO -  "digest": "e5101f9dee1e462f6cbd3897ea57eede41d23981825c7b20d91d23ab461875d54d3dfc24999aa58a31e8b01f49fb3140e05ffe5af2957ef1d1afb89fd0dfe1ad",
11:09:51     INFO -  "algorithm": "sha512",
11:09:51     INFO -  "filename": "gcc.tar.xz"
11:09:51     INFO -  },
11:09:51     INFO -  {
11:09:51     INFO -  "size": 167175,
11:09:51     INFO -  "digest": "0b71a936edf5bd70cf274aaa5d7abc8f77fe8e7b5593a208f805cc9436fac646b9c4f0b43c2b10de63ff3da671497d35536077ecbc72dba7f8159a38b580f831",
11:09:51     INFO -  "algorithm": "sha512",
11:09:51     INFO -  "filename": "sccache.tar.bz2"
11:09:51     INFO -  }
11:09:51     INFO -  ]
11:09:51     INFO -  =======================================================
11:09:51     INFO -  Fetching...
11:09:51     INFO -  retry: Calling run_with_timeout with args: (['/builds/tooltool.py', '--url', 'https://api.pub.build.mozilla.org/tooltool/', '--overwrite', '-m', '/builds/slave/try-lx-00000000000000000000000/build/src/browser/config/tooltool-manifests/linux32/releng.manifest', 'fetch', '-c', '/builds/tooltool_cache'], 300, None, None, False, True), kwargs: {}, attempt #1
11:09:51     INFO -  Executing: ['/builds/tooltool.py', '--url', 'https://api.pub.build.mozilla.org/tooltool/', '--overwrite', '-m', '/builds/slave/try-lx-00000000000000000000000/build/src/browser/config/tooltool-manifests/linux32/releng.manifest', 'fetch', '-c', '/builds/tooltool_cache']
11:09:51     INFO -  INFO - File setup.sh not present in local cache folder /builds/tooltool_cache

which is working for most builds because they're all public files.
(Assignee)

Comment 40

2 years ago
Created attachment 8599464 [details]
MozReview Request: bz://1155238/dustin

/r/7891 - Bug 1155238: replace direct links to old filenames with direct links to new filenames
/r/7893 - Bug 1155238: use the RelengAPI tooltool URL everywhere
/r/7895 - Bug 1155238: pass --authentication-file to tooltool
/r/7897 - Bug 1155238: pass --authentication-file to the old tooltool_wrapper.sh, too

Pull down these commits:

hg pull -r fbe718fe444062cc439d8ccd8736cbb77bdd6be3 https://reviewboard-hg.mozilla.org/build-mozharness
Attachment #8599464 - Flags: review?(mshal)
(Assignee)

Comment 41

2 years ago
Comment on attachment 8599464 [details]
MozReview Request: bz://1155238/dustin

I'm a little new to mozreview, but three of the four csets here have already been reviewed via bugzilla, so I'm only looking for r+ for the last one.
(Assignee)

Comment 42

2 years ago
https://reviewboard.mozilla.org/r/7891/#review6647

Carried forward from :jgriffin on bugzilla
(Assignee)

Comment 43

2 years ago
https://reviewboard.mozilla.org/r/7893/#review6649

r+ carried forward from :mshal on bugzilla
(Assignee)

Comment 44

2 years ago
https://reviewboard.mozilla.org/r/7895/#review6651

r+ carried forward from :jlund on bugzilla
(Assignee)

Comment 45

2 years ago
Looks like there are some XP and W7 hosts that don't have c:\builds\relengapi.tok installed -- getting some reds like https://treeherder.mozilla.org/logviewer.html#?job_id=7068715&repo=try

Mark will have a look-see.
(Assignee)

Comment 46

2 years ago
OK, that looks to have been per-machine issues.  I made one more try push with attachment 8599464 [details] included (using --authentication-file with tooltool_wrapper.sh), both to test that commit and to get a little more confidence that this isn't going to burn trees tomorrow morning.  Speaking of which, I'd like to land this tomorrow morning, mshal :)

https://treeherder.mozilla.org/#/jobs?repo=try&revision=c7b277b5d8b9
With these changes, we have lost the ability of a developer to run any mozharness jobs which requires tooltool.
This is important to allow people run mozharness without having to request a loaner.

Bug 1088208 contains the information about this feature.
(Assignee)

Comment 48

2 years ago
Only with the latest tooltool.py.  I didn't realize things were automatically pulling the latest.  Using an older tooltool works fine, or applying the mozharness patches here (which I'm working on pushing to production today).

Comment 49

2 years ago
https://hg.mozilla.org/integration/fx-team/rev/c982ce92229b
(Assignee)

Updated

2 years ago
Attachment #8593502 - Flags: checked-in+
(Assignee)

Updated

2 years ago
Attachment #8594017 - Flags: checked-in+
(Assignee)

Updated

2 years ago
Attachment #8594770 - Flags: checked-in+
(Assignee)

Updated

2 years ago
Attachment #8599464 - Flags: checked-in+
(Assignee)

Comment 50

2 years ago
https://hg.mozilla.org/build/mozharness/rev/06a2bc3e6b42

Like a dummy I forgot the r='s in the component patches.

Armen's asked me to make a new try push with `mozharness.json` pointing to the latest default.  I'll get that started shortly (I don't have a local clone of m-c so it will be a while).  Once that's done, I think we can bump the revision in m-c.
(Assignee)

Comment 51

2 years ago
Armen kindly pushed to try for me:
  https://treeherder.mozilla.org/#/jobs?repo=try&revision=67428d309732
https://reviewboard.mozilla.org/r/7891/#review6695

Ship It!

Updated

2 years ago
Attachment #8599464 - Flags: review?(mshal) → review+

Comment 53

2 years ago
https://hg.mozilla.org/integration/fx-team/rev/90cf2e995537
(Assignee)

Updated

2 years ago
Blocks: 1133842
(Assignee)

Comment 54

2 years ago
As the pulsebot posts are indicating, this has been completely landed, merged, bumped, and otherwise *'ed; I'll be watching for use of the old vhosts over the next few weeks, and eventually shut them off.
(Assignee)

Updated

2 years ago
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
mozharness production tag moved to: https://hg.mozilla.org/build/mozharness/rev/ca580fcfa627
mozharness production tag moved to: https://hg.mozilla.org/build/mozharness/rev/ca580fcfa627
mozharness production tag moved to: https://hg.mozilla.org/build/mozharness/rev/ca580fcfa627
mozharness production tag moved to: https://hg.mozilla.org/build/mozharness/rev/ca580fcfa627
mozharness production tag moved to: https://hg.mozilla.org/build/mozharness/rev/ca580fcfa627
https://hg.mozilla.org/mozilla-central/rev/c982ce92229b
https://hg.mozilla.org/mozilla-central/rev/90cf2e995537
Depends on: 1160845
mozharness production tag moved to: https://hg.mozilla.org/build/mozharness/rev/4d260376921f
mozharness production tag moved to: https://hg.mozilla.org/build/mozharness/rev/4d260376921f
mozharness production tag moved to: https://hg.mozilla.org/build/mozharness/rev/4d260376921f
mozharness production tag moved to: https://hg.mozilla.org/build/mozharness/rev/4d260376921f
mozharness production tag moved to: https://hg.mozilla.org/build/mozharness/rev/4d260376921f
Depends on: 1171063
(Assignee)

Comment 66

2 years ago
Comment on attachment 8599464 [details]
MozReview Request: bz://1155238/dustin
Attachment #8599464 - Attachment is obsolete: true
Attachment #8620062 - Flags: review+
Attachment #8620063 - Flags: review+
Attachment #8620064 - Flags: review+
Attachment #8620065 - Flags: review+
(Assignee)

Comment 67

2 years ago
Created attachment 8620062 [details]
MozReview Request: Bug 1155238: pass --authentication-file to the old tooltool_wrapper.sh, too
(Assignee)

Comment 68

2 years ago
Created attachment 8620063 [details]
MozReview Request: Bug 1155238: replace direct links to old filenames with direct links to new filenames
(Assignee)

Comment 69

2 years ago
Created attachment 8620064 [details]
MozReview Request: Bug 1155238: use the RelengAPI tooltool URL everywhere
(Assignee)

Comment 70

2 years ago
Created attachment 8620065 [details]
MozReview Request: Bug 1155238: pass --authentication-file to tooltool
You need to log in before you can comment on or make changes to this bug.