Issue metadata
Sign in to add a comment
|
Numerically sort step links |
||||||||||||||||||||
Issue descriptionSee step #26 in this: https://ci.chromium.org/buildbot/chromium.android/Marshmallow%2064%20bit%20Tester/18610 Compare that with step #18.7 in: https://build.chromium.org/deprecated/chromium.android/builders/Marshmallow%2064%20bit%20Tester/builds/18610 Milo sorts the links alphabetically. Consequently, the link to shard #10 in milo comes before shard #2. Buildbot however correctly sorts the links alphanumerically so the shard links are in the logical order. Low pri since it's not a big deal.
,
Jan 6 2018
Thanks for filing this. Marking for Afterglow.
,
Jan 8 2018
,
Jan 8 2018
happy to see that links in annotations.proto is not a map https://cs.chromium.org/chromium/infra/go/src/go.chromium.org/luci/common/proto/milo/annotations.proto?l=131
,
Jan 8 2018
Nice. Why not just render them in the order they're specified?
,
Jan 8 2018
sorry, comment4 meant to say that i am happy we don't have this problem in LUCI. In buildbot, unfortunately, links is a map: https://cs.chromium.org/chromium/infra/go/src/go.chromium.org/luci/milo/api/buildbot/structs.go?q=structs.go&sq=package:chromium&rcl=76cce0c4bcec238e865ed54beff6abd23e36f36d&l=47 and looks like this is how we are getting builds from buildbot pubsub https://cs.chromium.org/chromium/infra/go/src/go.chromium.org/luci/milo/buildsource/buildbot/pubsub.go?q=buildbot/pubsub.&sq=package:chromium&l=69
,
Jan 10
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 |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by hinoka@chromium.org
, Jan 5 2018