Remove remains of tc-vcs

RESOLVED FIXED in mozilla58

Status

Taskcluster
Task Configuration
RESOLVED FIXED
a month ago
a month ago

People

(Reporter: glandium, Assigned: glandium)

Tracking

unspecified
mozilla58

Details

MozReview Requests

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

Attachments

(6 attachments)

(Assignee)

Description

a month ago
AFAICT, tc-vcs is not used anymore. Time to get rid of the remains we have in the tree.
(Assignee)

Updated

a month ago
Depends on: 1383973
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 7

a month ago
mozreview-review
Comment on attachment 8919145 [details]
Bug 1409260 - Remove base-repository exception in .taskcluster.yml.

https://reviewboard.mozilla.org/r/190058/#review195382
Attachment #8919145 - Flags: review+

Comment 8

a month ago
mozreview-review
Comment on attachment 8919146 [details]
Bug 1409260 - Remove unused scripts from the desktop-build images.

https://reviewboard.mozilla.org/r/190060/#review195384
Attachment #8919146 - Flags: review+

Comment 9

a month ago
mozreview-review
Comment on attachment 8919147 [details]
Bug 1409260 - Remove mozharness tc-vcs support.

https://reviewboard.mozilla.org/r/190062/#review195386
Attachment #8919147 - Flags: review+

Comment 10

a month ago
mozreview-review
Comment on attachment 8919148 [details]
Bug 1409260 - Remove tc-vcs caches.

https://reviewboard.mozilla.org/r/190064/#review195388
Attachment #8919148 - Flags: review+

Comment 11

a month ago
mozreview-review
Comment on attachment 8919149 [details]
Bug 1409260 - Remove tc-vcs related files from docker images.

https://reviewboard.mozilla.org/r/190066/#review195390

::: taskcluster/docker/desktop-build/Dockerfile
(Diff revision 1)
>  MAINTAINER    Dustin J. Mitchell <dustin@mozilla.com>
>  
>  VOLUME /builds/worker/workspace
>  VOLUME /builds/worker/tooltool-cache
>  VOLUME /builds/worker/checkouts
> -VOLUME /builds/worker/.tc-vcs

Strictly speaking, the previous commit may have broken taskgraph until this commit because there is code that parses for VOLUME and ensures cache definitions are in sync. But as long as this all lands as a single push, we should be fine.
Attachment #8919149 - Flags: review+

Comment 12

a month ago
mozreview-review
Comment on attachment 8919150 [details]
Bug 1409260 - Remove comments about tc-vcs.

https://reviewboard.mozilla.org/r/190068/#review195392

Great cleanup!

FWIW my recollection was also that the android tasks were the last components using tc-vcs. Refactoring those was a bit difficult and I never had the will to do it. I'm glad nalexander tackled it as part of overhauling the android automation!

Also, with tc-vcs gone, I'm pretty sure we can remove node.js from some Docker images. Although with all the recent refactorings, I can't recall exactly where node.js is installed and what other functionality may be using it. IIRC there's a npm package related to tc-vcs that could almost certainly be removed without removing node.
Attachment #8919150 - Flags: review+
(Assignee)

Comment 13

a month ago
> IIRC there's a npm package related to tc-vcs that could almost certainly be removed without removing node.

There's one removed in taskcluster/docker/desktop1604-test/Dockerfile in one of the patches in this bug ;)
(Assignee)

Comment 14

a month ago
Also, it seems node is used for http/2 tests.

Comment 15

a month ago
mozreview-review
Comment on attachment 8919145 [details]
Bug 1409260 - Remove base-repository exception in .taskcluster.yml.

https://reviewboard.mozilla.org/r/190058/#review195504
Attachment #8919145 - Flags: review?(dustin) → review+

Comment 16

a month ago
mozreview-review
Comment on attachment 8919146 [details]
Bug 1409260 - Remove unused scripts from the desktop-build images.

https://reviewboard.mozilla.org/r/190060/#review195506
Attachment #8919146 - Flags: review?(dustin) → review+

Comment 17

a month ago
mozreview-review
Comment on attachment 8919147 [details]
Bug 1409260 - Remove mozharness tc-vcs support.

https://reviewboard.mozilla.org/r/190062/#review195508
Attachment #8919147 - Flags: review+

Comment 18

a month ago
mozreview-review
Comment on attachment 8919148 [details]
Bug 1409260 - Remove tc-vcs caches.

https://reviewboard.mozilla.org/r/190064/#review195514

I'm grinning like a fool
Attachment #8919148 - Flags: review?(dustin) → review+

Comment 19

a month ago
mozreview-review
Comment on attachment 8919149 [details]
Bug 1409260 - Remove tc-vcs related files from docker images.

https://reviewboard.mozilla.org/r/190066/#review195516

::: taskcluster/docker/desktop1604-test/Dockerfile:76
(Diff revision 1)
>  RUN usermod -a -G video worker
>  
>  RUN mkdir Documents; mkdir Pictures; mkdir Music; mkdir Videos; mkdir artifacts
>  
> -# install tc-vcs and tc-npm-cache
> -RUN npm install -g taskcluster-vcs@2.3.12 \
> +# install tc-npm-cache
> +RUN  npm install -g taskcluster-npm-cache@1.1.14 \

I have no idea what this is, so best to leave it in. ..
Attachment #8919149 - Flags: review?(dustin) → review+

Comment 20

a month ago
mozreview-review
Comment on attachment 8919150 [details]
Bug 1409260 - Remove comments about tc-vcs.

https://reviewboard.mozilla.org/r/190068/#review195518
Attachment #8919150 - Flags: review?(dustin) → review+

Comment 21

a month ago
mozreview-review-reply
Comment on attachment 8919149 [details]
Bug 1409260 - Remove tc-vcs related files from docker images.

https://reviewboard.mozilla.org/r/190066/#review195516

> I have no idea what this is, so best to leave it in. ..

Hrm, I vaguely recall that this was a b2g thing.

Comment 22

a month ago
mozreview-review-reply
Comment on attachment 8919149 [details]
Bug 1409260 - Remove tc-vcs related files from docker images.

https://reviewboard.mozilla.org/r/190066/#review195516

> Hrm, I vaguely recall that this was a b2g thing.

taskcluster-vcs provided tc-vcs IIRC.

While I'm here, there's an outside chance that someone is relying on tc-vcs to perform Git-based automation on GitHub. I recall someone having such one-off automation ~1 year ago. Can't remember who. If that's still true, I suppose removing tc-vcs will shine a spotlight on them and enlighten all of us of their requirements.

Comment 23

a month ago
mozreview-review-reply
Comment on attachment 8919149 [details]
Bug 1409260 - Remove tc-vcs related files from docker images.

https://reviewboard.mozilla.org/r/190066/#review195516

> taskcluster-vcs provided tc-vcs IIRC.
> 
> While I'm here, there's an outside chance that someone is relying on tc-vcs to perform Git-based automation on GitHub. I recall someone having such one-off automation ~1 year ago. Can't remember who. If that's still true, I suppose removing tc-vcs will shine a spotlight on them and enlighten all of us of their requirements.

Ah, I was referring to taskcluster-npm-cache.

I also recall some git automation using it, but we should probably be able to check some of our org repos for the use of it.  We can also audit the index service to see if it's being hit for those paths.

Comment 24

a month ago
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s 4841347dc20c -d 3915770a8869: rebasing 427140:4841347dc20c "Bug 1409260 - Remove base-repository exception in .taskcluster.yml. r=dustin,gps"
merging .taskcluster.yml
warning: conflicts while merging .taskcluster.yml! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
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 31

a month ago
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/autoland/rev/e325fb428095
Remove base-repository exception in .taskcluster.yml. r=dustin,gps
https://hg.mozilla.org/integration/autoland/rev/82781cec272f
Remove unused scripts from the desktop-build images. r=dustin,gps
https://hg.mozilla.org/integration/autoland/rev/b445d6f3372f
Remove mozharness tc-vcs support. r=dustin,gps
https://hg.mozilla.org/integration/autoland/rev/915021e13b67
Remove tc-vcs caches. r=dustin,gps
https://hg.mozilla.org/integration/autoland/rev/afb002c9c180
Remove tc-vcs related files from docker images. r=dustin,gps
https://hg.mozilla.org/integration/autoland/rev/4375412d40e3
Remove comments about tc-vcs. r=dustin,gps
https://hg.mozilla.org/mozilla-central/rev/e325fb428095
https://hg.mozilla.org/mozilla-central/rev/82781cec272f
https://hg.mozilla.org/mozilla-central/rev/b445d6f3372f
https://hg.mozilla.org/mozilla-central/rev/915021e13b67
https://hg.mozilla.org/mozilla-central/rev/afb002c9c180
https://hg.mozilla.org/mozilla-central/rev/4375412d40e3
Status: NEW → RESOLVED
Last Resolved: a month ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58

Comment 33

a month ago
mozreview-review
Comment on attachment 8919147 [details]
Bug 1409260 - Remove mozharness tc-vcs support.

https://reviewboard.mozilla.org/r/190062/#review196078

lgtm
Attachment #8919147 - Flags: review?(jlund) → review+
You need to log in before you can comment on or make changes to this bug.