Fix rate limiting for logger, increase refill rate

Signed-off-by: Ethan Mosbaugh <ethan@replicated.com>
(cherry picked from commit 50c6a5fb07c873fd5c9ae3fb6294e0b11c689de2)
Signed-off-by: Brian Goff <cpuguy83@gmail.com>
Upstream-commit: e6c9e2736f5d522491fe5283029edbc166a37d2a
Component: engine

Author: emosbaugh

components/engine/daemon/cluster/executor/container/controller.go

@@ -508,7 +508,9 @@ func (r *controller) Logs(ctx context.Context, publisher exec.LogPublisher, opti
 	var (
 		// use a rate limiter to keep things under control but also provides some
 		// ability coalesce messages.
-		limiter = rate.NewLimiter(rate.Every(time.Second), 10<<20) // 10 MB/s
+		// this will implement a "token bucket" of size 10 MB, initially full and refilled
+		// at rate 10 MB tokens per second.
+		limiter = rate.NewLimiter(10<<20, 10<<20) // 10 MB/s
 		msgctx  = api.LogContext{
 			NodeID:    r.task.NodeID,
 			ServiceID: r.task.ServiceID,