Closed Bug 1407279 Opened 7 years ago Closed 7 years ago

Docker image changes on mozilla-beta result in failed decision tasks

Categories

(Firefox Build System :: Task Configuration, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rail, Assigned: rail)

References

Details

Attachments

(1 file)

I tried to uplift a patch to mozilla-beta, which also touches one of the docker images under taskcluster/docker. The corresponding decision taks failed https://tools.taskcluster.net/groups/XugxWvNaRFOf3SR1hJHlsg/tasks/XugxWvNaRFOf3SR1hJHlsg/runs/0/logs/public%2Flogs%2Flive.log:

[task 2017-10-10T15:00:08.268737Z] "GET /index/v1/task/docker.images.v2.level-3.funsize-update-generator.hash.600cc7c926a7c83498b1dd523c8df4a848417220bf6ee0a61a9c14786ca6dec0 HTTP/1.1" 404 456
[task 2017-10-10T15:00:08.269916Z] Traceback (most recent call last):
[task 2017-10-10T15:00:08.269986Z]   File "/builds/worker/checkouts/gecko/taskcluster/mach_commands.py", line 154, in taskgraph_decision
[task 2017-10-10T15:00:08.270034Z]     return taskgraph.decision.taskgraph_decision(options)
[task 2017-10-10T15:00:08.270095Z]   File "/builds/worker/checkouts/gecko/taskcluster/taskgraph/decision.py", line 138, in taskgraph_decision
[task 2017-10-10T15:00:08.270151Z]     write_artifact('task-graph.json', tgg.morphed_task_graph.to_json())
[task 2017-10-10T15:00:08.270207Z]   File "/builds/worker/checkouts/gecko/taskcluster/taskgraph/generator.py", line 184, in morphed_task_graph
[task 2017-10-10T15:00:08.270255Z]     return self._run_until('morphed_task_graph')
[task 2017-10-10T15:00:08.270313Z]   File "/builds/worker/checkouts/gecko/taskcluster/taskgraph/generator.py", line 289, in _run_until
[task 2017-10-10T15:00:08.270350Z]     k, v = self._run.next()
[task 2017-10-10T15:00:08.270399Z]   File "/builds/worker/checkouts/gecko/taskcluster/taskgraph/generator.py", line 276, in _run
[task 2017-10-10T15:00:08.270432Z]     do_not_optimize)
[task 2017-10-10T15:00:08.270491Z]   File "/builds/worker/checkouts/gecko/taskcluster/taskgraph/optimize.py", line 42, in optimize_task_graph
[task 2017-10-10T15:00:08.270528Z]     existing_tasks=existing_tasks)
[task 2017-10-10T15:00:08.270588Z]   File "/builds/worker/checkouts/gecko/taskcluster/taskgraph/optimize.py", line 96, in annotate_task_graph
[task 2017-10-10T15:00:08.270629Z]     opt_result = optimize_task(task, params)
[task 2017-10-10T15:00:08.270685Z]   File "/builds/worker/checkouts/gecko/taskcluster/taskgraph/optimize.py", line 54, in optimize_task
[task 2017-10-10T15:00:08.270725Z]     opt_result = opt_fn(task, params, *args)
[task 2017-10-10T15:00:08.270780Z]   File "/builds/worker/checkouts/gecko/taskcluster/taskgraph/optimize.py", line 175, in opt_index_search
[task 2017-10-10T15:00:08.270819Z]     use_proxy=bool(os.environ.get('TASK_ID')))
[task 2017-10-10T15:00:08.270874Z]   File "/builds/worker/checkouts/gecko/taskcluster/taskgraph/util/taskcluster.py", line 90, in find_task_id
[task 2017-10-10T15:00:08.270919Z]     raise KeyError("index path {} not found".format(index_path))
[task 2017-10-10T15:00:08.270991Z] KeyError: u'index path docker.images.v2.level-3.funsize-update-generator.hash.600cc7c926a7c83498b1dd523c8df4a848417220bf6ee0a61a9c14786ca6dec0 not found'

Looks like https://hg.mozilla.org/mozilla-central/rev/83fdd1638d8e has made to beta (57 atm), but the corresponding function usage still uses a different exception https://dxr.mozilla.org/mozilla-beta/rev/b92b69f3503e71ea7a7184f3f56526aafa57d999/taskcluster/taskgraph/optimize.py#178

Mozilla-central has changed it in https://hg.mozilla.org/mozilla-central/rev/79bb52091750#l2.394. It's not the smallest thing to uplift, so I wonder if it's just easier to backout https://hg.mozilla.org/mozilla-central/rev/83fdd1638d8e from beta instead and let the huge change ride the trains.
Dustin, what do you think about backing out https://hg.mozilla.org/mozilla-central/rev/83fdd1638d8e from beta?
Flags: needinfo?(dustin)
Seems like a good fix.  I'm confused how things became dissociated here, but .. ok :)
Flags: needinfo?(dustin)
Assignee: nobody → rail
Comment on attachment 8917099 [details]
Bug 1407279 - Docker image changes on mozilla-beta result in failed decision tasks, Backed out changeset 83fdd1638d8e  a=release

https://reviewboard.mozilla.org/r/188110/#review193264

I assume we'll need to put this back in when the next uplift occurs?
Attachment #8917099 - Flags: review?(dustin) → review+
(In reply to Dustin J. Mitchell [:dustin] from comment #5)
> I assume we'll need to put this back in when the next uplift occurs?

No, no extra work is required. These beta-only changes will be discarded by the next uplift from mozilla-central to mozilla-beta.
Comment on attachment 8917099 [details]
Bug 1407279 - Docker image changes on mozilla-beta result in failed decision tasks, Backed out changeset 83fdd1638d8e  a=release

I'd like to land this on beta (only) to unblock LZMA/BZ2 related work. 

Unfortunately some TC related optimization work in bug 1401183 and bug 1383880 have landed on the same day, but several hours apart. That was enough to get one of the changes in gecko 57, and the other one in 58. Backing out the first part (this patch) from beta sound like the safest solution here. The backout will be discarded as a part of the next uplift.

[Feature/Bug causing the regression]: Bug 1401183 and bug 1383880
[User impact if declined]: None
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No, it affects decision tasks only
[Why is the change risky/not risky?]: It doesn't affect the browser code, automation (scheduling) only
Attachment #8917099 - Flags: approval-mozilla-beta?
Comment on attachment 8917099 [details]
Bug 1407279 - Docker image changes on mozilla-beta result in failed decision tasks, Backed out changeset 83fdd1638d8e  a=release

NPOTB, Beta57+
Attachment #8917099 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Product: TaskCluster → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: