letsencrypt / boulder

An ACME-based certificate authority, written in Go.
Mozilla Public License 2.0
5.22k stars 608 forks source link

Clean up NoWaitForReady #7843

Open jsha opened 1 week ago

jsha commented 1 week ago

In https://github.com/letsencrypt/boulder/issues/6834 we added a config NoWaitForReady that allows gRPC requests to instantly error when there are no backends available (vs waiting for a backend and eventually reporting a timeout). That config is now enabled in prod. We should deprecate and ignore it, effectively never waiting for readiness. As part of that process we should also search the codebase for FailFast (which is the older gRPC terminology for the feature) and clean up relevant comments.