retries and timeout when managing ACLs with the ClusterAdmin

[nkostoulas]

👋 We're trying to use the ACL methods of the ClusterAdmin and noticed there's no option for retries or timeouts even though these are specified in the Admin part of the Config and are used with Topics related methods. Is there a reason behind this? Can it be assumed that these are less likely to flake or drag on or should we be managing this on our end?

[dnwe]

No they're equally likely to hit a condition of ErrNoController so you're correct that they should really be wrappered in a similar ca.retryOnError(isErrNoController, ...) { ... } call

[dnwe]

@hindessm similar to #2520

[hindessm]

Is it similar though? Looking at KIP-140, it says:

CreateAclsRequest contains a list of ACLs to add. It can be sent to any broker.

and:

Just like CreateAclsRequest, DeleteAclsRequest can be sent to any broker.

I didn't spot an explicit statement regarding the Describe request but I don't imagine that would be more strict.

[github-actions]

Thank you for taking the time to raise this issue. However, it has not had any activity on it in the past 90 days and will be closed in 30 days if no updates occur.
Please check if the main branch has already resolved the issue since it was raised. If you believe the issue is still valid and you would like input from the maintainers then please comment to ask for it to be reviewed.