Skip to content
Snippets Groups Projects

Proof-of-concept implementation of lease renewal and price calculation using tahoe's iteration.

Open Administrator requested to merge github/fork/tp-la/deep-traverse into main

Created by: tomprince

This is completely untested. It isn't obvious if the code can use the existing fakes, and I don't have setup to test live.

Also, the code is really messy, though it'd be straightforward to refactor.

(Incidentally, I think I may know how to hook into the operations stuff, if we want to present the price result that way, rather than waiting)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Created by: meejah

    The main trade-off to consider here I think is that doing a traversal like this will be 100% accurate -- but it will cost a whole bunch of round-trips to the storage servers (linear with the number of Snapshots) versus e.g. "estimating" based on the fact that the Snapshot cap and the metadata cap will each be less than 1MB.

    I put "estimate" in scare-quotes because we do know that a Snapshot and the metadata are both definitely less than 1MB (and if we ever come anywhere near that size, we could do something else at that point). For example, we could put in unit-tests etc in magic-folder that fail if either one is more than 500KB.

  • Administrator restored source branch github/fork/tp-la/deep-traverse

    restored source branch github/fork/tp-la/deep-traverse

Please register or sign in to reply
Loading