f5-declarative-onboarding icon indicating copy to clipboard operation
f5-declarative-onboarding copied to clipboard

Fail quickly on bad "targetPassphrase" value

Open greaves-machine opened this issue 3 years ago • 3 comments

Is your feature request related to a problem? Please describe.

I'm sending DO declarations via Big-IQ. If I send DO with a clear and obvious error (incorrect Big-IP password for example), the server side takes at least 15 minutes to stop its retry loop. See below, a retry count of 350+ that decrements every 5 seconds.

Thu, 12 May 2022 10:04:05 GMT - finest: [f5-declarative-onboarding: restWorker.js] tryUntil: got error undefined Thu, 12 May 2022 10:04:05 GMT - finest: [f5-declarative-onboarding: restWorker.js] typeof err undefined Thu, 12 May 2022 10:04:05 GMT - finer: [f5-declarative-onboarding: restWorker.js] tryUntil error: tries left: 352 Thu, 12 May 2022 10:04:05 GMT - finest: [f5-declarative-onboarding: restWorker.js] tryUntil: retryOrReject: numRemaining: 352

Describe the solution you'd like

Declarative Onboarding should fail quickly on an obviously unrecoverable error like a bad password.

Describe alternatives you've considered

None

Additional context

Deploying via Big-IQ

greaves-machine avatar May 12 '22 10:05 greaves-machine

Thank you for your feedback. Unfortunately, making that value configurable by the user would be difficult since it is reused for many DO actions. Some of those actions will require longer retries while DO is waiting for configuration changes to apply on the BIG-IP.

However, we can look into making DO fail faster in the case where the BIG-IP password is incorrect. Would you like me to update your request to match this solution?

dstokesf5 avatar May 26 '22 19:05 dstokesf5

Thanks for the reply @dstokesf5 - yes please, that would definitely be a help

greaves-machine avatar May 27 '22 07:05 greaves-machine

I have added AUTOTOOL-3337 to our internal product backlog to fail quickly in this case instead of retrying with a bad password.

dstokesf5 avatar Jun 02 '22 19:06 dstokesf5