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>
| ... | ... |
@@ -93,9 +93,9 @@ func New(ctx logger.Context) (logger.Logger, error) {
|
| 93 | 93 |
} |
| 94 | 94 |
logrus.Debugf("logging driver fluentd configured for container:%s, host:%s, port:%d, tag:%s.", ctx.ContainerID, host, port, tag)
|
| 95 | 95 |
|
| 96 |
- // logger tries to recoonect 2**64 - 1 times |
|
| 96 |
+ // logger tries to recoonect 2**32 - 1 times |
|
| 97 | 97 |
// failed (and panic) after 204 years [ 1.5 ** (2**32 - 1) - 1 seconds] |
| 98 |
- log, err := fluent.New(fluent.Config{FluentPort: port, FluentHost: host, RetryWait: 1000, MaxRetry: math.MaxUint32})
|
|
| 98 |
+ log, err := fluent.New(fluent.Config{FluentPort: port, FluentHost: host, RetryWait: 1000, MaxRetry: math.MaxInt32})
|
|
| 99 | 99 |
if err != nil {
|
| 100 | 100 |
return nil, err |
| 101 | 101 |
} |