Address review comments from Daira.

1. Fix typo.
2. Rename config item googlestorage.bucket_name to googlestorage.bucket for
   consistency.

diff --git a/docs/backends/cloud.rst b/docs/backends/cloud.rst
index e8b258fbad30ffb2ece27858c9d548685a93f44e..c580da658c88911d3732be68922adb0b605440e1 100644 (file)
--- a/docs/backends/cloud.rst
+++ b/docs/backends/cloud.rst
@@ -168,11 +168,11 @@ the server's ``tahoe.cfg`` file:
 
 ``googlestorage.project_id = (string, required)``
 
-    This is the project number of the project you you created,
+    This is the project number of the project you created,
     e.g. ``123456``. You can find this number in the Google Cloud Storage
     section of the APIs console (the number following `x-goog-project-id`).
 
-``googlestorage.bucket_name = (string, required)``
+``googlestorage.bucket = (string, required)``
 
     This controls which bucket (aka container) will be used to hold
     shares. The Tahoe-LAFS storage server will only modify and access objects

diff --git a/src/allmydata/storage/backends/cloud/googlestorage/googlestorage_container.py b/src/allmydata/storage/backends/cloud/googlestorage/googlestorage_container.py
index 05f2f17e657f933c64e67dc2914c89a1aa362932..e06e754e9b87cfb1a2a6e35e8869ffe588a6d9eb 100644 (file)
--- a/src/allmydata/storage/backends/cloud/googlestorage/googlestorage_container.py
+++ b/src/allmydata/storage/backends/cloud/googlestorage/googlestorage_container.py
@@ -238,7 +238,7 @@ def configure_googlestorage_container(storedir, config):
     """
     account_email = config.get_config("storage", "googlestorage.account_email")
     private_key = config.get_private_config("googlestorage_private_key")
-    bucket_name = config.get_config("storage", "googlestorage.bucket_name")
+    bucket_name = config.get_config("storage", "googlestorage.bucket")
     # Only necessary if we do bucket creation/deletion, otherwise can be
     # removed:
     project_id = config.get_config("storage", "googlestorage.project_id")

diff --git a/src/allmydata/test/test_client.py b/src/allmydata/test/test_client.py
index 199520f1662765fb3caec12c9a8667318d39ce19..298bcdd6c1b16b8b5602a2220c5b833cf92f2745 100644 (file)
--- a/src/allmydata/test/test_client.py
+++ b/src/allmydata/test/test_client.py
@@ -437,11 +437,11 @@ class Basic(testutil.ReallyEqualMixin, unittest.TestCase):
 
     def test_googlestorage_config_required(self):
         """
-        account_email, bucket_name and project_id are all required by
+        account_email, bucket and project_id are all required by
         googlestorage configuration.
         """
         configs = ["googlestorage.account_email = u@example.com",
-                   "googlestorage.bucket_name = bucket",
+                   "googlestorage.bucket = bucket",
                    "googlestorage.project_id = 456"]
         for i in range(len(configs)):
             basedir = self.mktemp()
@@ -470,7 +470,7 @@ class Basic(testutil.ReallyEqualMixin, unittest.TestCase):
                        "enabled = true\n" +
                        "backend = cloud.googlestorage\n" +
                        "googlestorage.account_email = u@example.com\n" +
-                       "googlestorage.bucket_name = bucket\n" +
+                       "googlestorage.bucket = bucket\n" +
                        "googlestorage.project_id = 456\n")
         self.failUnlessRaises(MissingConfigEntry, client.Client, basedir)
 
@@ -489,7 +489,7 @@ class Basic(testutil.ReallyEqualMixin, unittest.TestCase):
                        "enabled = true\n" +
                        "backend = cloud.googlestorage\n" +
                        "googlestorage.account_email = u@example.com\n" +
-                       "googlestorage.bucket_name = bucket\n" +
+                       "googlestorage.bucket = bucket\n" +
                        "googlestorage.project_id = 456\n")
         c = client.Client(basedir)
         server = c.getServiceNamed("storage")