clearlinux/docker
2
0
Fork 0
mirror of https://github.com/clearlinux/docker.git synced 2026-09-03 20:21:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

Use math.MaxInt32 instead of math.MaxUint32

Browse Source 
I think it was original intention, because even half of a comment was about
MaxInt32.

Fix #15038

Signed-off-by: Alexander Morozov <lk4d4@docker.com>
(cherry picked from commit eb45602d2f)
This commit is contained in:
Alexander Morozov
2015-07-28 09:35:56 -07:00
committed by David Calavera
parent 546a704c63
commit 56d859d052
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
daemon/logger/fluentd/fluentd.go
+2 -2
View File
@@ -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
}