Skip to content

Conversation

@zoltan-frm
Copy link
Contributor

  • Added:LockLeaseRenewer interface to support renewing lock leases. renewLockLease function to extend lease duration based on retry attempts.
  • Updated certificate management functions to renew lock leases during long-running retry loops.
  • Added tests for lease renewal functionality in config_test.go to ensure correct behavior with and without lease support.

…t lease renewal.

- Added:`LockLeaseRenewer` interface to support renewing lock leases. `renewLockLease` function to extend lease duration based on retry attempts.
- Updated certificate management functions to renew lock leases during long-running retry loops.
- Added tests for lease renewal functionality in `config_test.go` to ensure correct behavior with and without lease support.
@zoltan-frm zoltan-frm force-pushed the feature/controlled-certstore-locks branch from 43b13c9 to 0412a9b Compare August 19, 2025 09:53
@zoltan-frm zoltan-frm marked this pull request as ready for review September 3, 2025 08:47
@zoltan-frm
Copy link
Contributor Author

Hi @mholt , this has been in production for a week with us. Although the number of ACME requests have not been dramatically reduced, we would be happy to have it, as it is still beneficial to avoid races across instances.

Copy link
Member

@mholt mholt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good, thanks! Will merge this in.

Am working on the "stateless" solving next.

@mholt mholt merged commit 7084df0 into caddyserver:master Sep 3, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants