Merge "Warning for empty password set for user create/set"

2 files changed, 20 insertions, 0 deletions

diff --git a/openstackclient/identity/v2_0/user.py b/openstackclient/identity/v2_0/user.py
index d2075150..bc091ce7 100644
--- a/openstackclient/identity/v2_0/user.py
+++ b/openstackclient/identity/v2_0/user.py
@@ -94,6 +94,10 @@ class CreateUser(command.ShowOne):
         if parsed_args.password_prompt:
             parsed_args.password = utils.get_password(self.app.stdin)
 
+        if not parsed_args.password:
+            LOG.warning(_("No password was supplied, authentication will fail "
+                          "when a user does not have a password."))
+
         try:
             user = identity_client.users.create(
                 parsed_args.name,
@@ -292,6 +296,10 @@ class SetUser(command.Command):
         if parsed_args.password_prompt:
             parsed_args.password = utils.get_password(self.app.stdin)
 
+        if '' == parsed_args.password:
+            LOG.warning(_("No password was supplied, authentication will fail "
+                          "when a user does not have a password."))
+
         user = utils.find_resource(
             identity_client.users,
             parsed_args.user,
diff --git a/openstackclient/identity/v3/user.py b/openstackclient/identity/v3/user.py
index dc47ef8d..1e086fb6 100644
--- a/openstackclient/identity/v3/user.py
+++ b/openstackclient/identity/v3/user.py
@@ -110,6 +110,10 @@ class CreateUser(command.ShowOne):
         if parsed_args.password_prompt:
             parsed_args.password = utils.get_password(self.app.stdin)
 
+        if not parsed_args.password:
+            LOG.warning(_("No password was supplied, authentication will fail "
+                          "when a user does not have a password."))
+
         try:
             user = identity_client.users.create(
                 name=parsed_args.name,
@@ -329,6 +333,10 @@ class SetUser(command.Command):
         if parsed_args.password_prompt:
             parsed_args.password = utils.get_password(self.app.stdin)
 
+        if '' == parsed_args.password:
+            LOG.warning(_("No password was supplied, authentication will fail "
+                          "when a user does not have a password."))
+
         user = utils.find_resource(
             identity_client.users,
             parsed_args.user,
@@ -408,6 +416,10 @@ class SetPasswordUser(command.Command):
             password = utils.get_password(
                 self.app.stdin, prompt="New Password:")
 
+        if '' == password:
+            LOG.warning(_("No password was supplied, authentication will fail "
+                          "when a user does not have a password."))
+
         identity_client.users.update_password(current_password, password)