1275613 - Jobs with a "file_patterns" clause should automatically be scheduled if their task definition changes

Status: RESOLVED FIXED

(Taskcluster Graveyard :: Scheduler, defect)

Description

[Andrew Halberstadt [:ahal]]

Currently you can tell a job to run only when certain file patterns have been modified. But unless the task definition of this job is explicitly listed, the job won't run when making changes to it!

I think it's safe to assume that we want to run a job every time it's definition changes, so we should just do this automatically.

We should also automatically run the job when a parent task.yml is modified, and potentially even when its docker image is modified. But I'm going to punt on these two for now as I don't see any easy way to implement it.

Comment 1

[Andrew Halberstadt [:ahal]]

Attachment: MozReview Request: Bug 1275613 - Always schedule a 'file_patterns' job when its task definition has changed, r?dustin

Currently, jobs which has 'file_patterns' set will only be run if a file matching one
of those patterns has been modified. This means that unless the task definition of the
job was explicitly included in those 'file_patterns', the job won't run when we modify
it!

I think it is safe to assume that when making changes to a job's task, we want it to be
run no matter what. So let's add the task definition to 'file_patterns' automatically.

Here's a try run which modifies the marionette harness task:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=01e6cf9d8599c64d61ac8ecb5320c1c4209e0a8b

Review commit: https://reviewboard.mozilla.org/r/55164/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/55164/

Comment 2

[Dustin J. Mitchell [:dustin] (he/him)]

Comment on attachment 8756414 [details]
MozReview Request: Bug 1275613 - Always schedule a 'file_patterns' job when its task definition has changed, r?dustin

https://reviewboard.mozilla.org/r/55164/#review51818

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/9bf50e4ab8ee

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/9bf50e4ab8ee