Implement per package instance counters. Design: https://docs.google.com/document/d/1RVG9OL11lwXpRc9Y-d6Wo-_WijJ1FV6821fr5-kT_kk/edit
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/01a706d5a431461af911838cf276c33d4b1d9750 commit 01a706d5a431461af911838cf276c33d4b1d9750 Author: David Sansome <dsansome@chromium.org> Date: Tue Aug 23 06:28:23 2016 Instrument chrome_infra_packages with standard ts_mon metrics BUG= 639709 Change-Id: I6e2ec52fcffd290243502e12af5908c93f78cdd0 Reviewed-on: https://chromium-review.googlesource.com/373740 Commit-Queue: Dave Sansome <dsansome@chromium.org> Reviewed-by: Vadim Shtayura <vadimsh@chromium.org> [modify] https://crrev.com/01a706d5a431461af911838cf276c33d4b1d9750/appengine/chrome_infra_packages/admin.py [modify] https://crrev.com/01a706d5a431461af911838cf276c33d4b1d9750/appengine/chrome_infra_packages/app.yaml [modify] https://crrev.com/01a706d5a431461af911838cf276c33d4b1d9750/appengine/chrome_infra_packages/apps.py [modify] https://crrev.com/01a706d5a431461af911838cf276c33d4b1d9750/appengine/chrome_infra_packages/cas/api.py [modify] https://crrev.com/01a706d5a431461af911838cf276c33d4b1d9750/appengine/chrome_infra_packages/cipd/api.py [modify] https://crrev.com/01a706d5a431461af911838cf276c33d4b1d9750/appengine/chrome_infra_packages/cron.yaml [add] https://crrev.com/01a706d5a431461af911838cf276c33d4b1d9750/appengine/chrome_infra_packages/gae_ts_mon [modify] https://crrev.com/01a706d5a431461af911838cf276c33d4b1d9750/appengine_module/gae_ts_mon/__init__.py
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/9916abf4b84fa01a7f6f1e9e4b6b9c1e099e0d1b commit 9916abf4b84fa01a7f6f1e9e4b6b9c1e099e0d1b Author: David Sansome <dsansome@chromium.org> Date: Wed Sep 07 01:04:18 2016 Add a package counter API. Design doc: https://docs.google.com/document/d/1RVG9OL11lwXpRc9Y-d6Wo-_WijJ1FV6821fr5-kT_kk/edit BUG= 639709 Change-Id: Ic93f7a491b8ea81b97449ea22ef80e4b3a753164 Reviewed-on: https://chromium-review.googlesource.com/381434 Commit-Queue: Dave Sansome <dsansome@chromium.org> Reviewed-by: Vadim Shtayura <vadimsh@chromium.org> [modify] https://crrev.com/9916abf4b84fa01a7f6f1e9e4b6b9c1e099e0d1b/appengine/chrome_infra_packages/cipd/acl.py [modify] https://crrev.com/9916abf4b84fa01a7f6f1e9e4b6b9c1e099e0d1b/appengine/chrome_infra_packages/cipd/api.py [modify] https://crrev.com/9916abf4b84fa01a7f6f1e9e4b6b9c1e099e0d1b/appengine/chrome_infra_packages/cipd/impl.py [modify] https://crrev.com/9916abf4b84fa01a7f6f1e9e4b6b9c1e099e0d1b/appengine/chrome_infra_packages/cipd/test/api_test.py [modify] https://crrev.com/9916abf4b84fa01a7f6f1e9e4b6b9c1e099e0d1b/appengine/chrome_infra_packages/cipd/test/impl_test.py
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-go.git/+/6f90e9fa9ff7d57f4fafffbbf7f6916c57e7b673 commit 6f90e9fa9ff7d57f4fafffbbf7f6916c57e7b673 Author: dsansome <dsansome@chromium.org> Date: Wed Sep 07 03:39:45 2016 Add counter-write and counter-read commands. BUG= 639709 Review-Url: https://codereview.chromium.org/2317633002 [modify] https://crrev.com/6f90e9fa9ff7d57f4fafffbbf7f6916c57e7b673/cipd/client/cipd/client.go [modify] https://crrev.com/6f90e9fa9ff7d57f4fafffbbf7f6916c57e7b673/cipd/client/cipd/remote.go [modify] https://crrev.com/6f90e9fa9ff7d57f4fafffbbf7f6916c57e7b673/cipd/client/cipd/remote_test.go [modify] https://crrev.com/6f90e9fa9ff7d57f4fafffbbf7f6916c57e7b673/cipd/client/cmd/cipd/main.go
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-go.git/+/dfbbe36b81c423acea2b915687b4c19e7b4c05cb commit dfbbe36b81c423acea2b915687b4c19e7b4c05cb Author: dsansome <dsansome@chromium.org> Date: Thu Sep 08 05:11:40 2016 Allow COUNTER_WRITER ACLs to be listed and edited. BUG= 639709 Review-Url: https://codereview.chromium.org/2313223002 [modify] https://crrev.com/dfbbe36b81c423acea2b915687b4c19e7b4c05cb/cipd/client/cmd/cipd/main.go
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/e499c7c669900e4af39af8fd3848a8a432dc07f4 commit e499c7c669900e4af39af8fd3848a8a432dc07f4 Author: David Sansome <dsansome@chromium.org> Date: Wed Sep 07 03:45:22 2016 Add a COUNTER_WRITER ACL BUG= 639709 Change-Id: I9becc874bd22d10637993ba42c5772cb36eb5089 Reviewed-on: https://chromium-review.googlesource.com/381747 Reviewed-by: Vadim Shtayura <vadimsh@chromium.org> Commit-Queue: Dave Sansome <dsansome@chromium.org> [modify] https://crrev.com/e499c7c669900e4af39af8fd3848a8a432dc07f4/appengine/chrome_infra_packages/cipd/acl.py [modify] https://crrev.com/e499c7c669900e4af39af8fd3848a8a432dc07f4/appengine/chrome_infra_packages/cipd/api.py [modify] https://crrev.com/e499c7c669900e4af39af8fd3848a8a432dc07f4/appengine/chrome_infra_packages/cipd/test/acl_test.py
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/3bf3e573a78b4dad6fd1ff78992dd3509c520a81 commit 3bf3e573a78b4dad6fd1ff78992dd3509c520a81 Author: David Sansome <dsansome@chromium.org> Date: Thu Sep 08 07:36:10 2016 Oops, fix a typo and add another test... BUG= 639709 TBR=vadimsh Change-Id: I455ec68baf90d914083a913b32d33deab7258e96 Reviewed-on: https://chromium-review.googlesource.com/382893 Reviewed-by: Dave Sansome <dsansome@chromium.org> Commit-Queue: Dave Sansome <dsansome@chromium.org> [modify] https://crrev.com/3bf3e573a78b4dad6fd1ff78992dd3509c520a81/appengine/chrome_infra_packages/cipd/impl.py [modify] https://crrev.com/3bf3e573a78b4dad6fd1ff78992dd3509c520a81/appengine/chrome_infra_packages/cipd/test/impl_test.py
I think this is complete. Or was there anything left to do that I forgot?
Comment 1 by bugdroid1@chromium.org
, Aug 23 2016