Expose defaultUnaryMaxRetries and to make it configurable

[lilic]

etcd/client/v3/client.go

Line 241 in 4cbb949

grpc.WithUnaryInterceptor(c.unaryClientInterceptor(withMax(defaultUnaryMaxRetries), rrBackoff)),

uses the defaultUnaryMaxRetries to set the max retries, which is then checked if it is 0 and if it is we skip retries. But from my understanding, this can never be set or configured by users so it will always retry 100 times.

Retrying 100 times can result in very noisy logs (if the log level is warning for example), but also unnecessary retries if this is not something the user wants. I am proposing to make max retries configurable. In our (OpenShift) use case of the operator, we reconcile often enough that we do not require to retry 100 times.

Sorry if I missed something while going through the code!

[lilic]

cc @hexfusion for context, this comes from our noisy logs, where we see the retrying of unary invoker failed.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 21 days if no further activity occurs. Thank you for your contributions.

[MatteoGioioso]

Hello, was this implemented? I cannot find a way to configure it.