Buildbot ui for a failing build lists the steps that failed at the top of the page: https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_swarming_rel/builds/280068 Milo ui does not: https://ci.chromium.org/buildbot/tryserver.chromium.android/android_n5x_swarming_rel/280068 I find this useful when I'm linked to a failing build so that I can immediately tell what went wrong without having to scan the entire set of steps. Additionally, some builds have several hundred steps and more often than not I'm only interested in the failing steps. Without the list at the top, I have to scroll all the way down until I find a failing step. With the list there, I can simply ctrl-c and ctrl-f the step name to immediately jump down to its location.
looks like it is a regression https://2107-1af25a3-dot-luci-milo.appspot.com/buildbot/tryserver.chromium.android/android_n5x_swarming_rel/ i will take a look
wrong, that's a different page. Looks like not a regression
Non parity between buildbot and Milo's emulation of buildbot can be considered a bug. I'll take a look.
This appears to be a regression and not parity. Marking as Blocker for UI.
Ryan, what's the status of this?
The following revision refers to this bug: https://chromium.googlesource.com/infra/luci/luci-go.git/+/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0 commit 5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0 Author: Ryan Tseng <hinoka@google.com> Date: Tue Nov 28 01:46:37 2017 Milo: Use Step text instead of status on build page if possible This displays the step text instead of our custom status message if the step text is available. This is closer in line with previous behavior and user expectations. Bug: 773893 Change-Id: Ic9edda7170d30801c90bafbd3df0b0ef370e3964 Reviewed-on: https://chromium-review.googlesource.com/792531 Commit-Queue: Ryan Tseng <hinoka@chromium.org> Reviewed-by: Nodir Turakulov <nodir@chromium.org> [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/buildsource/buildbot/build.go [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/buildsource/buildbot/builder.go [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/buildsource/buildbot/expectations/CrWinGoma.30608.build.json [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/buildsource/buildbot/expectations/chromium_presubmit.426944.build.json [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/buildsource/buildbot/expectations/gerritCL.1234.build.json [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/buildsource/buildbot/expectations/newline.1234.build.json [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/buildsource/buildbot/expectations/win_chromium_rel_ng.246309.build.json [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/appengine/templates/pages/build.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/buildbot.build-Debug_page-_CrWinGoma_30608.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/buildbot.build-Debug_page-_chromium_presubmit_426944.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/buildbot.build-Debug_page-_gerritCL_1234.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/buildbot.build-Debug_page-_newline_1234.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/buildbot.build-Debug_page-_win_chromium_rel_ng_246309.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/swarming.build-Basic_successful_build.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/swarming.build-build-canceled.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/swarming.build-build-exception.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/swarming.build-build-expired.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/swarming.build-build-finished-logdog-expired-stream.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/swarming.build-build-gerrit.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/swarming.build-build-internal.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/swarming.build-build-link.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/swarming.build-build-nested.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/swarming.build-build-patch-failure.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/swarming.build-build-pending-logdog.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/swarming.build-build-pending.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/swarming.build-build-running-logdog-no-annotation-stream.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/swarming.build-build-running-logdog.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/swarming.build-build-running.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/swarming.build-build-timeout.html [modify] https://crrev.com/5fb02a6c49ac8bae39e0197e4817b16ad9e3cda0/milo/frontend/expectations/swarming.build-build-unicode.html
Comment 1 by no...@chromium.org
, Oct 12 2017Labels: -Pri-3 Pri-2
Owner: no...@chromium.org
Status: Assigned (was: Untriaged)