Ticket #1678: 1678-incident-on-truncate-v3.darcs.patch

3 patches for repository https://tahoe-lafs.org/source/tahoe/ticket999-S3-backend:

Thu Feb 16 04:17:32 GMT 2012  david-sarah@jacaranda.org
  * Make S3 error tracebacks include the status code and response, and trigger an incident. fixes #1589

Thu Feb 16 18:48:08 GMT 2012  david-sarah@jacaranda.org
  * S3 backend: make truncated GET Bucket responses trigger an incident. Does not include tests. refs #1678

Mon Feb 20 17:57:57 GMT 2012  david-sarah@jacaranda.org
  * Test that a truncated bucket listing triggers an incident. refs #1678

New patches:

[Make S3 error tracebacks include the status code and response, and trigger an incident. fixes #1589
david-sarah@jacaranda.org**20120216041732
 Ignore-this: 8a9be026e9db19ae9325760aee139687
] {
hunk ./src/allmydata/storage/backends/s3/mock_s3.py 3
 
 from twisted.internet import defer
+from twisted.web.error import Error
 
 from zope.interface import implements
hunk ./src/allmydata/storage/backends/s3/mock_s3.py 6
-from allmydata.storage.backends.s3.s3_common import IS3Bucket
+from allmydata.storage.backends.s3.s3_common import IS3Bucket, S3BucketMixin
 from allmydata.util.time_format import iso_utc
 from allmydata.util import fileutil
 from allmydata.util.deferredutil import async_iterate
hunk ./src/allmydata/storage/backends/s3/mock_s3.py 23
 
 MAX_KEYS = 1000
 
-class MockS3Bucket(object):
+class MockS3Bucket(S3BucketMixin):
     implements(IS3Bucket)
     """
     I represent a mock S3 bucket that stores its data in the local filesystem,
hunk ./src/allmydata/storage/backends/s3/mock_s3.py 39
     def __repr__(self):
         return ("<%s at %r>" % (self.__class__.__name__, self._storagedir,))
 
-    def create(self):
+    def _create(self):
         return defer.execute(self._not_implemented)
 
hunk ./src/allmydata/storage/backends/s3/mock_s3.py 42
-    def delete(self):
+    def _delete(self):
         return defer.execute(self._not_implemented)
 
     def _iterate_dirs(self):
hunk ./src/allmydata/storage/backends/s3/mock_s3.py 56
                     shnumstr = sharefp.basename()
                     yield (sharefp, "%s/%s" % (sikey, shnumstr))
 
-    def list_all_objects(self):
+    def _list_all_objects(self):
         contents = []
         def _next_share(res):
             if res is None:
hunk ./src/allmydata/storage/backends/s3/mock_s3.py 78
         # This method is also called by tests.
         sharefp = self._storagedir.preauthChild(object_name)
         if must_exist and not sharefp.exists():
-            raise MockS3Error(404, "not found")
+            raise MockS3Error("", 404, "not found")
         return sharefp
 
hunk ./src/allmydata/storage/backends/s3/mock_s3.py 81
-    def put_object(self, object_name, data, content_type=None, metadata={}):
+    def _put_object(self, object_name, data, content_type, metadata):
         assert content_type is None, content_type
         assert metadata == {}, metadata
         sharefp = self._get_filepath(object_name)
hunk ./src/allmydata/storage/backends/s3/mock_s3.py 89
         sharefp.setContent(data)
         return defer.succeed(None)
 
-    def get_object(self, object_name):
+    def _get_object(self, object_name):
         return defer.succeed(self._get_filepath(object_name, must_exist=True).getContent())
 
hunk ./src/allmydata/storage/backends/s3/mock_s3.py 92
-    def head_object(self, object_name):
+    def _head_object(self, object_name):
         return defer.execute(self._not_implemented)
 
hunk ./src/allmydata/storage/backends/s3/mock_s3.py 95
-    def delete_object(self, object_name):
+    def _delete_object(self, object_name):
         self._get_filepath(object_name, must_exist=True).remove()
         return defer.succeed(None)
 
hunk ./src/allmydata/storage/backends/s3/mock_s3.py 102
     def _not_implemented(self):
         raise NotImplementedError
 
+    # methods that use error handling from S3BucketMixin
+
+    def create(self):
+        return self._do_aws(self._create)
+
+    def delete(self):
+        return self._do_aws(self._delete)
+
+    def list_all_objects(self):
+        return self._do_aws(self._list_all_objects)
+
+    def put_object(self, object_name, data, content_type=None, metadata={}):
+        return self._do_aws(self._put_object, object_name, data, content_type, metadata)
+
+    def get_object(self, object_name):
+        return self._do_aws(self._get_object, object_name)
+
+    def head_object(self, object_name):
+        return self._do_aws(self._head_object, object_name)
+
+    def delete_object(self, object_name):
+        return self._do_aws(self._delete_object, object_name)
+
 
hunk ./src/allmydata/storage/backends/s3/mock_s3.py 126
-class MockS3Error(Exception):
+class MockS3Error(Error):
     """
     A error class providing custom methods on S3 errors.
     """
hunk ./src/allmydata/storage/backends/s3/mock_s3.py 130
-    def __init__(self, error_code, error_message, request_id="", host_id=""):
-        Exception.__init__(self, "%r: %r" % (error_code, error_message))
-        self.error_code = error_code
-        self.error_message = error_message
+    def __init__(self, xml_bytes, status, message=None, response=None, request_id="", host_id=""):
+        Error.__init__(self, status, message, response)
+        self.original = xml_bytes
+        self.status = str(status)
+        self.message = str(message)
         self.request_id = request_id
         self.host_id = host_id
 
hunk ./src/allmydata/storage/backends/s3/mock_s3.py 139
     def get_error_code(self):
-        return self.error_code
+        return self.status
 
     def get_error_message(self):
hunk ./src/allmydata/storage/backends/s3/mock_s3.py 142
-        return self.error_message
+        return self.message
 
     def parse(self, xml_bytes=""):
         raise NotImplementedError
hunk ./src/allmydata/storage/backends/s3/s3_bucket.py 2
 
-from twisted.internet.defer import maybeDeferred
-
 from zope.interface import implements
hunk ./src/allmydata/storage/backends/s3/s3_bucket.py 3
-from allmydata.storage.backends.s3.s3_common import IS3Bucket
+from allmydata.storage.backends.s3.s3_common import IS3Bucket, S3BucketMixin
 
 
hunk ./src/allmydata/storage/backends/s3/s3_bucket.py 6
-class S3Bucket(object):
+class S3Bucket(S3BucketMixin):
     implements(IS3Bucket)
     """
     I represent a real S3 bucket, accessed using the txaws library.
hunk ./src/allmydata/storage/backends/s3/s3_bucket.py 45
         return ("<%s %r>" % (self.__class__.__name__, self.bucketname,))
 
     def create(self):
-        return maybeDeferred(self.client.create, self.bucketname)
+        return self._do_aws(self.client.create, self.bucketname)
 
     def delete(self):
hunk ./src/allmydata/storage/backends/s3/s3_bucket.py 48
-        return maybeDeferred(self.client.delete, self.bucketname)
+        return self._do_aws(self.client.delete, self.bucketname)
 
     # We want to be able to do prefix queries, but txaws 0.2 doesn't implement that.
     def list_all_objects(self):
hunk ./src/allmydata/storage/backends/s3/s3_bucket.py 52
-        return maybeDeferred(self.client.get_bucket, self.bucketname)
+        return self._do_aws(self.client.get_bucket, self.bucketname)
 
     def put_object(self, object_name, data, content_type=None, metadata={}):
hunk ./src/allmydata/storage/backends/s3/s3_bucket.py 55
-        return maybeDeferred(self.client.put_object, self.bucketname,
-                             object_name, data, content_type, metadata)
+        return self._do_aws(self.client.put_object, self.bucketname,
+                            object_name, data, content_type, metadata)
 
     def get_object(self, object_name):
hunk ./src/allmydata/storage/backends/s3/s3_bucket.py 59
-        return maybeDeferred(self.client.get_object, self.bucketname, object_name)
+        return self._do_aws(self.client.get_object, self.bucketname, object_name)
 
     def head_object(self, object_name):
hunk ./src/allmydata/storage/backends/s3/s3_bucket.py 62
-        return maybeDeferred(self.client.head_object, self.bucketname, object_name)
+        return self._do_aws(self.client.head_object, self.bucketname, object_name)
 
     def delete_object(self, object_name):
hunk ./src/allmydata/storage/backends/s3/s3_bucket.py 65
-        return maybeDeferred(self.client.delete_object, self.bucketname, object_name)
+        return self._do_aws(self.client.delete_object, self.bucketname, object_name)
 
     def put_policy(self, policy):
         """
hunk ./src/allmydata/storage/backends/s3/s3_bucket.py 74
         query = self.client.query_factory(
             action='PUT', creds=self.client.creds, endpoint=self.client.endpoint,
             bucket=self.bucketname, object_name='?policy', data=policy)
-        return maybeDeferred(query.submit)
+        return self._do_aws(query.submit)
 
     def get_policy(self):
         query = self.client.query_factory(
hunk ./src/allmydata/storage/backends/s3/s3_bucket.py 80
             action='GET', creds=self.client.creds, endpoint=self.client.endpoint,
             bucket=self.bucketname, object_name='?policy')
-        return maybeDeferred(query.submit)
+        return self._do_aws(query.submit)
 
     def delete_policy(self):
         query = self.client.query_factory(
hunk ./src/allmydata/storage/backends/s3/s3_bucket.py 86
             action='DELETE', creds=self.client.creds, endpoint=self.client.endpoint,
             bucket=self.bucketname, object_name='?policy')
-        return maybeDeferred(query.submit)
+        return self._do_aws(query.submit)
hunk ./src/allmydata/storage/backends/s3/s3_common.py 2
 
-import re
+import re, sys
+
+from twisted.internet import defer
+from foolscap.logging import log
 
 from zope.interface import Interface
 
hunk ./src/allmydata/storage/backends/s3/s3_common.py 75
         Delete an object from this bucket.
         Once deleted, there is no method to restore or undelete an object.
         """
+
+
+class TahoeS3Error(Exception):
+    pass
+
+
+class S3BucketMixin:
+    def _do_aws(self, operation, *args):
+        d = defer.maybeDeferred(operation, *args)
+        def _handle_error(f):
+            f.trap(self.S3Error)
+            try:
+                raise f.value
+            except self.S3Error:
+                typ, val, tb = sys.exc_info()
+                xml = f.value.original
+                fargs = f.value.args
+                err = TahoeS3Error(xml, *fargs)
+                log.msg(format="%(err)s", err=err, level=log.WEIRD)
+                raise err.__class__, err, tb
+        d.addErrback(_handle_error)
+        return d
hunk ./src/allmydata/test/test_storage.py 9
 from twisted.internet import defer
 from twisted.application import service
 from twisted.python.filepath import FilePath
+from twisted.python.failure import Failure
 from foolscap.api import fireEventually
hunk ./src/allmydata/test/test_storage.py 11
+from foolscap.logging.log import WEIRD
 
 from allmydata import interfaces
 from allmydata.util import fileutil, hashutil, base32, pollmixin, time_format
hunk ./src/allmydata/test/test_storage.py 21
 from allmydata.storage.backends.disk.disk_backend import DiskBackend
 from allmydata.storage.backends.disk.immutable import load_immutable_disk_share, create_immutable_disk_share
 from allmydata.storage.backends.disk.mutable import MutableDiskShare
+from allmydata.storage.backends.s3 import s3_common
 from allmydata.storage.backends.s3.s3_backend import S3Backend
hunk ./src/allmydata/test/test_storage.py 23
-from allmydata.storage.backends.s3.mock_s3 import MockS3Bucket
+from allmydata.storage.backends.s3.mock_s3 import MockS3Bucket, MockS3Error
 from allmydata.storage.bucket import BucketWriter, BucketReader
 from allmydata.storage.common import DataTooLargeError, UnknownContainerVersionError, \
      UnknownMutableContainerVersionError, UnknownImmutableContainerVersionError
hunk ./src/allmydata/test/test_storage.py 1338
 
 
 class ServerWithS3Backend(ServerTest, CreateS3Backend, unittest.TestCase):
-    pass
+    def test_s3_errors(self):
+        def call_put_object(self, object_name, data, content_type=None, metadata={}):
+            return defer.fail(MockS3Error("XML", 500, "Internal error", "response"))
+        self.patch(MockS3Bucket, '_put_object', call_put_object)
+
+        s = {"level": 0}
+        def call_log_msg(*args, **kwargs):
+            s["level"] = max(s["level"], kwargs["level"])
+        self.patch(s3_common.log, 'msg', call_log_msg)
+
+        ss = self.create("test_s3_errors")
+
+        d = self.allocate(ss, "vid", [0], 75)
+        d.addCallback(lambda (already, writers): for_items(self._write_and_close, writers))
+        # shouldFail would check repr(res.value.args[0]) which is not what we want
+        def done(res):
+            if isinstance(res, Failure):
+                res.trap(s3_common.TahoeS3Error)
+                self.failUnlessIn("('XML', 500, 'Internal error', 'response')", str(res.value))
+                self.failUnless(s["level"] >= WEIRD, s["level"])
+            else:
+                self.fail("was supposed to raise TahoeS3Error, not get %r" % (res,))
+        d.addBoth(done)
+        return d
 
 
 class ServerWithDiskBackend(ServerTest, CreateDiskBackend, unittest.TestCase):
}
[S3 backend: make truncated GET Bucket responses trigger an incident. Does not include tests. refs #1678
david-sarah@jacaranda.org**20120216184808
 Ignore-this: 932e3ae0279cb335d5e337823db89ade
] hunk ./src/allmydata/storage/backends/s3/s3_common.py 26
     #d = self._s3bucket.list_objects('shares/%s/' % (prefix,), marker)
     d = s3bucket.list_all_objects()
     def _filter(res):
+        if res.is_truncated:
+            log.msg(format="truncated get_bucket response (%(num)d objects)", num=len(res.contents), level=log.WEIRD)
         res.contents = [item for item in res.contents if item.key.startswith(prefix)]
         return res
     d.addCallback(_filter)
[Test that a truncated bucket listing triggers an incident. refs #1678
david-sarah@jacaranda.org**20120220175757
 Ignore-this: f78d11c6803bb2b488460d05ced22ae8
] {
hunk ./src/allmydata/test/test_storage.py 23
 from allmydata.storage.backends.disk.mutable import MutableDiskShare
 from allmydata.storage.backends.s3 import s3_common
 from allmydata.storage.backends.s3.s3_backend import S3Backend
-from allmydata.storage.backends.s3.mock_s3 import MockS3Bucket, MockS3Error
+from allmydata.storage.backends.s3.mock_s3 import MockS3Bucket, MockS3Error, BucketListing
 from allmydata.storage.bucket import BucketWriter, BucketReader
 from allmydata.storage.common import DataTooLargeError, UnknownContainerVersionError, \
      UnknownMutableContainerVersionError, UnknownImmutableContainerVersionError
hunk ./src/allmydata/test/test_storage.py 378
             f2.close()
 
 
+class S3Common(unittest.TestCase):
+    def test_list_objects_truncated(self):
+        # A truncated bucket listing should cause an incident.
+
+        class MockBucket(object):
+            def list_all_objects(self):
+                return defer.succeed(BucketListing("bucket", "", "/", 0, True, contents=[]))
+
+        s = {"level": 0}
+        def call_log_msg(*args, **kwargs):
+            s["level"] = max(s["level"], kwargs["level"])
+        self.patch(s3_common.log, 'msg', call_log_msg)
+
+        d = s3_common.list_objects(MockBucket(), "")
+        d.addCallback(lambda ign: self.failUnless(s["level"] >= WEIRD, s["level"]))
+        return d
+
+
 class ServerMixin:
     def setUp(self):
         self.sparent = LoggingServiceParent()
}

Context:

[Correct some cases where it was incorrectly assumed that FilePath.basename always returns a str, rather than unicode.
david-sarah@jacaranda.org**20111120044556
 Ignore-this: d0dc8bafe4820d7a155100962a98f3e0
] 
[Changes to support desktop products (allowing to use less powerful credentials on the storage server).
david-sarah@jacaranda.org**20111103222422
 Ignore-this: 9702bdf288f58bc9130dfe59c2b04e4b
] 
[update NEWS for the 1.9.0 release
warner@lothar.com**20111031052252
 Ignore-this: df9d16c29e54803869e7172c90f96972
] 
[more docs updates
warner@lothar.com**20111031050831
 Ignore-this: af8c82696b1b1c712238dd631342f8c7
] 
[improve relnotes
warner@lothar.com**20111031033704
 Ignore-this: 904c768314dfcb34581b9d585df8dc13
] 
[update relnotes, rotate known_issues, for 1.9
Brian Warner <warner@lothar.com>**20111031032439
 Ignore-this: e2f63298b7665fd7d7460e2ebc70699d
] 
[add user-oriented notes to NEWS and mutable.rst about SDMF-vs-MDMF
Brian Warner <warner@lothar.com>**20111031030512
 Ignore-this: 78e5518b8f8c5d529329f291537c9b07
] 
[show-tool-versions.py: remove setuptools_trial, unused
Brian Warner <warner@lothar.com>**20111031004742
 Ignore-this: c1dc9dd7138f0a59c633340520f6d783
] 
[Makefile/upload-tarballs: remove bash-ism in shell conditional
Brian Warner <warner@lothar.com>**20111031004721
 Ignore-this: 9ffe81a2cfafac5a53fc0503e0141c0d
 
 The "[" command is defined to accept "=" as an is-equal test. Bash extends
 this to accept "==" too, but normal /bin/sh does not. I think this command
 was developed on a box where /bin/sh is bash, but on standard ubuntu boxes,
 /bin/sh is a smaller+faster non-Bash shell, and this gave "[: 1: X:
 unexpected operator" errors.
] 
[update project-home URLs: /trac/tahoe/ was replaced by /trac/tahoe-lafs/
Brian Warner <warner@lothar.com>**20111030191428
 Ignore-this: cb37679ad62845787064d94587caa48e
] 
[quickstart.rst: remove trailing whitespace
Brian Warner <warner@lothar.com>**20111030174411
 Ignore-this: 4c2738c023916de8cd3ad77bf9a9dcc7
] 
[quickstart.rst: update the release URL in preparation for 1.9
Brian Warner <warner@lothar.com>**20111030173318
 Ignore-this: f00af87789bbbdc4c520c486367cebd6
] 
[and about.rst
warner@lothar.com**20111029195816
 Ignore-this: 1cddc3c3cf57cf7f588a660ba0b065c1
] 
[update README for HTTPS too
warner@lothar.com**20111029194240
 Ignore-this: 37c5f34927ef5aaa46e111563fa5d14a
] 
[small changes to test migrated trac and posthook
warner@lothar.com**20111029191807
 Ignore-this: 8110a6c1b4ea318528e8023ce1f75fae
] 
[more http->https changes
warner@lothar.com**20111029190905
 Ignore-this: 9d0e6ed5f24858f7742f584da50cadc0
] 
[tahoe-lafs.org is now HTTPS-always. Update most URLs.
warner@lothar.com**20111029183946
 Ignore-this: 1d490ee1983b1ef30045a545250f18a8
] 
[undo the effects of a patch I hadn't intended to commit, named "debugprint the values of blocks and hashes thereof; make the test data and the seg size small in order to make the debugprints easy to look at"
zooko@zooko.com**20111028220349
 Ignore-this: 6390dd943f87d4340368b1e174cba6be
 
 rolling back:
 
 Thu Sep 29 23:46:28 MDT 2011  zooko@zooko.com
   * debugprint the values of blocks and hashes thereof; make the test data and the seg size small in order to make the debugprints easy to look at
 
     M ./src/allmydata/mutable/publish.py -1 +2
     M ./src/allmydata/mutable/retrieve.py +3
     M ./src/allmydata/test/test_mutable.py -2 +2
] 
[docs/about.rst: correct the description of availability to take into account that shares may not be stored on distinct servers.
david-sarah@jacaranda.org**20111025132550
 Ignore-this: 975fde31c73512c4c0f278182d576861
] 
[debugprint the values of blocks and hashes thereof; make the test data and the seg size small in order to make the debugprints easy to look at
zooko@zooko.com**20110930054628
 Ignore-this: bcfedc06aeedb090dfb02440f6e6c3bc
] 
[S3 backend: support DevPay. Includes tests of config option, but interoperability with S3 has not been tested yet. refs #999
david-sarah@jacaranda.org**20111027215314
 Ignore-this: 554870770ce7ce393d8cea195d365ab9
] 
[docs/backends/S3.rst: document the requirement for the storage server to have the correct time to within 15 minutes. refs #999
david-sarah@jacaranda.org**20111025100827
 Ignore-this: dee22e41bee85abe107c4a8457321ce6
] 
[S3 backend: the s3.region option is unnecessary; it is only used for EC2 endpoints, and we only need an S3 one. Also simplify wording in S3.rst. refs #999
david-sarah@jacaranda.org**20111023235033
 Ignore-this: b28b72e9047f996b7774e9279daadd9
] 
[S3 backend: remove support for [storage]readonly option. refs #999, #1568
david-sarah@jacaranda.org**20111022045644
 Ignore-this: 546eee72a9c708a7f101178fcc044261
] 
[mock_s3.py: remove bucketname argument to MockS3Bucket constructor, since it is not needed. refs #999
david-sarah@jacaranda.org**20111022045341
 Ignore-this: 872d91c8475e1fbbccb6fd580aad8c4e
] 
[test_system.py: check that there is no error output from invocations of 'tahoe debug'. refs #999
david-sarah@jacaranda.org**20111021044102
 Ignore-this: 8dfb3a668bfebdcbf7f3f295d345fb98
] 
[scripts/debug.py: in catalog-shares, gracefully handle the case where a share has no leases (for example because it is an S3 share). refs #999
david-sarah@jacaranda.org**20111021034138
 Ignore-this: 28ce846977095e9bfcb55364e2388e70
] 
[test_system.py: fix SystemWithS3Backend.test_mutable by only requiring the line specifying which nodeid the lease secrets are for when the node has a disk backend. refs #999
david-sarah@jacaranda.org**20111021032106
 Ignore-this: dc5e6e6f9f7d5a898408f2070afca4a2
] 
[test_system.py: ensure that subclasses of SystemTest use different test directories. refs #999
david-sarah@jacaranda.org**20111021014630
 Ignore-this: 938ab8addc92dcd14fccc179ae095feb
] 
[test_system.py: make checks in _test_runner more picky about field names to avoid accidental suffix matches. refs #999
david-sarah@jacaranda.org**20111021014527
 Ignore-this: f46c364efd86b0b40f6448bc0a95c495
] 
[test_system.py: rename ServerTestWith*Backend to ServerWith*Backend, for consistency with tst_storage.py. refs #999
david-sarah@jacaranda.org**20111021011001
 Ignore-this: 47435b4ef02d4267d528b3a99bb2ec07
] 
[test_system.py: fix a typo. refs #999
david-sarah@jacaranda.org**20111021010846
 Ignore-this: f8daf272b0266d6df6a13f88f2dd67
] 
[test_system.py: enable system tests to run against S3 backend as well as disk backend. refs #999
david-sarah@jacaranda.org**20111021001632
 Ignore-this: bbe5c7479a05db40165800211c68ce54
] 
[Add a '[storage]backend = mock_s3' option for use by tests. Move mock_s3.py to src/allmydataa/storage/backends/s3 since it is now imported by non-test code. refs #999
david-sarah@jacaranda.org**20111021001518
 Ignore-this: 8e0643a81a55b319b0f757f7da9acfd6
] 
[mutable/retrieve: don't write() after we've been pauseProducer'ed
Brian Warner <warner@lothar.com>**20111017002400
 Ignore-this: 417880ec53285c4887f8080e1ddeedc8
 
 This fixes a test failure found against current Twisted trunk in
 test_mutable.Filenode.test_retrieve_producer_mdmf (when it uses
 PausingAndStoppingConsumer). There must be some sort of race: I could
 make it fail against Twisted-11.0 if I just increased the 0.5s delay in
 test_download.PausingAndStoppingConsumer to about 0.6s, and could make
 Twisted-trunk pass by reducing it to about 0.3s .
 
 I fixed the test (as opposed to the bug) by replacing the delay with a
 simple reliable eventually(), and adding extra asserts to fail the test
 if the consumer's write() method is called while the producer is
 supposed to be paused
 
 The bug itself was that mutable.retrieve.Retrieve wasn't checking the
 "stopped" flag after resuming from a pause, and thus delivered one
 segment to a consumer that wasn't expecting it. I split out
 stopped-flag-checking to separate function, which is now called
 immediately after _check_for_paused(). I also cleaned up some Deferred
 usage and whitespace.
] 
[remove interpreter shbang lines from non-executables
Brian Warner <warner@lothar.com>**20111014172301
 Ignore-this: a1ad931ed2e4379fed9bf480382ad801
 
 thanks to Greg Troxel for the catch
] 
[TAG allmydata-tahoe-1.9.0b1
warner@lothar.com**20111014055532
 Ignore-this: f00238ed3d8d1f5e15b0262c4373a3c3
] 
[NEWS: mention --format, bring up-to-date
warner@lothar.com**20111014055500
 Ignore-this: 6c70a87cd8894cee954fd2deeada61f6
] 
[CLI: don't deprecate --mutable, small docs fixes. refs #1561
Brian Warner <warner@lothar.com>**20111014040002
 Ignore-this: 6133c130e7060fc4240194bc08ed9c9d
 
 Also don't accept 'tahoe mkdir --format=chk'.
] 
[add --format= to 'tahoe put'/'mkdir', remove --mutable-type. Closes #1561
Brian Warner <warner@lothar.com>**20111014031500
 Ignore-this: ac38ac429847942e6383e7374bf0e1bf
] 
[web/filenode.py: rely on Request.notifyFinish. Closes #1366.
Brian Warner <warner@lothar.com>**20111013201219
 Ignore-this: cf7677bf15cb8e469ec16c3372fdfa35
 
 This is safe now that tahoe depends upon Twisted>=10.1, since notifyFinish
 first appeared in Twisted-9.0
] 
[docs: fix several imprecise or inaccurate values in performance.rst
zooko@zooko.com**20110508124228
 Ignore-this: f1ecc5cb32eebec9760c8fc437799eb4
 add cpu values for each operation
 sort the list of values into the same order in each operation
 refs #1398
] 
[oops, missed a test failure
Brian Warner <warner@lothar.com>**20111013163713
 Ignore-this: d8cb188d8dd664e335f19b9fa342da4a
] 
[misc mutable-type fixes:
warner@lothar.com**20111013163229
 Ignore-this: ab62dc2f27aa1f793e7bd02e360ee471
 
 * fix tahoe.cfg control of default mutable type
 * tolerate arbitrary case in [client]mutable.format value
 * small docs improvements
 * use get_mutable_type() as a format-is-mutable predicate
 * tighten up error message
] 
[webapi: use all-caps "SDMF"/"MDMF" acronyms in t=json response
warner@lothar.com**20111013163143
 Ignore-this: 945eaa5ce7f108793b0bb6cae0239965
 
 docs: upcase examples of t=json output and format= input
] 
[webapi.rst: fix whitespace (detabify) t=json examples
warner@lothar.com**20111013163056
 Ignore-this: c095687413876507c5cc46864459c054
] 
[webapi: handle format=, remove mutable-type=
warner@lothar.com**20111013162951
 Ignore-this: de7d9c5516385d002dbc21f31c23204c
 
 * fix CLI commands (put, mkdir) to send format=, not mutable-type=
 * fix tests
 * test_cli: fix tests that observe t=json output, don't ignore failures in
   'tahoe put'
 * fix handling of version= to make it easier to use the default
 * interpret ?mutable=true&format=MDMF as MDMF, not SDMF
] 
[docs/frontends/webapi.rst: document the format argument
kevan@isnotajoke.com**20111010025529
 Ignore-this: 2a7b8d711dc369bd9a23e2853824cfb0
] 
[Tests for ref #1547
david-sarah@jacaranda.org**20111002035316
 Ignore-this: 933f2b6ff148523f40475fe2d2578170
] 
[Change the file upload forms on directory and welcome pages to use a 3-way radio button to select immutable, SDMF, or MDMF. Add '(experimental)' to the label for creating an MDMF directory. Also improve the spacing of form elements. refs #1547
david-sarah@jacaranda.org**20111002034503
 Ignore-this: 46a8b966fddc8ccaa7e70bffbd68b52f
] 
[test_web.py: minor cleanups, mainly to make the first argument to shouldFail tests consistent
david-sarah@jacaranda.org**20111002040332
 Ignore-this: 234ba793f78f112717e02755e1fa81b5
] 
[Tests for ref #1552
david-sarah@jacaranda.org**20111002040036
 Ignore-this: abdc5c39d90ea7f314834fff7ecd6784
] 
[test_storage.py: the part of test_remove that checks non-existence of the share directory after deleting a share, is only applicable to the disk backend; but, we can check that the shareset has no overhead at that point. refs #999
david-sarah@jacaranda.org**20111020173349
 Ignore-this: 8c7b93afeabf090d2615db81a4556fd6
] 
[test_storage.py: reenable MutableServer.test_container_size for the S3 backend. refs #999
david-sarah@jacaranda.org**20111020115355
 Ignore-this: 33a2e41ec2d3b917dc5be897ce50c152
] 
[Disk backend: make sure that the size limit is checked before writing. Also, the size limit is on the data length, not the container size. refs #999
david-sarah@jacaranda.org**20111020114919
 Ignore-this: 32a278af8e2a6464e193b635c7f17ff4
] 
[S3 backend: the mutable size limit should be on the data length, not the container size. Also simplify by removing _check_size_limit. refs #999
david-sarah@jacaranda.org**20111020114529
 Ignore-this: ef64c7f68969ff550f81af2f17e5b14a
] 
[S3 backend: new_length argument to MutableS3Share.writev should only be able to truncate the share (after applying writes), not extend it. refs #999
david-sarah@jacaranda.org**20111020114356
 Ignore-this: 3da64b01af2ea86aa1b73d9e3af65024
] 
[S3 backend: make precondition failures show more information. refs #999
david-sarah@jacaranda.org**20111020111611
 Ignore-this: bb2fc7ad6962e2a4f394eaf5cde1795d
] 
[S3 backend: make sure that the container size limit is checked before writing. refs #999
david-sarah@jacaranda.org**20111020111519
 Ignore-this: add18e2273a84d75bd491a13c6f6451a
] 
[test_storage.py: reduce duplicated code by factoring 'create' methods into CreateS3Backend and CreateDiskBackend classes. refs #999
david-sarah@jacaranda.org**20111020111350
 Ignore-this: 1362efb0293bc7e62fafd9f1227c47d9
] 
[S3 backend: finish implementation of mutable shares. refs #999
david-sarah@jacaranda.org**20111020030522
 Ignore-this: 53bf3fdc4d243069880b39295601af06
] 
[test_storage.py: move the test_container_size test to MutableServerWithDiskBackend for now, because it tries to create a very large container which will wedge your machine. refs #999
david-sarah@jacaranda.org**20111020030447
 Ignore-this: 62f2b5df800bf1ab631853e8ad4a4267
] 
[Disk backend: fix incorrect arguments in a call to create_mutable_disk_share. refs #999
david-sarah@jacaranda.org**20111020030301
 Ignore-this: ec4496f4a0cbb32eb5f9d6d32cc0221d
] 
[storage/backends/disk/mutable.py: correct a typo. refs #999
david-sarah@jacaranda.org**20111020012427
 Ignore-this: af896e2496c15e06d1eec6c0af49b695
] 
[Enable mutable tests for S3 backend (they all fail, as expected). refs #999
david-sarah@jacaranda.org**20111019061735
 Ignore-this: f058dba13bd4f1d4273739d2823605f
] 
[S3 backend: remove max_space option. refs #999
david-sarah@jacaranda.org**20111018224057
 Ignore-this: 6fc56d5ea43dae4ac9604a7cf15e7ce6
] 
[test_storage.py, test_crawler.py: change 'bucket' terminology to 'shareset' where appropriate. refs #999
david-sarah@jacaranda.org**20111018183242
 Ignore-this: b8cf782836d1558fc99ff13fa08b2b9f
] 
[Add some __repr__ methods. refs #999
david-sarah@jacaranda.org**20111018064409
 Ignore-this: 9b901ee8aaaa9f9895f6a3b4e5f41a21
] 
[Fix race conditions in crawler tests. (storage.LeaseCrawler.test_unpredictable_future may still be racy.) refs #999
david-sarah@jacaranda.org**20111018064303
 Ignore-this: 58b2791250d14f7e8a6284190d7872e8
] 
[Allow crawlers and storage servers to use a deterministic clock, for testing. We do not yet take advantage of this in tests. refs #999
david-sarah@jacaranda.org**20111018063926
 Ignore-this: 171bf7275a978a93108b0835d06834ea
] 
[Change IShareSet.get_shares[_synchronous] to return a pair (list of share objects, set of corrupt shnums). This is necessary to allow crawlers to record but skip over corrupt shares. This patch also changes the behaviour of storage servers to ignore corrupt shares on read, which may or may not be what we want. Note that the S3 backend does not yet report corrupt shares. refs #999
david-sarah@jacaranda.org**20111018063423
 Ignore-this: 35abee65334aa3a92471266f5789f452
] 
[test_storage.py: cleanup to style of test_limited_history to match other tests. refs #999
david-sarah@jacaranda.org**20111016044311
 Ignore-this: 3024764ffb419917eeeb3ecd554fb421
] 
[Change accesses of ._sharehomedir on a disk shareset to _get_sharedir(). refs #999
david-sarah@jacaranda.org**20111016044229
 Ignore-this: 5b2b9e11f56f0af0588c69ca930f60fd
] 
[Disk backend: make sure that disk shares with a storageindex of None (as sometimes created by test code) can be printed using __repr__. refs #999
david-sarah@jacaranda.org**20111016035131
 Ignore-this: a811b53c20fdde5ad60471c5e4961a24
] 
[scripts/debug.py: fix stale code in describe_share that had not been updated for changes in share interfaces. refs #999
david-sarah@jacaranda.org**20111016034913
 Ignore-this: 7f2469392b9e6fc64c354ce5a5568a68
] 
[test_storage.py: fix a bug in _backdate_leases (it was returning too early). refs #999
david-sarah@jacaranda.org**20111016014038
 Ignore-this: 658cf2e2c0dc87032988f1d4db62f267
] 
[Undo partial asyncification of crawlers, and enable crawlers only for the disk backend. refs #999
david-sarah@jacaranda.org**20111014061902
 Ignore-this: ce7303610878b1b051cf54604796bdde
] 
[test_storage.py: fix two bugs in test_no_st_blocks -- the _cleanup function was being called too early, and we needed to treat directories as using no space in order for the configured-sharebytes == configured-diskbytes check to be correct. refs #999
david-sarah@jacaranda.org**20111014025840
 Ignore-this: a20434e28eda165bed2021f0dafa676c
] 
[test_storage.py: print more info when checks fail. refs #999
david-sarah@jacaranda.org**20111013234159
 Ignore-this: 2989f30c24362ee6a80a7f8f3d5aad9
] 
[test_storage.py: remove some redundant coercions to bool. refs #999
david-sarah@jacaranda.org**20111013233520
 Ignore-this: 3fa9baaf7e41831a24b8cfa0ef5ec5e4
] 
[test_storage: in test_no_st_blocks, print the rec 'dict' if checking one of its fields fails. refs #999
david-sarah@jacaranda.org**20111013232802
 Ignore-this: cf18a119d80f11b1ba8681c4285c0198
] 
[test_storage: fix some typos introduced when asyncifying test_immutable_leases. refs #999
david-sarah@jacaranda.org**20111013232618
 Ignore-this: 28a5e1377d7198191d5771e09826af5b
] 
[test_storage: rename the two test_leases methods to ServerTest.test_immutable_leases and MutableServer.test_mutable_leases. refs #999
david-sarah@jacaranda.org**20111013232538
 Ignore-this: 7a3ccfd237db7a3c5053fe90c3bed1f3
] 
[test_storage.py: fix a typo (d vs d2) in test_remove_incoming. refs #999
david-sarah@jacaranda.org**20111013222822
 Ignore-this: 71ad69489698865748cd32bc2c8b2fc1
] 
[docs/backends/S3.rst: note that storage servers should use different buckets. refs #999
david-sarah@jacaranda.org**20111013050647
 Ignore-this: fec8d3bf114bbcf20165a5850aa25aac
] 
[S3 backend: keep track of incoming shares, so that the storage server can avoid creating BucketWriters for shnums that have an incoming share. refs #999
david-sarah@jacaranda.org**20111013035040
 Ignore-this: f1c33357553d68748f970c0c9e19d538
] 
[test_storage.py: test_read_old_share and test_write_and_read_share should only expect to be able to read input share data. refs #999
david-sarah@jacaranda.org**20111013032825
 Ignore-this: bec4a26f13f105cc84261f2e1b028302
] 
[misc/check-interfaces.py: print a warning if a .pyc or .pyo file exists without a corresponding .py file.
david-sarah@jacaranda.org**20111012233609
 Ignore-this: 35f04939360c6d3b1e8e0c2e9e712d80
] 
[storage/backends/base.py: allow readv to work for both mutable and immutable shares. refs #999
david-sarah@jacaranda.org**20111012232802
 Ignore-this: a266704981739e7c1217f352aee153fe
] 
[S3 backend: correct list_objects to list_all_objects in IS3Bucket. refs #999
david-sarah@jacaranda.org**20111012232713
 Ignore-this: 7f9dc7946e9866f71e16f3a595f0218e
] 
[Null backend: make NullShareSet inherit from ShareSet, which should implement readv correctly. Remove its implementation of testv_and_readv_and_writev since the one from ShareSet should work (if it doesn't that would be a separate bug). refs #999
david-sarah@jacaranda.org**20111012232600
 Ignore-this: 404757cc6f7e29c2b927258af31d55ce
] 
[Remove test_backends.py, since all its tests are now redundant with tests in test_storage.py or test_client.py. refs #999
david-sarah@jacaranda.org**20111012232316
 Ignore-this: f601a8165058773075ce80d96586b0d9
] 
[test_storage.py: add test_write_and_read_share and test_read_old_share originally from test_backends.py. refs #999
david-sarah@jacaranda.org**20111012232124
 Ignore-this: 805cd42094d3948ffdf957f44e0d146d
] 
[test_download.py: fix and reenable Corruption.test_each_byte. Add a comment noting that catalog_detection = True has bitrotted. refs #999
david-sarah@jacaranda.org**20111012041219
 Ignore-this: b9fa9ce7406811cd5a9d4a49666b1ab0
] 
[no_network.py: fix delete_all_shares. refs #999
david-sarah@jacaranda.org**20111012033458
 Ignore-this: bfe9225562454f153a921277b43ac848
] 
[S3 backend: fix corruption advisories and listing of shares for mock S3 bucket. refs #999
david-sarah@jacaranda.org**20111012033443
 Ignore-this: 9d655501062888be6ee391e426c90a13
] 
[test_storage.py: asyncify some more tests, and fix create methods. refs #999
david-sarah@jacaranda.org**20111012025739
 Ignore-this: 1574d8175917665f44d278d13f815bb9
] 
[test_storage.py: add a test that we can create a share, exercising the backend's get_share and get_shares methods. This may explicate particular kinds of backend failure better than the existing tests. refs #999
david-sarah@jacaranda.org**20111012025514
 Ignore-this: f52983e4f3d96ea26ef25856d4cc92ce
] 
[test_storage.py: Move test_seek to its own class, since it is independent of the backend. Also move test_reserved_space to ServerWithDiskBackend, since reserved_space is specific to that backend. refs #999
david-sarah@jacaranda.org**20111012025149
 Ignore-this: 281de8befe51e24cd638886fb5063cd2
] 
[util/deferredutil.py: remove unneeded utility functions. refs #999
david-sarah@jacaranda.org**20111012024440
 Ignore-this: 17380afd9079442785d0cb78876c7fd5
] 
[Move configuration of each backend into the backend itself. refs #999
david-sarah@jacaranda.org**20111012014004
 Ignore-this: c337c43e4c4a05617de62f4acf7119d0
] 
[test_storage.py: fix test failures in MDMFProxies. refs #999
david-sarah@jacaranda.org**20111012000848
 Ignore-this: 798f2a4e960ee444e401a10748afeb08
] 
[test_storage.py: cosmetics. refs #999
david-sarah@jacaranda.org**20111012000442
 Ignore-this: d3514fa8d69f38d3b45204e2224152d5
] 
[storage/backends/disk/disk_backend.py: trivial fix to a comment. #refs 999
david-sarah@jacaranda.org**20111011165704
 Ignore-this: b9031b01ef643cb973a41af277d941c0
] 
[check-miscaptures.py: report the number of files that were not analysed due to syntax errors (and don't count them in the number of suspicious captures). refs #1555
david-sarah@jacaranda.org**20111009050301
 Ignore-this: 62ee03f4b8a96c292e75c097ad87d52e
] 
[check-miscaptures.py: handle corner cases around default arguments correctly. Also make a minor optimization when there are no assigned variables to consider. refs #1555
david-sarah@jacaranda.org**20111009045023
 Ignore-this: f49ece515620081da1d745ae6da19d21
] 
[check-miscaptures.py: Python doesn't really have declarations; report the topmost assignment. refs #1555
david-sarah@jacaranda.org**20111009044800
 Ignore-this: 4905c9dfe7726f433333e216a6760a4b
] 
[check-miscaptures.py: handle destructuring function arguments correctly. refs #1555
david-sarah@jacaranda.org**20111009044710
 Ignore-this: f9de7d95e94446507a206c88d3f98a23
] 
[check-miscaptures.py: check while loops and list comprehensions as well as for loops. Also fix a pyflakes warning. refs #1555
david-sarah@jacaranda.org**20111009044022
 Ignore-this: 6526e4e315ca6461b1fbc2da5568e444
] 
[Fix pyflakes warnings in misc/ directories other than misc/build_helpers. refs #1557
david-sarah@jacaranda.org**20111007033031
 Ignore-this: 7daf5862469732d8cabc355266622b74
] 
[Makefile: include misc/ directories other than misc/build_helpers in SOURCES. refs #1557
david-sarah@jacaranda.org**20111007032958
 Ignore-this: 31376ec01401df7972e83341dc65aa05
] 
[util/happinessutil.py: suppress a warning from check-miscaptures. (It is not a bug because the capturing function is only used by a 'map' in the same iteration.) refs #1556
david-sarah@jacaranda.org**20111009052106
 Ignore-this: 16a62844bae083800d6b6a7334abc9bc
] 
[misc/coding_tools/make-canary-files.py: fix a suspicious capture reported by check-miscaptures (although it happens not to be a bug because the callback will be processed synchronously). refs #1556
david-sarah@jacaranda.org**20111009050531
 Ignore-this: 2d1a696955a4c1f7d9c649d4ecefd7de
] 
[Fix two pyflakes warnings about unused imports. refs #999
david-sarah@jacaranda.org**20111011051745
 Ignore-this: 23c17f8eb36a30f4e3b662a778bc4bb7
] 
[test_storage.py: fix asyncification of three tests in MDMFProxies. refs #999
david-sarah@jacaranda.org**20111011051319
 Ignore-this: a746cc2ed1f4fbcf95bad7624a0544e9
] 
[test_storage.py: fix a trivial bug in MDMFProxies.test_write. refs #999
david-sarah@jacaranda.org**20111011045645
 Ignore-this: 943c6da82eca7b2d247cfb7d75afc9b7
] 
[test_storage.py: fix a typo in test_null_backend. refs #999
david-sarah@jacaranda.org**20111011045133
 Ignore-this: ddf00d1d65182d520904168827c792c4
] 
[test_storage.py: fix a bug introduced by asyncification of test_allocate. refs #999
david-sarah@jacaranda.org**20111011044131
 Ignore-this: 1460b8a713081c8bbe4d298ab39f264f
] 
[test_storage.py: make MutableServer.test_leases pass. refs #999
david-sarah@jacaranda.org**20111011002917
 Ignore-this: dce275f6508a7cfe31c0af82483eea97
] 
[test/common.py: in shouldFail and shouldHTTPError, when the raised exception does not include the expected substring (or, for shouldHTTPError, when the status code is wrong), mention which test that happened in.
david-sarah@jacaranda.org**20111011002227
 Ignore-this: 836cabe9ef774617122905b214a0b8e8
] 
[test/mock_s3.py: fix a bug that was causing us to use the wrong directory for share files. refs #999
david-sarah@jacaranda.org**20111010231344
 Ignore-this: bc63757f5dd8d31643bd9919f2ecd98c
] 
[Add fileutil.fp_list(fp) which is like fp.children(), but returns [] in case of a directory that does not exist. Use it to simplify the disk backend and mock S3 bucket implementations. refs #999
david-sarah@jacaranda.org**20111010231146
 Ignore-this: fd4fa8b1446fc7e5c03631b4092c20cc
] 
[S3 backend: move the implementation of list_objects from s3_bucket.py to s3_common.py, making s3_bucket.py simpler and list_objects easier to test independently. refs #999
david-sarah@jacaranda.org**20111010230751
 Ignore-this: 2f9a8f75671e87d2caba2ac6c6d4bdfd
] 
[Make unlink() on share objects consistently idempotent. refs #999
david-sarah@jacaranda.org**20111010204417
 Ignore-this: 1dc559fdd89b7135cec64ffca62fc96a
] 
[Null backend: implement unlink and readv more correctly. refs #999
david-sarah@jacaranda.org**20111010204404
 Ignore-this: 3386bc2a1cd0ff6268def31c5c5ce3a1
] 
[test_download.py: fix test_download_failover (it should tolerate non-existing shares in _clobber_most_shares). refs #999
david-sarah@jacaranda.org**20111010204142
 Ignore-this: db648ffcbd5e37cf236f49ecc1e720fc
] 
[interfaces.py: resolve another conflict with trunk. refs #999
david-sarah@jacaranda.org**20111010200903
 Ignore-this: 163067eab9a5c71e12c6cac058a03832
] 
[interfaces.py: fix a typo in the name of IMutableSlotWriter.put_encprivkey. refs #393
david-sarah@jacaranda.org**20111010194642
 Ignore-this: eb65439e8dd891c169b43b1679c29238
] 
[interfaces.py: resolve conflicts with trunk. refs #999
david-sarah@jacaranda.org**20111010195634
 Ignore-this: 8e02c7933392491ba3deb678c5bc5876
] 
[interfaces.py: remove get_extension_params and set_extension_params methods from IMutableFileURI. refs #393, #1526
david-sarah@jacaranda.org**20111010194842
 Ignore-this: 6012be6fcc12f560aeeeac0be2d337d1
] 
[Instrument some assertions to report the failed values. refs #999
david-sarah@jacaranda.org**20111010191733
 Ignore-this: 4e886faa5909bf703af8228194ae759c
] 
[test_storage.py: move some tests that were not applicable to all backends out of ServerTest. refs #999
david-sarah@jacaranda.org**20111010181214
 Ignore-this: d2310591c71c4d2d2c5ff4e316f15542
] 
[storage/backends/disk/mutable.py: put back a correct assertion that had been disabled. storage/base.py: fix the bug that was causing that assertion to fail. refs #999
david-sarah@jacaranda.org**20111009232142
 Ignore-this: dbb644f596bf3c42575b9d9fadc2c9d9
] 
[test_storage.py: fix a trivial bug in LeaseCrawler.test_unpredictable_future. refs #999
david-sarah@jacaranda.org**20111007195753
 Ignore-this: 357539b4cf8b7455c1787ac591b0ee23
] 
[Asyncification, and resolution of conflicts. #999
david-sarah@jacaranda.org**20111007193418
 Ignore-this: 29b15345aecd3adeef2c2392ca90d4ff
] 
[disk backend: size methods should no longer return Deferreds. refs #999
david-sarah@jacaranda.org**20111007193327
 Ignore-this: 8d32ffdbb81d88a30352f344e385feff
] 
[Ensure that helper classes are not treated as test cases. Also fix a missing mixin. refs #999
david-sarah@jacaranda.org**20111007081439
 Ignore-this: a222110248f378d91c232799bcd5d3a6
] 
[More miscapture fixes. refs #999
david-sarah@jacaranda.org**20111007080916
 Ignore-this: 85004d4e3a609a2ef70a38164897ff02
] 
[Partially asyncify crawlers. refs #999
david-sarah@jacaranda.org**20111007080657
 Ignore-this: f6a15b81592bfff33ccf09301dbdfca1
] 
[unlink() on share objects should be idempotent. refs #999
david-sarah@jacaranda.org**20111007080615
 Ignore-this: ff87d0b30fc81dd8e90bc5c6852955eb
] 
[Make sure that get_size etc. work correctly on an ImmutableS3ShareForWriting after it has been closed. Also simplify by removing the _end_offset attribute. refs #999
david-sarah@jacaranda.org**20111007080342
 Ignore-this: 9d8ce463daee2ef1b7dc33aca70a0379
] 
[Remove an inapplicable comment. refs #999
david-sarah@jacaranda.org**20111007080128
 Ignore-this: 1c7fae3ffc9ac412ad9ab6e411ef9be7
] 
[Remove unused load method and _loaded attribute from s3/mutable.py. refs #999
david-sarah@jacaranda.org**20111007080051
 Ignore-this: b92ed543f7dcf56df0510de8515230e1
] 
[Fix a duplicate umid. refs #999
david-sarah@jacaranda.org**20111007080001
 Ignore-this: bb9519d15220ab7350731abf19038c2e
] 
[Fix some miscapture bugs. refs #999
david-sarah@jacaranda.org**20111007075915
 Ignore-this: 88dc8ec49e93e5d62c2abb8f496706cb
] 
[Add a _get_sharedir() method on IShareSet, implemented by the disk and mock S3 backends, for use by tests. refs #999
david-sarah@jacaranda.org**20111007075645
 Ignore-this: 9e9ce0d244785da8ac4a3c0aa948ddce
] 
[Add misc/coding_tools/check-miscaptures.py to detect incorrect captures of variables declared in a for loop, and a 'make check-miscaptures' Makefile target to run it. (It is also run by 'make code-checks'.) This is a rewritten version that reports much fewer false positives, by determining captured variables more accurately. fixes #1555
david-sarah@jacaranda.org**20111007074121
 Ignore-this: 51318e9678d132c374ea557ab955e79e
] 
[Add a get_share method to IShareSet, to get a specific share. refs #999
david-sarah@jacaranda.org**20111007075426
 Ignore-this: 493ddfe83414208f08a22b9f327d6b69
] 
[Fix some more potential bugs in test code exposed by check-miscaptures.py. refs #1556
david-sarah@jacaranda.org**20111007033847
 Ignore-this: aec8a543e9b5c3563b60692c647439a8
] 
[Fix some potential bugs (in non-test code) exposed by check-miscaptures.py. refs #1556
david-sarah@jacaranda.org**20111007032444
 Ignore-this: bac9ed65b21c2136c4db2482b3c093f7
] 
[Fix some potential bugs in test code exposed by check-miscaptures.py. refs #1556
david-sarah@jacaranda.org**20111007023443
 Ignore-this: e48b2c2d200521d6f28c737994ce3a2a
] 
[misc/simulators/hashbasedsig.py: simplify by removing unnecessary local function that captured a variable declared in a for loop (this was not a bug, but the code was unclear). Also fix a pyflakes warning about an import. refs #1556
david-sarah@jacaranda.org**20111007023001
 Ignore-this: 446c94efae02ded5e85eb3335ca5e69
] 
[immutable/literal.py: add pauseProducing method to LiteralProducer. refs #1537
david-sarah@jacaranda.org**20111003195239
 Ignore-this: 385ee3379a2819381937357f1eac457
] 
[docs: fix the rst formatting of COPYING.TGPPL.rst
zooko@zooko.com**20111003043333
 Ignore-this: c5fbc83f4a3db81a0c95b27053c463c5
 Now it renders correctly both on trac and with rst2html --verbose from docutils v0.8.1.
] 
[MDMF: remove extension fields from caps, tolerate arbitrary ones. Fixes #1526
Brian Warner <warner@lothar.com>**20111001233553
 Ignore-this: 335e1690aef1146a2c0b8d8c18c1cb21
 
 The filecaps used to be produced with hints for 'k' and segsize, but they
 weren't actually used, and doing so had the potential to limit how we change
 those filecaps in the future. Also the parsing code had some problems dealing
 with other numbers of extensions. Removing the existing fields and making the
 parser tolerate (and ignore) extra ones makes MDMF more future-proof.
] 
[More asyncification of tests. Also fix some bugs due to capture of slots in for loops. refs #999
david-sarah@jacaranda.org**20111004010813
 Ignore-this: 15bf68748ab737d1edc24552ce192f8b
] 
[s3/s3_common.py: remove incorrect 'self' arguments from interface methods in IS3Bucket. refs #999
david-sarah@jacaranda.org**20111004010745
 Ignore-this: d5f66be90062292164d3f017aef3d6f4
] 
[no_network.py: Clean up whitespace around code changed by previous patch.
david-sarah@jacaranda.org**20111004010407
 Ignore-this: 647ec8a9346dca1a41212ab250619b72
] 
[no_network.py: Fix potential bugs in some tests due to capture of slots in for loops.
david-sarah@jacaranda.org**20111004010231
 Ignore-this: 9c496877613a3befd54979e5de6e63d2
] 
[Add a share._get_filepath() method used by tests to get the FilePath for a share, rather than accessing the _home attribute. refs #999
david-sarah@jacaranda.org**20111004004604
 Ignore-this: ec873e356b7ebd74f52336dd92dea8aa
] 
[s3/immutable.py: minor simplification in ImmutableS3ShareForReading. refs #999
david-sarah@jacaranda.org**20110930212714
 Ignore-this: d71e2466231f695891a6b8d1df945687
] 
[free up the buffer used to hold data while it is being written to ImmutableS3ShareForWriting
zooko@zooko.com**20110930060238
 Ignore-this: 603b2c8bb1f4656bdde5876ac95aa5c9
] 
[FIX THE BUG!
zooko@zooko.com**20110930032140
 Ignore-this: fd32c4ac3054ae6fc2b9433f113b2fd6
] 
[fix another bug in ImmutableShareS3ForWriting
zooko@zooko.com**20110930025701
 Ignore-this: 6ad7bd17111b12d96991172fbe04d76
] 
[really fix the bug in ImmutableS3ShareForWriting
zooko@zooko.com**20110930023501
 Ignore-this: 36a7804433cab667566d119af7223425
] 
[Add dummy lease methods to immutable S3 share objects. refs #999
david-sarah@jacaranda.org**20110930021703
 Ignore-this: 7c21f140020edd64027c71be0f32c2b2
] 
[fix bug in ImmutableS3ShareForWriting
zooko@zooko.com**20110930020535
 Ignore-this: f7f63d2fc2086903a195cc000f306b88
] 
[test_storage.py: Server class uses ShouldFailMixin. refs #999
david-sarah@jacaranda.org**20110930001349
 Ignore-this: 4cf1ef21bbf85d7fe52ab660f59ff237
] 
[mock_s3.py: fix bug in MockS3Error constructor. refs #999
david-sarah@jacaranda.org**20110930001326
 Ignore-this: 4d0ebd9120fc8e99b15924c671cd0927
] 
[return res
zooko@zooko.com**20110930000446
 Ignore-this: 6f73b3e389612c73c6590007229ad8e
] 
[fix doc to say that secret access key goes into private/s3secret
zooko@zooko.com**20110930000256
 Ignore-this: c054ff78041a05b3177b3c1b3e9d4ae7
] 
[s3_bucket.py: fix an incorrect argument signature for list_objects. refs #999
david-sarah@jacaranda.org**20110929235646
 Ignore-this: f02e3a23f28fadef71c70fd0b1592ba6
] 
[Make sure that the statedir is created before trying to use it. refs #999
david-sarah@jacaranda.org**20110929234845
 Ignore-this: b5f0529b1f2a5b5250c2ee2091cbe24b
] 
[test/mock_s3.py: fix a typo. refs #999
david-sarah@jacaranda.org**20110929234808
 Ignore-this: ccdff591f9b301f7f486454a4366c2b3
] 
[test_storage.py: only run test_large_share on the disk backend. (It will wedge your machine if run on the S3 backend with MockS3Bucket.) refs #999
david-sarah@jacaranda.org**20110929234725
 Ignore-this: ffa7c08458ee0159455b6f1cd1c3ff48
] 
[Fixes to S3 config parsing, with tests. refs #999
david-sarah@jacaranda.org**20110929225014
 Ignore-this: 19aa5a3e9575b0c2f77b19fe1bcbafcb
] 
[Add missing src/allmydata/test/mock_s3.py (mock implementation of an S3 bucket). refs #999
david-sarah@jacaranda.org**20110929212229
 Ignore-this: a1433555d4bb0b8b36fb80feb122187b
] 
[Make the s3.region option case-insensitive (txaws expects uppercase). refs #999
david-sarah@jacaranda.org**20110929211606
 Ignore-this: def83d3fa368c315573e5f1bad5ee7f9
] 
[Fix missing add_lease method on ImmutableS3ShareForWriting. refs #999
david-sarah@jacaranda.org**20110929211524
 Ignore-this: 832f0d94f912b17006b0dbaab94846b6
] 
[Add missing src/allmydata/storage/backends/s3/s3_bucket.py. refs #999
david-sarah@jacaranda.org**20110929211416
 Ignore-this: aa783c5d7c32af172b5c5a3d62c3faf2
] 
[scripts/debug.py: repair stale code, and use the get_disk_share function defined by disk_backend instead of duplicating it. refs #999
david-sarah@jacaranda.org**20110929211252
 Ignore-this: 5dda548e8703e35f0c103467346627ef
] 
[Fix a bug in the new config parsing code when reserved_space is not present for a disk backend. refs #999
david-sarah@jacaranda.org**20110929211106
 Ignore-this: b05bd3c4ff7d90b5ecb1e6a54717b735
] 
[test_storage.py: Avoid using the same working directory for different test classes. refs #999
david-sarah@jacaranda.org**20110929210954
 Ignore-this: 3a01048e941c61c603eec603d064bebb
] 
[More asycification of tests. refs #999
david-sarah@jacaranda.org**20110929210727
 Ignore-this: 87690a62f89a07e63b859c24948d262d
] 
[Fix a bug in disk_backend.py. refs #999
david-sarah@jacaranda.org**20110929182511
 Ignore-this: 4f9a62adf03fc3221e46b54f7a4a960b
] 
[docs/backends/S3.rst: add s3.region option. Also minor changes to configuration.rst. refs #999
david-sarah@jacaranda.org**20110929182442
 Ignore-this: 2992ead5f8d9357a0d9b912b1e0bd932
] 
[Updates to test_backends.py. refs #999
david-sarah@jacaranda.org**20110929182016
 Ignore-this: 3bac19179308e6f27e54c45c7cad4dc6
] 
[Implement selection of backends from tahoe.cfg options. Also remove the discard_storage parameter from the disk backend. refs #999
david-sarah@jacaranda.org**20110929181754
 Ignore-this: c7f78e7db98326723033f44e56858683
] 
[test_storage.py: fix an incorrect argument in construction of S3Backend. refs #999
david-sarah@jacaranda.org**20110929081331
 Ignore-this: 33ad68e0d3a15e3fa1dda90df1b8365c
] 
[Move the implementation of lease methods to disk_backend.py, and add stub implementations in s3_backend.py that raise NotImplementedError. Fix the lease methods in the disk backend to be synchronous. Also make sure that get_shares() returns a Deferred list sorted by shnum. refs #999
david-sarah@jacaranda.org**20110929081132
 Ignore-this: 32cbad21c7236360e2e8e84a07f88597
] 
[Make the make_bucket_writer method synchronous. refs #999
david-sarah@jacaranda.org**20110929080712
 Ignore-this: 1de299e791baf1cf1e2a8d4b593e8ba1
] 
[Add get_s3_share function in place of S3ShareSet._load_shares. refs #999
david-sarah@jacaranda.org**20110929080530
 Ignore-this: f99665979612e42ecefa293bda0db5de
] 
[Complete the splitting of the immutable IStoredShare interface into IShareForReading and IShareForWriting. Also remove the 'load' method from shares, and other minor interface changes. refs #999
david-sarah@jacaranda.org**20110929075544
 Ignore-this: 8c923051869cf162d9840770b4a08573
] 
[split Immutable S3 Share into for-reading and for-writing classes, remove unused (as far as I can tell) methods, use cStringIO for buffering the writes
zooko@zooko.com**20110929055038
 Ignore-this: 82d8c4488a8548936285a975ef5a1559
 TODO: define the interfaces that the new classes claim to implement
] 
[Comment out an assertion that was causing all mutable tests to fail. THIS IS PROBABLY WRONG. refs #999
david-sarah@jacaranda.org**20110929041110
 Ignore-this: 1e402d51ec021405b191757a37b35a94
] 
[Fix some incorrect or incomplete asyncifications. refs #999
david-sarah@jacaranda.org**20110929040800
 Ignore-this: ed70e9af2190217c84fd2e8c41de4c7e
] 
[Add some debugging assertions that share objects are not Deferred. refs #999
david-sarah@jacaranda.org**20110929040657
 Ignore-this: 5c7f56a146f5a3c353c6fe5b090a7dc5
] 
[scripts/debug.py: take account of some API changes. refs #999
david-sarah@jacaranda.org**20110929040539
 Ignore-this: 933c3d44b993c041105038c7d4514386
] 
[Make get_sharesets_for_prefix synchronous for the time being (returning a Deferred breaks crawlers). refs #999
david-sarah@jacaranda.org**20110929040136
 Ignore-this: e94b93d4f3f6173d9de80c4121b68748
] 
[More asyncification of tests. refs #999
david-sarah@jacaranda.org**20110929035644
 Ignore-this: 28b650a9ef593b3fd7524f6cb562ad71
] 
[no_network.py: add some assertions that the things we wrap using LocalWrapper are not Deferred (which is not supported and causes hard-to-debug failures). refs #999
david-sarah@jacaranda.org**20110929035537
 Ignore-this: fd103fbbb54fbbc17b9517c78313120e
] 
[Add some debugging code (switched off) to no_network.py. When switched on (PRINT_TRACEBACKS = True), this prints the stack trace associated with the caller of a remote method, mitigating the problem that the traceback normally gets lost at that point. TODO: think of a better way to preserve the traceback that can be enabled by default. refs #999
david-sarah@jacaranda.org**20110929035341
 Ignore-this: 2a593ec3ee450719b241ea8d60a0f320
] 
[Use factory functions to create share objects rather than their constructors, to allow the factory to return a Deferred. Also change some methods on IShareSet and IStoredShare to return Deferreds. Refactor some constants associated with mutable shares. refs #999
david-sarah@jacaranda.org**20110928052324
 Ignore-this: bce0ac02f475bcf31b0e3b340cd91198
] 
[Work in progress for asyncifying the backend interface (necessary to call txaws methods that return Deferreds). This is incomplete so lots of tests fail. refs #999
david-sarah@jacaranda.org**20110927073903
 Ignore-this: ebdc6c06c3baa9460af128ec8f5b418b
] 
[mutable/publish.py: don't crash if there are no writers in _report_verinfo. refs #999
david-sarah@jacaranda.org**20110928014126
 Ignore-this: 9999c82bb3057f755a6e86baeafb8a39
] 
[scripts/debug.py: fix incorrect arguments to dump_immutable_share. refs #999
david-sarah@jacaranda.org**20110928014049
 Ignore-this: 1078ee3f06a2f36b29e0cf694d2851cd
] 
[test_system.py: more debug output for a failing check in test_filesystem. refs #999
david-sarah@jacaranda.org**20110928014019
 Ignore-this: e8bb77b8f7db12db7cd69efb6e0ed130
] 
[test_system.py: incorrect arguments were being passed to the constructor for MutableDiskShare. refs #999
david-sarah@jacaranda.org**20110928013857
 Ignore-this: e9719f74e7e073e37537f9a71614b8a0
] 
[Undo an incompatible change to RIStorageServer. refs #999
david-sarah@jacaranda.org**20110928013729
 Ignore-this: bea4c0f6cb71202fab942cd846eab693
] 
[mutable/publish.py: resolve conflicting patches. refs #999
david-sarah@jacaranda.org**20110927073530
 Ignore-this: 6154a113723dc93148151288bd032439
] 
[test_storage.py: fix test_no_st_blocks. refs #999
david-sarah@jacaranda.org**20110927072848
 Ignore-this: 5f12b784920f87d09c97c676d0afa6f8
] 
[Cleanups to S3 backend (not including Deferred changes). refs #999
david-sarah@jacaranda.org**20110927071855
 Ignore-this: f0dca788190d92b1edb1ee1498fb34dc
] 
[Cleanups to disk backend. refs #999
david-sarah@jacaranda.org**20110927071544
 Ignore-this: e9d3fd0e85aaf301c04342fffdc8f26
] 
[test_storage.py: fix test_status_bad_disk_stats. refs #999
david-sarah@jacaranda.org**20110927071403
 Ignore-this: 6108fee69a60962be2df2ad11b483a11
] 
[util/deferredutil.py: add some utilities for asynchronous iteration. refs #999
david-sarah@jacaranda.org**20110927070947
 Ignore-this: ac4946c1e5779ea64b85a1a420d34c9e
] 
[Add 'has-immutable-readv' to server version information. refs #999
david-sarah@jacaranda.org**20110923220935
 Ignore-this: c3c4358f2ab8ac503f99c968ace8efcf
] 
[Minor cleanup to disk backend. refs #999
david-sarah@jacaranda.org**20110923205510
 Ignore-this: 79f92d7c2edb14cfedb167247c3f0d08
] 
[Update the S3 backend. refs #999
david-sarah@jacaranda.org**20110923205345
 Ignore-this: 5ca623a17e09ddad4cab2f51b49aec0a
] 
[Update the null backend to take into account interface changes. Also, it now records which shares are present, but not their contents. refs #999
david-sarah@jacaranda.org**20110923205219
 Ignore-this: 42a23d7e253255003dc63facea783251
] 
[Make EmptyShare.check_testv a simple function. refs #999
david-sarah@jacaranda.org**20110923204945
 Ignore-this: d0132c085f40c39815fa920b77fc39ab
] 
[The cancel secret needs to be unique, even if it isn't explicitly provided. refs #999
david-sarah@jacaranda.org**20110923204914
 Ignore-this: 6c44bb908dd4c0cdc59506b2d87a47b0
] 
[Implement readv for immutable shares. refs #999
david-sarah@jacaranda.org**20110923204611
 Ignore-this: 24f14b663051169d66293020e40c5a05
] 
[Remove redundant si_s argument from check_write_enabler. refs #999
david-sarah@jacaranda.org**20110923204425
 Ignore-this: 25be760118dbce2eb661137f7d46dd20
] 
[interfaces.py: add fill_in_space_stats method to IStorageBackend. refs #999
david-sarah@jacaranda.org**20110923203723
 Ignore-this: 59371c150532055939794fed6c77dcb6
] 
[Add incomplete S3 backend. refs #999
david-sarah@jacaranda.org**20110923041314
 Ignore-this: b48df65699e3926dcbb87b5f755cdbf1
] 
[Move advise_corrupt_share to allmydata/storage/backends/base.py, since it will be common to the disk and S3 backends. refs #999
david-sarah@jacaranda.org**20110923041115
 Ignore-this: 782b49f243bd98fcb6c249f8e40fd9f
] 
[A few comment cleanups. refs #999
david-sarah@jacaranda.org**20110923041003
 Ignore-this: f574b4a3954b6946016646011ad15edf
] 
[mutable/publish.py: elements should not be removed from a dictionary while it is being iterated over. refs #393
david-sarah@jacaranda.org**20110923040825
 Ignore-this: 135da94bd344db6ccd59a576b54901c1
] 
[Blank line cleanups.
david-sarah@jacaranda.org**20110923012044
 Ignore-this: 8e1c4ecb5b0c65673af35872876a8591
] 
[Reinstate the cancel_lease methods of ImmutableDiskShare and MutableDiskShare, since they are needed for lease expiry. refs #999
david-sarah@jacaranda.org**20110922183323
 Ignore-this: a11fb0dd0078ff627cb727fc769ec848
] 
[Fix most of the crawler tests. refs #999
david-sarah@jacaranda.org**20110922183008
 Ignore-this: 116c0848008f3989ba78d87c07ec783c
] 
[Fix some more test failures. refs #999
david-sarah@jacaranda.org**20110922045451
 Ignore-this: b726193cbd03a7c3d343f6e4a0f33ee7
] 
[uri.py: resolve a conflict between trunk and the pluggable-backends patches. refs #999
david-sarah@jacaranda.org**20110921222038
 Ignore-this: ffeeab60d8e71a6a29a002d024d76fcf
] 
[Fix more shallow bugs, mainly FilePathification. Also, remove the max_space_per_bucket parameter from BucketWriter since it can be obtained from the _max_size attribute of the share (via a new get_allocated_size() accessor). refs #999
david-sarah@jacaranda.org**20110921221421
 Ignore-this: 600e3ccef8533aa43442fa576c7d88cf
] 
[More fixes to tests needed for pluggable backends. refs #999
david-sarah@jacaranda.org**20110921184649
 Ignore-this: 9be0d3a98e350fd4e17a07d2c00bb4ca
] 
[docs/backends/S3.rst, disk.rst: describe type of space settings as 'quantity of space', not 'str'. refs #999
david-sarah@jacaranda.org**20110921031705
 Ignore-this: a74ed8e01b0a1ab5f07a1487d7bf138
] 
[docs/backends/S3.rst: remove Issues section. refs #999
david-sarah@jacaranda.org**20110921031625
 Ignore-this: c83d8f52b790bc32488869e6ee1df8c2
] 
[Fix some incorrect attribute accesses. refs #999
david-sarah@jacaranda.org**20110921031207
 Ignore-this: f1ea4c3ea191f6d4b719afaebd2b2bcd
] 
[docs/backends: document the configuration options for the pluggable backends scheme. refs #999
david-sarah@jacaranda.org**20110920171737
 Ignore-this: 5947e864682a43cb04e557334cda7c19
] 
[Work-in-progress, includes fix to bug involving BucketWriter. refs #999
david-sarah@jacaranda.org**20110920033803
 Ignore-this: 64e9e019421454e4d08141d10b6e4eed
] 
[Pluggable backends -- all other changes. refs #999
david-sarah@jacaranda.org**20110919233256
 Ignore-this: 1a77b6b5d178b32a9b914b699ba7e957
] 
[Pluggable backends -- new and moved files, changes to moved files. refs #999
david-sarah@jacaranda.org**20110919232926
 Ignore-this: ec5d2d1362a092d919e84327d3092424
] 
[interfaces.py: 'which -> that' grammar cleanup.
david-sarah@jacaranda.org**20110825003217
 Ignore-this: a3e15f3676de1b346ad78aabdfb8cac6
] 
[test/test_runner.py: BinTahoe.test_path has rare nondeterministic failures; this patch probably fixes a problem where the actual cause of failure is masked by a string conversion error.
david-sarah@jacaranda.org**20110927225336
 Ignore-this: 6f1ad68004194cc9cea55ace3745e4af
] 
[docs/configuration.rst: add section about the types of node, and clarify when setting web.port enables web-API service. fixes #1444
zooko@zooko.com**20110926203801
 Ignore-this: ab94d470c68e720101a7ff3c207a719e
] 
[TAG allmydata-tahoe-1.9.0a2
warner@lothar.com**20110925234811
 Ignore-this: e9649c58f9c9017a7d55008938dba64f
] 
Patch bundle hash:
1c1ccb4330b0a9ef240741001e0bb26cc5da8250