Update intree configs to no longer reference git.m.o

RESOLVED FIXED

Status

Release Engineering
General
RESOLVED FIXED
2 years ago
14 days ago

People

(Reporter: hwine, Assigned: jgriffin)

Tracking

unspecified
Dependency tree / graph

Firefox Tracking Flags

(firefox50 fixed)

Details

MozReview Requests

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

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
Moving into a separate bug from bug 1277838 comment #3, where :Callek noted:
> 
> We do have a few things specifying git.m.o in tree though for tooltool:
> 
> https://dxr.mozilla.org/mozilla-central/search?q=git.mozilla.org%2Fbuild%2Ftooltool&redirect=true
> 
> Such as:
> https://dxr.mozilla.org/mozilla-central/source/testing/taskcluster/scripts/builder/install-packages.sh#10

And there may be non-tooltool reference as well.
(Assignee)

Comment 1

2 years ago
Created attachment 8765626 [details]
Bug 1277957 - Point to tooltool at GitHub,

Review commit: https://reviewboard.mozilla.org/r/60906/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/60906/
Attachment #8765626 - Flags: review?(dustin)

Comment 3

2 years ago
Pushed by jgriffin@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/912ea9267130
Point to tooltool at GitHub, r=dustin
(Assignee)

Updated

2 years ago
Assignee: nobody → jgriffin
(Assignee)

Comment 5

2 years ago
Not sure what happened here, maybe Dustin can tell.
Flags: needinfo?(jgriffin) → needinfo?(dustin)
Those tasks are using tc-vcs to check out that repository, and that requires a primed cache to be in place first.  I think greg's the resident expert in creating those caches.
Flags: needinfo?(dustin) → needinfo?(garndt)

Comment 7

2 years ago
New release has been published (and hooks updated) to include this repo:
https://github.com/taskcluster/taskcluster-vcs/releases/tag/v2.3.37

Manually created a task to cache it in the meantime:
https://tools.taskcluster.net/task-inspector/#e0vSm-UcSG2wMZyyiCLgyw/0
Flags: needinfo?(garndt)
Thanks greg!  With that done, I think this patch can re-land (although it's worth a try push)
(Assignee)

Comment 9

2 years ago
(In reply to Dustin J. Mitchell [:dustin] from comment #8)
> Thanks greg!  With that done, I think this patch can re-land (although it's
> worth a try push)

TaskCluster doesn't pick up the failed jobs in a try push (see https://treeherder.mozilla.org/#/jobs?repo=try&revision=ef57a93e79b0), so I'll just have to try landing again.

Comment 12

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/13fa96518284
https://hg.mozilla.org/mozilla-central/rev/310e08727914
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox50: --- → fixed
Resolution: --- → FIXED

Updated

2 years ago
Blocks: 1296150
Component: General Automation → General
Product: Release Engineering → Release Engineering
You need to log in before you can comment on or make changes to this bug.