Opened at 2021-08-18T15:13:18Z
Last modified at 2021-09-01T14:44:46Z
#3773 closed task
Merge adding lease with renewing lease (for immutables) — at Initial Version
Reported by: | itamarst | Owned by: | itamarst |
---|---|---|---|
Priority: | normal | Milestone: | HTTP Storage Protocol |
Component: | unknown | Version: | n/a |
Keywords: | Cc: | ||
Launchpad Bug: |
Description
- From a client perspective, the goal is just to have a lease, whether it's a new lease or renewing a lease is irrelevant; renewing is essentially an optimization detail on server side.
- Cancelling leases is not actually supported.
- The way lease renewal secrets are generated is likely to change in the switchover from foolscap to HTTP.
- Internally, the server _already_ treats renewing a lease as potentially adding a new one if renewal fails.
As such, the distinction between adding a lease and renewing a lease is unnecessary (presuming item 4 is actually correct!).
Therefore, we should merge the two into a single operation:
- Validate item 4 above, since it's critical requirement for the rest of this.
- In Foolscap client, switch to only using renewal code path.
- Make sure server creating new lease when renewal fails is well tested, so users don't lose data.
- On server-side, leave new lease Foolscap endpoint in place for backwards compat.
- In new GBS HTTP protocol spec, switch to a single end point for lease creation/renewal, and remove all references to lease cancellation secret.
Note: See
TracTickets for help on using
tickets.