Merge "Add default limit for container/object"

2 files changed, 21 insertions, 0 deletions

diff --git a/openstackclient/object/v1/container.py b/openstackclient/object/v1/container.py
index f00cc150..2f0d4ac2 100644
--- a/openstackclient/object/v1/container.py
+++ b/openstackclient/object/v1/container.py
@@ -15,11 +15,18 @@
 
 """Container v1 action implementations"""
 
+import logging
+
 from osc_lib.cli import parseractions
 from osc_lib.command import command
 from osc_lib import utils
 import six
 
+from openstackclient.i18n import _
+
+
+LOG = logging.getLogger(__name__)
+
 
 class CreateContainer(command.Lister):
     """Create new container"""
@@ -38,6 +45,10 @@ class CreateContainer(command.Lister):
 
         results = []
         for container in parsed_args.containers:
+            if len(container) > 256:
+                LOG.warning(
+                    _('Container name is %s characters long, the default limit'
+                      ' is 256'), len(container))
             data = self.app.client_manager.object_store.container_create(
                 container=container,
             )
diff --git a/openstackclient/object/v1/object.py b/openstackclient/object/v1/object.py
index 39dba3d5..db61d638 100644
--- a/openstackclient/object/v1/object.py
+++ b/openstackclient/object/v1/object.py
@@ -15,12 +15,18 @@
 
 """Object v1 action implementations"""
 
+import logging
 
 from osc_lib.cli import parseractions
 from osc_lib.command import command
 from osc_lib import utils
 import six
 
+from openstackclient.i18n import _
+
+
+LOG = logging.getLogger(__name__)
+
 
 class CreateObject(command.Lister):
     """Upload object to container"""
@@ -44,6 +50,10 @@ class CreateObject(command.Lister):
 
         results = []
         for obj in parsed_args.objects:
+            if len(obj) > 1024:
+                LOG.warning(
+                    _('Object name is %s characters long, default limit'
+                      ' is 1024'), len(obj))
             data = self.app.client_manager.object_store.object_create(
                 container=parsed_args.container,
                 object=obj,