Opened at 2012-09-30T19:01:36Z

Closed at 2017-08-10T12:39:00Z

#1818 closed task (wontfix)

leasedb: track leases in a sqlite database, not inside shares

Reported by: warner Owned by: warner
Priority: normal Milestone: 1.14.0
Component: code-storage Version: 1.9.2
Keywords: leasedb leases accounting docs Cc: exarkun@…
Launchpad Bug:

Description

This is a subtask/dependency of #666 (Accounting). I'm splitting it off into a separate ticket to keep the original one focussed on the longer-term issues.

Attachments (1)

merge-some-test-rejs.diff.txt​ (18.8 KB) - added by zooko at 2012-11-27T16:29:18Z.

Download all attachments as: .zip

Change History (16)

comment:1 Changed at 2012-09-30T22:15:43Z by davidsarah

  • Keywords leasedb leases accounting added
  • Milestone changed from soon to 1.11.0
  • Status changed from new to assigned

comment:3 Changed at 2012-11-18T01:59:56Z by davidsarah

The current work on this ticket is at ​https://github.com/davidsarah/tahoe-lafs/commits/1818-leasedb.

Version 0, edited at 2012-11-18T01:59:56Z by davidsarah (next)

comment:4 Changed at 2012-11-18T06:52:44Z by davidsarah

  • Keywords review-needed added
  • Owner changed from davidsarah to zooko
  • Status changed from assigned to new

Done. :-)

comment:5 Changed at 2012-11-18T07:11:06Z by zooko

  • Status changed from new to assigned

comment:6 follow-up: Changed at 2012-11-27T16:28:51Z by zooko

Here is a patch against ​https://tahoe-lafs.org/~davidsarah/working.tar.lz which manually fixes the .rej's for all the test files except test_storage.py and test_crawler.py. This patch also does test/common.py. I didn't go out of my way to remove filepathification, if I could leave it in place more easily.

Changed at 2012-11-27T16:29:18Z by zooko

comment:7 in reply to: ↑ 6 Changed at 2012-11-27T22:15:36Z by davidsarah

Replying to zooko:

Here is a patch against ​https://tahoe-lafs.org/~davidsarah/working.tar.lz which manually fixes the .rej's for all the test files except test_storage.py and test_crawler.py.

OK, that's been applied to my local copy now. (It is more relevant to #1819.)

Last edited at 2012-11-27T22:16:07Z by davidsarah (previous) (diff)

comment:8 Changed at 2013-03-01T01:34:47Z by davidsarah

Note that the fix to #1921 is not on the 1818-leasedb branch, and in general I need to review all patches on 1909-cloud-openstack 1819-cloud-merge to see whether they are applicable to 1818-leasedb.

Last edited at 2013-11-21T22:46:53Z by daira (previous) (diff)

comment:9 Changed at 2013-05-30T15:54:25Z by zooko

Instead of reviewing this ticket, review #1819, the code of which supercedes this ticket's code.

comment:10 Changed at 2013-05-30T16:01:22Z by daira

The patches relating to leasedb are roughly those up to and including ​86a6bbd3. (There might be a few minor fixes after that.)

comment:11 Changed at 2013-07-22T20:52:54Z by daira

  • Milestone changed from 1.11.0 to 1.12.0

comment:12 Changed at 2016-03-22T05:02:25Z by warner

  • Milestone changed from 1.12.0 to 1.13.0

Milestone renamed

comment:13 Changed at 2016-06-28T18:17:14Z by warner

  • Milestone changed from 1.13.0 to 1.14.0

renaming milestone

comment:14 Changed at 2017-08-08T13:08:19Z by exarkun

  • Cc exarkun@… added
  • Owner changed from zooko to warner
  • Status changed from assigned to new

The existence of this as an open needs-review ticket is confusing.

I propose closing this ticket so it doesn't distract from the rest of the leasedb / accounting / cloud backend efforts.

comment:15 Changed at 2017-08-10T12:39:00Z by exarkun

  • Keywords review-needed removed
  • Resolution set to wontfix
  • Status changed from new to closed

Closing. See ticket:2237.

Note: See TracTickets for help on using tickets.