The "task details" view of the task inspector omits task.extra. It should be displayed as JSON just like the task payload.
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)
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 :)
Created attachment 8862068 [details] [diff] [review] 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 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?
Pull request created at: https://github.com/taskcluster/taskcluster-tools/pull/209
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?
The pull request has been updated with the requested changes (for the case when task.extra does not exist).
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
Landed in master via https://github.com/taskcluster/taskcluster-tools/commit/eca8f9c7aba1f23405c3415bad101bcbbe493eb7