Merge pull request #3482 from powerhome/bugfix/keepalive-mismatch

Tiller should only enforce what we expect from Helm
pull/4207/head^2
Matthew Fisher 8 years ago committed by GitHub
commit 37d26009b7

@ -157,11 +157,15 @@ func start() {
logger.Fatalf("Could not create server TLS configuration: %v", err) logger.Fatalf("Could not create server TLS configuration: %v", err)
} }
opts = append(opts, grpc.Creds(credentials.NewTLS(cfg))) opts = append(opts, grpc.Creds(credentials.NewTLS(cfg)))
}
opts = append(opts, grpc.KeepaliveParams(keepalive.ServerParameters{ opts = append(opts, grpc.KeepaliveParams(keepalive.ServerParameters{
MaxConnectionIdle: 10 * time.Minute, MaxConnectionIdle: 10 * time.Minute,
// If needed, we can configure the max connection age // If needed, we can configure the max connection age
})) }))
} opts = append(opts, grpc.KeepaliveEnforcementPolicy(keepalive.EnforcementPolicy{
MinTime: time.Duration(20) * time.Second, // For compatibility with the client keepalive.ClientParameters
}))
rootServer = tiller.NewServer(opts...) rootServer = tiller.NewServer(opts...)

Loading…
Cancel
Save