New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 808529 link

Starred by 1 user
Status: Fixed
Owner:
qyears...@chromium.org
Closed: Mar 2018
Cc:
qyears...@chromium.org
█ emso@chromium.org
mar...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature



Sign in to add a comment

Add a debug page showing information for a run ID

Project Member Reported by qyears...@chromium.org, Feb 2 2018 
Logging for many request types in Tricium includes the run ID but not much other information about the run.

For debugging purposes, I think it will be useful to have a handler that shows some basic details about a run specified in the URL.

For example, "tricium-dev.appspot.com/run/1234512345" could list out a tree of the related entities and all of their fields:

 - AnalyzeRequest
  - AnalyzeRequestResult
  - WorkflowRun
   - WorkflowRunResult
  - FunctionRuns(s)
   - FunctionRunResult(s)
   - WorkerRun(s)
    - WorkerRunResult(s)
    - Comment(s)
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 20 2018 

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/bab8b1b9876bd8ddc3a01a5b5440f8cef631aea8

commit bab8b1b9876bd8ddc3a01a5b5440f8cef631aea8
Author: Quinten Yearsley <qyearsley@chromium.org>
Date: Tue Feb 20 17:10:17 2018

Add a /run page which shows details given a run ID

Purpose:
Most of the errors logged in other handlers will log a run ID, but not much else.
For now, this will help with debugging other errors. In the initial version, the
run page shows basic information about a run, including a link to the Gerrit change.

Also, eventually I think we'd like to add a user-facing page that we can link
to for run details.

Example page:
https://13988-129681c-tainted-qyearsley-dot-tricium-dev.appspot.com/run/6224707376381952

Bug:  808529 
Change-Id: I0b043ea5e39eab5e7650383cc3fa3c9d034e36a6
Reviewed-on: https://chromium-review.googlesource.com/912658
Commit-Queue: Quinten Yearsley <qyearsley@chromium.org>
Reviewed-by: Marc-Antoine Ruel <maruel@chromium.org>

[add] https://crrev.com/bab8b1b9876bd8ddc3a01a5b5440f8cef631aea8/go/src/infra/tricium/appengine/templates/pages/run.html
[modify] https://crrev.com/bab8b1b9876bd8ddc3a01a5b5440f8cef631aea8/go/src/infra/tricium/appengine/frontend/handlers_test.go
[modify] https://crrev.com/bab8b1b9876bd8ddc3a01a5b5440f8cef631aea8/go/src/infra/tricium/appengine/common/track/db.go
[modify] https://crrev.com/bab8b1b9876bd8ddc3a01a5b5440f8cef631aea8/go/src/infra/tricium/appengine/common/track/db_test.go
[modify] https://crrev.com/bab8b1b9876bd8ddc3a01a5b5440f8cef631aea8/go/src/infra/tricium/appengine/frontend/handlers.go
[add] https://crrev.com/bab8b1b9876bd8ddc3a01a5b5440f8cef631aea8/go/src/infra/tricium/appengine/frontend/run_test.go
[add] https://crrev.com/bab8b1b9876bd8ddc3a01a5b5440f8cef631aea8/go/src/infra/tricium/appengine/frontend/run.go
[modify] https://crrev.com/bab8b1b9876bd8ddc3a01a5b5440f8cef631aea8/go/src/infra/tricium/appengine/frontend/init.go
[modify] https://crrev.com/bab8b1b9876bd8ddc3a01a5b5440f8cef631aea8/go/src/infra/tricium/appengine/common/testing/testing.go
Project Member

Comment 2 by bugdroid1@chromium.org, Feb 26 2018 

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/d358ebccb605973d340b520d346927998de5cba9

commit d358ebccb605973d340b520d346927998de5cba9
Author: Quinten Yearsley <qyearsley@chromium.org>
Date: Mon Feb 26 16:23:15 2018

Follow-up fix for run page: fix URL and worker list

Follow-up to https://crrev.com/c/912658. This fixes two minor issues:

 1. The Gerrit URL shouldn't include the last two digits of the
    change number, which are in the git ref.

 2. The worker list under each function should only include workers
    for that function.

Bug:  808529 
Change-Id: Ib5b5c498538cc9fe48f2b4e2a8880e8c4ccf44b2
Reviewed-on: https://chromium-review.googlesource.com/932946
Commit-Queue: Marc-Antoine Ruel <maruel@chromium.org>
Reviewed-by: Marc-Antoine Ruel <maruel@chromium.org>

[modify] https://crrev.com/d358ebccb605973d340b520d346927998de5cba9/go/src/infra/tricium/appengine/frontend/run.go
[modify] https://crrev.com/d358ebccb605973d340b520d346927998de5cba9/go/src/infra/tricium/appengine/frontend/run_test.go

Comment 3 by qyears...@chromium.org, Mar 27 2018

Owner: qyears...@chromium.org
Status: Fixed (was: Available)
Done in late Feb, but not very pretty. Example page:
https://tricium-dev.appspot.com/run/5642383161032704
►

Sign in to add a comment