Azure: change 'container_name' config entry to 'container' for consistency with OpenS...

Also fix a hidden bug in a test.

Signed-off-by: Daira Hopwood <david-sarah@jacaranda.org>

diff --git a/docs/backends/cloud.rst b/docs/backends/cloud.rst
index aafcae3d98ed96ba8010b89deee358f5a17bd632..a7a296b3f85f75eff71c3f9cd859659b98231f0e 100644 (file)
--- a/docs/backends/cloud.rst
+++ b/docs/backends/cloud.rst
@@ -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

diff --git a/src/allmydata/storage/backends/cloud/msazure/msazure_container.py b/src/allmydata/storage/backends/cloud/msazure/msazure_container.py
index c2936b04ee3aab5ca2590174bc64028fd4c3bc6d..ddeb3a10ab959e4d47b501da3afda1b21624b94a 100644 (file)
--- a/src/allmydata/storage/backends/cloud/msazure/msazure_container.py
+++ b/src/allmydata/storage/backends/cloud/msazure/msazure_container.py
@@ -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)
 

diff --git a/src/allmydata/test/test_client.py b/src/allmydata/test/test_client.py
index dd5d182ec5f1c1f1601ae01ff288a96924215253..d61629e64bea33be7c133fcb924436536ac2bf26 100644 (file)
--- a/src/allmydata/test/test_client.py
+++ b/src/allmydata/test/test_client.py
@@ -500,11 +500,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)
@@ -530,8 +529,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')
@@ -548,7 +547,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)