New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Increase ACME control loop & order creation back-off #1195
Increase ACME control loop & order creation back-off #1195
Conversation
…ff to 1h. Fire Event when Order fails. Signed-off-by: James Munnelly <james@munnelly.eu>
/retest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: kragniz The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@munnerz want to open an issue for this? |
What this PR does / why we need it:
In future I'd like to make (3) also exponentially back-off as-per the Let's Encrypt client guidelines. This will require us to record more information about how many failures have passed, so I am deferring this for now.
(1) should also help reduce the number of times we re-attempt calls to the ACME server. With a base of 5s and a max of 30m, we will quickly only be re-attempting client calls every 30 minutes in the event they are persistently failing (i.e. due to certain endpoints hitting rate limits)
We are now able to use these higher numbers as a result of refactoring the way we use errors in the ACME orders & challenges process. For normal operation, this should be a non-issue for the majority of users. Only users who have got misconfigured setups will see long wait times.
Release note: