search

taskcluster / taskcluster-tools

Tools for debugging, inspecting and managing Taskcluster
https://tools.taskcluster.net/
Mozilla Public License 2.0
26 stars 91 forks source link

Bug 1474953 - catch task status 404s and show in table #582

Closed davehouse closed 5 years ago

davehouse commented 5 years ago

When a task is not found (queue task and status requests), the worker explorer currently shows an error for the first task not found. As a user, I would like to see the worker information (first claim, and action buttons), and any valid tasks as well as the unknown task ids.

Example worker view: https://tools.taskcluster.net/provisioners/releng-hardware/worker-types/gecko-t-osx-1010/workers/mdc1/t-yosemite-r7-273

Before: image

After: image

davehouse commented 5 years ago

Great! Thank you for the Nit fixes.

davehouse commented 5 years ago

@helfi92 I noticed that this breaks sorting because the recentTasks are always(?) in reverse order and the sort reverses the list for the ones with times, but undefined ones are left in their original place.

helfi92 commented 5 years ago

Right, it would break the sorting. Perhaps we could simply have those tasks be listed either at the top of end of the list. Thoughts?

davehouse commented 5 years ago

I would like to see the tasks in their chronological order, which seems to be preserved in the task list despite the tasks not being found. What do you think of reversing the list before we sort it? It is a kludge, but the api doesn't give the times although it gives the taskids and runids.

helfi92 commented 5 years ago

Sounds good to me. Feel free to open the PR.

davehouse commented 5 years ago

Thank you. I opened https://github.com/taskcluster/taskcluster-tools/pull/585 to add the .reverse()