[ambari] branch branch-feature-AMBARI-14714 updated: [AMBARI-23004] Create initial version of InstanceManager to create instance layout - additional fix (dsen) (#441)

[dsen]

This is an automated email from the ASF dual-hosted git repository.

dsen pushed a commit to branch branch-feature-AMBARI-14714
in repository https://gitbox.apache.org/repos/asf/ambari.git


The following commit(s) were added to refs/heads/branch-feature-AMBARI-14714 by 
this push:
     new d8be60c  [AMBARI-23004] Create initial version of InstanceManager to 
create instance layout - additional fix (dsen) (#441)
d8be60c is described below

commit d8be60c7c869ad52373f411bd5db5ca45f2de358
Author: Dmitry Sen <d...@apache.org>
AuthorDate: Thu Feb 22 17:52:56 2018 +0200

    [AMBARI-23004] Create initial version of InstanceManager to create instance 
layout - additional fix (dsen) (#441)
---
 .../src/main/python/instance_manager/mpack-instance-manager.py         | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/mpack-instance-manager/src/main/python/instance_manager/mpack-instance-manager.py
 
b/mpack-instance-manager/src/main/python/instance_manager/mpack-instance-manager.py
index f5c9451..38c29f8 100644
--- 
a/mpack-instance-manager/src/main/python/instance_manager/mpack-instance-manager.py
+++ 
b/mpack-instance-manager/src/main/python/instance_manager/mpack-instance-manager.py
@@ -56,7 +56,8 @@ def check_required_options(options, action, parser):
     if not options.module_name:
       missing_options.append("module-name")
 
-  parser.error("Missing following required command options: 
{0}".format(missing_options))
+  if missing_options:
+    parser.error("Missing following required command options: 
{0}".format(missing_options))
 
 
 def init_create_parser_options(parser):

-- 
To stop receiving notification emails like this one, please contact
d...@apache.org.