Use `add_lease` instead of `renew_lease` (!226) · Merge requests · Administrator / ZKAPAuthorizer

Merged Jean-Paul Calderone requested to merge 220.use_add_lease into main 3 years ago

Sep 15, 2021

spell the exception name right · be4d9cf9
Jean-Paul Calderone authored 3 years ago

be4d9cf9
Switch to using add_lease for lease maintenance · 17daf840
Jean-Paul Calderone authored 3 years ago

17daf840

Make sure that test_renewed is exercising the interesting code path · 553e3269

Jean-Paul Calderone authored 3 years ago

Previously Hypothesis spent a long while putting storage indexes on the
storage servers and creating a list of nodes with non-overlapping storage
indexes and then asserting that, out of no relevant shares, none had expired
leases.

Now all storage servers in the test will have all of the relevant shares and
so all should be visited and updated.

553e3269

Use `add_lease` instead of `renew_lease`

Merge request reports

Activity