59 bytes, text/x-review-board-request
One thing that the first change in bug 1430005 revealed is that the rewrite_when_to_optimization function adds the taskcluster/docker/$docker-image files to "skip-unless-changed"... but it only does so if the job has a docker image explicitly set, which a lot of jobs don't have. So it would seem this should be added much later, after the last possible place where ['worker']['docker-image'] can be set. OTOH, it's also wrong, because taskcluster/docker/$docker-image is over-simplifying the files that affect docker images, and aiui, tasks are not optimized anyways if their dependencies (which docker images are) are not optimized. So it would seem this is not necessary at all.
Comment on attachment 8942688 [details] Bug 1430491: stop depending, incorrectly, on docker images; https://reviewboard.mozilla.org/r/212946/#review218674
Attachment #8942688 - Flags: review?(mh+mozilla) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/cc223a707701 stop depending, incorrectly, on docker images; r=glandium
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
You need to log in before you can comment on or make changes to this bug.