Issue metadata
Sign in to add a comment
|
Buildbucket should supply 'log_location' and 'luci_project' as first-class fields in the build message |
||||||||||||||||||||||||
Issue descriptionCurrently they must be inferred from the swarming tags, which is pretty gross. Ex: https://chromium.googlesource.com/infra/luci/luci-go/+/679e324d6e5a7c75db9356ddf134577b8d1b504e/milo/buildsource/buildbucket/pubsub.go#188
,
Nov 1 2017
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/43fc206f86628011113b4723b38f206d1690842d commit 43fc206f86628011113b4723b38f206d1690842d Author: Michael Anthony Knyszek <mknyszek@google.com> Date: Wed Nov 01 01:31:10 2017 [buildbucket] Add project to Build message. This change not only adds project to the Build message, but also propagates the project name from the API level to the rest of the backend. This is to prepare for bucket names to be contained within a project's namespace. Bug: 776300 Change-Id: Ia968a0edb669ac5e3be44351d12552c15238d872 Reviewed-on: https://chromium-review.googlesource.com/743806 Commit-Queue: Michael Knyszek <mknyszek@google.com> Reviewed-by: Nodir Turakulov <nodir@chromium.org> [modify] https://crrev.com/43fc206f86628011113b4723b38f206d1690842d/appengine/cr-buildbucket/api_common.py [modify] https://crrev.com/43fc206f86628011113b4723b38f206d1690842d/appengine/cr-buildbucket/model.py [modify] https://crrev.com/43fc206f86628011113b4723b38f206d1690842d/appengine/cr-buildbucket/test/api_test.py [modify] https://crrev.com/43fc206f86628011113b4723b38f206d1690842d/appengine/cr-buildbucket/service.py [modify] https://crrev.com/43fc206f86628011113b4723b38f206d1690842d/appengine/cr-buildbucket/api.py [modify] https://crrev.com/43fc206f86628011113b4723b38f206d1690842d/appengine/cr-buildbucket/test/service_test.py
,
Nov 10 2017
the "luci_project" piece of this bug is fixed by #2
,
Apr 30 2018
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by no...@chromium.org
, Oct 19 2017Status: Assigned (was: Untriaged)