Recipes: ok_ret=any makes steps print all possible return values. |
||||||
Issue descriptionMaybe we could print something more readable? E.g. 'any' if we mean any? See e.g.: https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fclient.v8.branches%2FAuto-tag%2F623%2F%2B%2Frecipes%2Fsteps%2Fgit_branch%2F0%2Fstdout It prints as part of the step data: ok_ret: frozenset([0, 1, 2, ..., -3, -1]) The ... are from me. They skip the other ~500 numbers in the output.
,
May 16 2018
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
,
May 16 2018
Now it's not in one line anymore, but instead in 512 separate lines :( - recent link: https://logs.chromium.org/v/?s=chromium%2Fbb%2Fclient.v8.branches%2FAuto-tag%2F1727%2F%2B%2Frecipes%2Fsteps%2Fgit_branch%2F0%2Fstdout It's still not high prio, but it'd still be nice to improve this output.
,
Aug 29
,
Sep 14
The change didn't get picked up for whatever reason, but this was fixed in https://crrev.com/c/1195592
,
Sep 18
Cool, thanks! Looks good now, example: https://logs.chromium.org/logs/v8/buildbucket/cr-buildbucket.appspot.com/8935055391562383376/+/steps/git_branch/0/stdout |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by iannucci@chromium.org
, May 15 2017Summary: Recipes: ok_ret=any makes steps print all possible return values. (was: Recipes: ok_ret=any makes steps print all possible return valures.)