I have evaluated TODO retry remove if feasible (#112383)

Hi team, hope u all doing well.

I have checked TODO that to remove "retry" if feasible but it's important i think that it shouldn't be removed because it was used in every file on your repo.

Update idempotency.go

Update idempotency.go

Update idempotency.go
This commit is contained in:
B Aravind 2022-09-14 11:01:00 +05:30 committed by GitHub
parent d5e5f48b88
commit b307321c0a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -341,8 +341,6 @@ func PatchNode(client clientset.Interface, nodeName string, patchFn func(*v1.Nod
// GetConfigMapWithRetry tries to retrieve a ConfigMap using the given client,
// retrying if we get an unexpected error.
//
// TODO: evaluate if this can be done better. Potentially remove the retry if feasible.
func GetConfigMapWithRetry(client clientset.Interface, namespace, name string) (*v1.ConfigMap, error) {
var cm *v1.ConfigMap
var lastError error