1357802 - Show task.extra in task inspector

Status: RESOLVED FIXED

(Taskcluster :: UI, enhancement)

Description

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

The "task details" view of the task inspector omits task.extra.  It should be displayed as JSON just like the task payload.

Comment 1

[Swapnesh Kumar Sahoo [:swapneshks]]

Hi.. I'd like to work on this bug.
I'm a beginner and still catching up to things. (Have worked on only 2-3 moz bugs till now)

Comment 2

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

Hi!  This is a frontend fix, in a React application.  The source code is at https://github.com/taskcluster/taskcluster-tools

You can see the issue here:
  https://tools.taskcluster.net/task-inspector/#FLLNwbUVT1eoiRUaAJNobw/

The raw task definition being displayed is
  https://queue.taskcluster.net/v1/task/FLLNwbUVT1eoiRUaAJNobw
and you can see it has an "extra" property with a bunch of data in it.  However, that doesn't appear in the UI at the first link.

You should be able to duplicate the error using your local install (with the same taskId), and then get started fixing it :)

Comment 3

[Swapnesh Kumar Sahoo [:swapneshks]]

Attachment: Fix for displaying task.extra in "Task Details" view

In this patch I am assuming that the contents of task.extra have to be parsed as JSON. 
Do let me know if this patch is appropriate.

Comment 4

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

Comment on attachment 8862068 [details] [diff] [review]
Fix for displaying task.extra in "Task Details" view

Review of attachment 8862068 [details] [diff] [review]:
-----------------------------------------------------------------

That looks good!  Can you submit a Github pull request so we can get the automated tests run?

Comment 5

[Swapnesh Kumar Sahoo [:swapneshks]]

Pull request created at:

https://github.com/taskcluster/taskcluster-tools/pull/209

Comment 6

[Swapnesh Kumar Sahoo [:swapneshks]]

Hi Dustin

Could you provide me with a task definition that does not have task.extra so that I can test it and apply the corresponding fix as discussed in the pull request?

Comment 7

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

https://tools.taskcluster.net/task-inspector/#VnfsiYQWTt-qC6TMzu6nXg/

Comment 8

[Swapnesh Kumar Sahoo [:swapneshks]]

The pull request has been updated with the requested changes (for the case when task.extra does not exist).

Comment 9

[[github robot]]

Commit pushed to master at https://github.com/taskcluster/taskcluster-tools

https://github.com/taskcluster/taskcluster-tools/commit/eca8f9c7aba1f23405c3415bad101bcbbe493eb7
Bug 1357802 - Show task.extra in task inspector (#209)

* Bug 1357802 - Show task.extra in task inspector

Add fix for displaying "Extra" from task.extra in "Task Details" view

* Bug 1357802 - Add condition for no task.extra

Add condition to skip task.extra rendering when it is non-existent

* Bug 1357802 - Treat task.extra as an object

task.extra will be a plain object when not specified

Comment 10

[:Eli Perelman]

Landed in master via https://github.com/taskcluster/taskcluster-tools/commit/eca8f9c7aba1f23405c3415bad101bcbbe493eb7