Code review comment for lp:~mfoord/juju-core/instancepoller-aggregate

Revision history for this message
Michael Foord (mfoord) wrote :

Reviewers: mp+209966_code.launchpad.net,

Message:
Please take a look.

Description:
worker/instancepoller: batch instancepoller

use ratelimit Token Bucket to batch
requests in the instance poller.

Mitigates launchpad issue 1277397

https://code.launchpad.net/~mfoord/juju-core/instancepoller-aggregate/+merge/209966

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/74900044/

Affected files (+324, -18 lines):
   A [revision details]
   M dependencies.tsv
   A worker/instancepoller/aggregate.go
   A worker/instancepoller/aggregate_test.go
   M worker/instancepoller/worker.go
   M worker/instancepoller/worker_test.go

« Back to merge proposal