Milo/LUCI: Annotation proto contains a really dumb encoding for property maps |
||||
Issue descriptionSee https://cs.chromium.org/chromium/infra/go/src/go.chromium.org/luci/common/proto/milo/annotations.proto?q=milo+annotation&sq=package:chromium&l=133 ("Property"). In particular this should be the 'well known type' proto.Struct, which is designed to embed JSON inside proto, and has a much more natural API from inside Go/Python.
,
Sep 23 2017
,
Sep 24
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
,
Sep 24
I believe we won't make any changes to this in favor of going forward with buildbucket v2. |
||||
►
Sign in to add a comment |
||||
Comment 1 by no...@chromium.org
, Sep 15 2017