Closed Bug 1234076 Opened 6 years ago Closed 6 years ago
Upgrade builder image to use tc-vcs 2
58 bytes, text/x-review-board-request
This version gives us artifact checking and some other nice things to prevent full cloning when things go wrong.
Review commit: https://reviewboard.mozilla.org/r/28977/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/28977/
This patch not only upgrades tc-vcs, but also allows those builds to use the new in-tree docker image building.
Comment on attachment 8701495 [details] MozReview Request: Bug 1234076 - upgrade tc-vcs to 2.3.17 r=wcosta https://reviewboard.mozilla.org/r/28977/#review25761 ::: testing/docker/builder/Dockerfile:26 (Diff revision 1) > Isn't necessary to update the VERSION file anymore? How inherited images will know the image has been upgrade, directory diff?
Comment on attachment 8701495 [details] MozReview Request: Bug 1234076 - upgrade tc-vcs to 2.3.17 r=wcosta Review request updated; see interdiff: https://reviewboard.mozilla.org/r/28977/diff/1-2/
This latest change will remove the concept of a docker registry image complete for 'builder'. All tasks using the 'builder' image will not use the in-tree docker image support. VERSION file isn't necessary as the contents of the directory are hashed and compared to determine if the image should be rebuilt.
Comment on attachment 8701495 [details] MozReview Request: Bug 1234076 - upgrade tc-vcs to 2.3.17 r=wcosta https://reviewboard.mozilla.org/r/28977/#review25963
Attachment #8701495 - Flags: review?(wcosta) → review+
https://hg.mozilla.org/integration/b2g-inbound/rev/4ad0371b1a072e4153972e4cc76dc773b1e001b9 Bug 1234076 - upgrade tc-vcs to 2.3.17 r=wcosta
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.