Issue metadata
Sign in to add a comment
|
builder links should be to public buildbot URLs by default |
||||||||||||||||||||||||
Issue descriptionWhen I file a bug and link to buildbot for revert, it's important that it be a publicly accessible URL so that if the author of the patch doesn't work at Google, they can actually see the failure without needing to get help from a Googler. So, the builder links thing should link to buildbot by default. I understand that some bots only have an internal one, and so should link to the appropriate internal URL. Why is there a radio button at all for this? Why isn't it just linking to buildbot unless it's an internal only bot, in which case it links to uberchromegw?
,
Apr 19 2017
uberchromegw offers extra features like canceling/retrying builds, comments on steps etc. The radio button is to support sheriffs who want to use these features. It is probably better to link to public by default but uberchromegw for now, if we know it's an internal builder. Eventually Milo should probably implement the extra uberchromegw features and we can delegate the ACLing to it once we link to it for all builds.
,
Jun 29 2017
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by zhangtiff@chromium.org
, Apr 19 2017