From 905c09e034173d4dc7e780905f1f69d28bbc5d70 Mon Sep 17 00:00:00 2001 From: Alexander Morozov Date: Mon, 27 Jul 2015 13:11:08 -0700 Subject: [PATCH] Use math.MaxInt32 instead of math.MaxUint32 I think it was original intention, because even half of a comment was about MaxInt32. Fix #15038 Signed-off-by: Alexander Morozov Upstream-commit: eb45602d2fa0fac8a694d2afb1c59ef60b0e1f77 Component: engine --- components/engine/daemon/logger/fluentd/fluentd.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/components/engine/daemon/logger/fluentd/fluentd.go b/components/engine/daemon/logger/fluentd/fluentd.go index 726d2be758..97205ddee2 100644 --- a/components/engine/daemon/logger/fluentd/fluentd.go +++ b/components/engine/daemon/logger/fluentd/fluentd.go @@ -93,9 +93,9 @@ func New(ctx logger.Context) (logger.Logger, error) { } logrus.Debugf("logging driver fluentd configured for container:%s, host:%s, port:%d, tag:%s.", ctx.ContainerID, host, port, tag) - // logger tries to recoonect 2**64 - 1 times + // logger tries to recoonect 2**32 - 1 times // failed (and panic) after 204 years [ 1.5 ** (2**32 - 1) - 1 seconds] - log, err := fluent.New(fluent.Config{FluentPort: port, FluentHost: host, RetryWait: 1000, MaxRetry: math.MaxUint32}) + log, err := fluent.New(fluent.Config{FluentPort: port, FluentHost: host, RetryWait: 1000, MaxRetry: math.MaxInt32}) if err != nil { return nil, err }