commit | cd881a2971e96fb157dc8a8a92baf813bbd7c751 | [log] [tgz] |
---|---|---|
author | Allen Xie <zhihuixie@google.com> | Tue Apr 02 04:17:47 2024 |
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | Wed Apr 24 00:23:10 2024 |
tree | 1a87510e73ef0117750fa74a010c7bd50b508a37 | |
parent | c4a051f8bb21e57a44702b8041aa796fac8b30e7 [diff] |
Reland "rdb-publish: Upload invocation-level properties to ResultDB" This is a reland of commit b40d618abc6303a78a0c49f1312f8976fd5d29bc Relands this CL because RDB has already updated the size limit of the properties field (https://crrev.com/c/5446183). The size limit update has also been released to prod. Original change's description: > rdb-publish: Upload invocation-level properties to ResultDB > > Design Doc: http://docs/document/d/1Wiex9yIUVA0i4afqu2JDBvxSn_9gKZR4sWHWf5ldLHA?tab=t.0#heading=h.d7np0od0228r > > The `-inv-properties-file` flag has been supported in RDB API: > https://source.chromium.org/chromium/infra/infra/+/main:recipes-py/recipe_modules/resultdb/api.py;l=736-740?q=recipes-py%2Frecipe_modules%2Fresultdb%2Fapi.py > > This CL is to make use of this flag to upload inv-level properties. Then > the inv-level properties will be populated in the `parent.properties` > field in RDB BQ schema. > > BUG=b:331721226 > TEST=Ran LED test based on the built container from CQ dry run. > > LED test link: https://ci.chromium.org/raw/build/logs.chromium.org/chromeos/led/zhihuixie_google.com/54d82995f1bb8f0acda7d8ae28b7ff2ac7e565bcea0b1a5a948eef80f33f2ac2/+/build.proto?server=chromeos-swarming.appspot.com > > RDB upload log: https://screenshot.googleplex.com/AEbAtsxiV3be6B7.png > RDB data: https://screenshot.googleplex.com/6w34q9iKuvbAb2v.png > > Change-Id: I44c997e287e946504b6e6cb3a3aecdef18e365bb > Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/dev-util/+/5401642 > Reviewed-by: Azizur Rahman <azrahman@google.com> > Commit-Queue: Zhihui Xie <zhihuixie@google.com> > Reviewed-by: Jason Kusuma <jkusuma@google.com> > Reviewed-by: Weiwei Lin <weiweilin@google.com> > Tested-by: Zhihui Xie <zhihuixie@google.com> Bug: b:331721226 Change-Id: I06f07363877f388d0e6e499663dbdcf37c333c6c Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/dev-util/+/5465009 Commit-Queue: Zhihui Xie <zhihuixie@google.com> Tested-by: Zhihui Xie <zhihuixie@google.com> Reviewed-by: Jason Kusuma <jkusuma@google.com> Reviewed-by: Azizur Rahman <azrahman@google.com> Reviewed-by: Weiwei Lin <weiweilin@google.com>