Address review comments from Daira.
authorItamar Turner-Trauring <itamar@futurefoundries.com>
Thu, 18 Apr 2013 13:04:33 +0000 (09:04 -0400)
committerDaira Hopwood <daira@jacaranda.org>
Fri, 10 Jul 2015 04:28:15 +0000 (05:28 +0100)
1. Fix typo.
2. Rename config item googlestorage.bucket_name to googlestorage.bucket for
   consistency.

docs/backends/cloud.rst
src/allmydata/storage/backends/cloud/googlestorage/googlestorage_container.py
src/allmydata/test/test_client.py

index e8b258fbad30ffb2ece27858c9d548685a93f44e..c580da658c88911d3732be68922adb0b605440e1 100644 (file)
@@ -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
index 05f2f17e657f933c64e67dc2914c89a1aa362932..e06e754e9b87cfb1a2a6e35e8869ffe588a6d9eb 100644 (file)
@@ -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")
index 55118698b01e13e54c7ab1af3037d41143ff919e..9703afdefc1806f397030e6f21780989c54f067b 100644 (file)
@@ -432,11 +432,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()
@@ -465,7 +465,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)
 
@@ -484,7 +484,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")