Azure: change 'container_name' config entry to 'container' for consistency with OpenS...
authorDaira Hopwood <daira@jacaranda.org>
Sat, 18 May 2013 01:56:11 +0000 (02:56 +0100)
committerDaira Hopwood <daira@jacaranda.org>
Fri, 16 Oct 2015 16:58:53 +0000 (17:58 +0100)
Also fix a hidden bug in a test.

Signed-off-by: Daira Hopwood <david-sarah@jacaranda.org>
docs/backends/cloud.rst
src/allmydata/storage/backends/cloud/msazure/msazure_container.py
src/allmydata/test/test_client.py

index aafcae3d98ed96ba8010b89deee358f5a17bd632..a7a296b3f85f75eff71c3f9cd859659b98231f0e 100644 (file)
@@ -224,7 +224,7 @@ server's ``tahoe.cfg`` file:
     This is the account name (subdomain) you chose when creating the account,
     e.g. ``mydomain``.
 
-``msazure.container_name = (string, required)``
+``msazure.container = (string, required)``
 
     This controls which container will be used to hold shares. The Tahoe-LAFS
     storage server will only modify and access objects in the configured
index c2936b04ee3aab5ca2590174bc64028fd4c3bc6d..ddeb3a10ab959e4d47b501da3afda1b21624b94a 100644 (file)
@@ -233,7 +233,7 @@ def configure_msazure_container(storedir, config):
     Configure the MS Azure storage container.
     """
     account_name = config.get_config("storage", "msazure.account_name")
-    container_name = config.get_config("storage", "msazure.container_name")
+    container_name = config.get_config("storage", "msazure.container")
     account_key = config.get_private_config("msazure_account_key")
     return MSAzureStorageContainer(account_name, account_key, container_name)
 
index 6b7ecafe0baf4b1e3d8a0e77a4808bf9769d1d56..0163150a8d249743c1c3fe03524d4867cc7db6ac 100644 (file)
@@ -505,11 +505,10 @@ class Basic(testutil.ReallyEqualMixin, unittest.TestCase):
 
     def test_msazure_config_required(self):
         """
-        account_name and container_name are all required by MS Azure
-        configuration.
+        account_name and container are all required by MS Azure configuration.
         """
         configs = ["mszure.account_name = theaccount",
-                   "msazure.container_name = bucket"]
+                   "msazure.container = bucket"]
         for i in range(len(configs)):
             basedir = self.mktemp()
             os.mkdir(basedir)
@@ -535,8 +534,8 @@ class Basic(testutil.ReallyEqualMixin, unittest.TestCase):
                        "[storage]\n" +
                        "enabled = true\n" +
                        "backend = cloud.msazure\n" +
-                       "googlestorage.account_name = theaccount\n" +
-                       "googlestorage.container_name = bucket\n")
+                       "msazure.account_name = theaccount\n" +
+                       "msazure.container = bucket\n")
         self.failUnlessRaises(MissingConfigEntry, client.Client, basedir)
 
     @mock.patch('allmydata.storage.backends.cloud.msazure.msazure_container.MSAzureStorageContainer')
@@ -553,7 +552,7 @@ class Basic(testutil.ReallyEqualMixin, unittest.TestCase):
                        "enabled = true\n" +
                        "backend = cloud.msazure\n" +
                        "msazure.account_name = theaccount\n" +
-                       "msazure.container_name = bucket\n")
+                       "msazure.container = bucket\n")
         c = client.Client(basedir)
         server = c.getServiceNamed("storage")
         self.failUnless(isinstance(server.backend, CloudBackend), server.backend)