New issue
Advanced search Search tips

Issue 685944 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature



Sign in to add a comment

Add /log/$log URL

Project Member Reported by davidri...@chromium.org, Jan 27 2017

Issue description

With buildbot one can take waterfall/builder/build number/step and build a path to the logs.  To do so with logdog requires parsing the annotations and searching for all streams which match the step name.

Make a helper which does the annotation parsing and lookup and redirects to the correct logdog viewer page.
 

Comment 1 by no...@chromium.org, Mar 10 2017

Labels: -Type-Bug luci Type-Feature
Status: Fixed (was: Untriaged)
/logdog/build handler does that, e.g.

https://luci-milo.appspot.com/logdog/build/boringssl/swarm/chromium-swarm.appspot.com/34cd931d400abf11/+/annotations

Comment 2 by d...@chromium.org, Mar 10 2017

:D
I'm not quite sure I follow.

If we have a build, we can construct the path to it (eg "chromeos/cyan-chrome-pfq/912"), how do we used the /logdog/build and specify the step name (eg "VMTest (attempt 1)") with a generated URL that gets us to the logs, similar to how we could with buildbot?
Cc: no...@chromium.org

Comment 5 by no...@chromium.org, Mar 20 2017

Owner: ----
Status: Available (was: Fixed)
I've misinterpreted your request.

Comment 6 by no...@chromium.org, Mar 20 2017

Components: Infra>Platform>LogDog

Comment 7 by d...@chromium.org, Mar 20 2017

Components: -Infra>Platform>LogDog
I don't think this is a LogDog component bug, since all of the data is available. Milo will need to:

1) Load BuildInfo for the build in question.
2) Look up the requested step against the step protobuf.
3) Redirect to the log viewer.

This is all Milo code here.
Project Member

Comment 8 by sheriffbot@chromium.org, Apr 6 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Owner: hinoka@chromium.org
Status: Assigned (was: Untriaged)
Ryan, can you help triage this? 
Is this already possible to do (I think not, but would love to be proven wrong)?
Is this on our (Milo/LogDog) plate?

Summary: Add /log/$log URL (was: Add smart milo log viewer which knows how to parse annotations.)
I believe this is the /log/$log url schema as proposed in https://docs.google.com/document/d/1wnUOrusBq3-DU3NQ8bDIH6Tn6QQzS-C_VnopWHq5np8/edit#heading=h.w32d72usmw4o

Is that what the OP is asking for?
Milo does not currently do this, but it could.

Sign in to add a comment