{"action":"created","issue":{"url":"https://api.github.com/repos/bazelbuild/remote-apis/issues/36","repository_url":"https://api.github.com/repos/bazelbuild/remote-apis","labels_url":"https://api.github.com/repos/bazelbuild/remote-apis/issues/36/labels{/name}","comments_url":"https://api.github.com/repos/bazelbuild/remote-apis/issues/36/comments","events_url":"https://api.github.com/repos/bazelbuild/remote-apis/issues/36/events","html_url":"https://github.com/bazelbuild/remote-apis/pull/36","id":377462552,"node_id":"MDExOlB1bGxSZXF1ZXN0MjI4Mzk5OTI0","number":36,"title":"Add an additional_info_url fields to ExecuteResponse.","user":{"login":"EdSchouten","id":736085,"node_id":"MDQ6VXNlcjczNjA4NQ==","avatar_url":"https://avatars1.githubusercontent.com/u/736085?v=4","gravatar_id":"","url":"https://api.github.com/users/EdSchouten","html_url":"https://github.com/EdSchouten","followers_url":"https://api.github.com/users/EdSchouten/followers","following_url":"https://api.github.com/users/EdSchouten/following{/other_user}","gists_url":"https://api.github.com/users/EdSchouten/gists{/gist_id}","starred_url":"https://api.github.com/users/EdSchouten/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/EdSchouten/subscriptions","organizations_url":"https://api.github.com/users/EdSchouten/orgs","repos_url":"https://api.github.com/users/EdSchouten/repos","events_url":"https://api.github.com/users/EdSchouten/events{/privacy}","received_events_url":"https://api.github.com/users/EdSchouten/received_events","type":"User","site_admin":false},"labels":[{"id":954200317,"node_id":"MDU6TGFiZWw5NTQyMDAzMTc=","url":"https://api.github.com/repos/bazelbuild/remote-apis/labels/cla:%20yes","name":"cla: yes","color":"c2e0c6","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2018-11-05T15:54:47Z","updated_at":"2018-11-06T09:33:45Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/bazelbuild/remote-apis/pulls/36","html_url":"https://github.com/bazelbuild/remote-apis/pull/36","diff_url":"https://github.com/bazelbuild/remote-apis/pull/36.diff","patch_url":"https://github.com/bazelbuild/remote-apis/pull/36.patch"},"body":"As mentioned in https://github.com/bazelbuild/bazel/pull/6591, Bazel\r\nBuildbarn now ships with a web service that can be used to explore data\r\nstored in the CAS and AC.\r\n\r\nBy adding an extra field to ExecuteResponse, we can let Bazel\r\nautomatically report URLs to the user pointing to the correct page.\r\nThese URLs could, for example, be printed for build steps that fail."},"comment":{"url":"https://api.github.com/repos/bazelbuild/remote-apis/issues/comments/436188225","html_url":"https://github.com/bazelbuild/remote-apis/pull/36#issuecomment-436188225","issue_url":"https://api.github.com/repos/bazelbuild/remote-apis/issues/36","id":436188225,"node_id":"MDEyOklzc3VlQ29tbWVudDQzNjE4ODIyNQ==","user":{"login":"finnball","id":31440399,"node_id":"MDQ6VXNlcjMxNDQwMzk5","avatar_url":"https://avatars2.githubusercontent.com/u/31440399?v=4","gravatar_id":"","url":"https://api.github.com/users/finnball","html_url":"https://github.com/finnball","followers_url":"https://api.github.com/users/finnball/followers","following_url":"https://api.github.com/users/finnball/following{/other_user}","gists_url":"https://api.github.com/users/finnball/gists{/gist_id}","starred_url":"https://api.github.com/users/finnball/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/finnball/subscriptions","organizations_url":"https://api.github.com/users/finnball/orgs","repos_url":"https://api.github.com/users/finnball/repos","events_url":"https://api.github.com/users/finnball/events{/privacy}","received_events_url":"https://api.github.com/users/finnball/received_events","type":"User","site_admin":false},"created_at":"2018-11-06T09:33:45Z","updated_at":"2018-11-06T09:33:45Z","author_association":"NONE","body":"> > I'm unsure if support for URLs with additional information really fits into the Remote Execution API. It may be an idea to have this as some kind of Any field whereby you can pack in your own implementation specific protos.\r\n> \r\n> Do you have specific concerns? I don't like Any in that it hurts cross compatibility between clients and services - it's poking a hole into a well specified protocol.\r\n\r\nApologies, I amended my answer as you posted after realising it wasn't a sensible suggestion. For me, this is more of a metadata result than something essential to the `ExecutionResponse`."}}
{"actor":{"display_login":"finnball"}}