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

New Issue

warner · 2012-09-30T19:01:36Z

warner commented

2012-09-30 19:01:36 +00:00

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.

warner added the

labels 2012-09-30 19:01:36 +00:00

warner added this to the soon milestone 2012-09-30 19:01:36 +00:00

tahoe-lafs modified the milestone from soon to 1.11.0

2012-09-30 22:15:43 +00:00

davidsarah commented

2012-10-01 22:55:12 +00:00

New leasedb design doc: https://github.com/daira/tahoe-lafs/blob/666-accounting/docs/specifications/leasedb.rst

New leasedb design doc: <https://github.com/daira/tahoe-lafs/blob/666-accounting/docs/specifications/leasedb.rst>

davidsarah commented

2012-11-18 01:59:56 +00:00

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

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

davidsarah commented

2012-11-18 06:52:44 +00:00

Done. :-)

zooko commented

2012-11-27 16:28:51 +00:00

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.

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.

zooko commented

2012-11-27 16:29:18 +00:00

Attachment merge-some-test-rejs.diff.txt (19230 bytes) added

**Attachment** merge-some-test-rejs.diff.txt (19230 bytes) added

merge-some-test-rejs.diff.txt

19 KiB

davidsarah commented

2012-11-27 22:15:36 +00:00

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.)

Replying to [zooko](/tahoe-lafs/trac-2024-07-25/issues/1818#issuecomment-89658): > 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.)

davidsarah commented

2013-03-01 01:34:47 +00:00

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.

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.

zooko commented

2013-05-30 15:54:25 +00:00

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

daira commented

2013-05-30 16:01:22 +00:00

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

The patches relating to leasedb are roughly those up to and including [86a6bbd3](https://github.com/LeastAuthority/tahoe-lafs/commit/86a6bbd3872b8fb9109e946c12284ee623851859). (There might be a few minor fixes after that.)

tahoe-lafs modified the milestone from 1.11.0 to 1.12.0

2013-07-22 20:52:54 +00:00

warner commented

2016-03-22 05:02:25 +00:00

Milestone renamed

warner modified the milestone from 1.12.0 to 1.13.0

2016-03-22 05:02:25 +00:00

warner commented

2016-06-28 18:17:14 +00:00

renaming milestone

warner modified the milestone from 1.13.0 to 1.14.0

2016-06-28 18:17:14 +00:00

exarkun commented

2017-08-08 13:08:19 +00:00

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.

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.

zooko was unassigned by exarkun

2017-08-08 13:08:19 +00:00

warner was assigned by exarkun

2017-08-08 13:08:19 +00:00

exarkun commented

2017-08-10 12:39:00 +00:00

Closing. See ticket:2237.

exarkun added the

wontfix

label 2017-08-10 12:39:00 +00:00

exarkun closed this issue

2017-08-10 12:39:00 +00:00