cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jason Bausewein <jason.bausew...@tier3.com>
Subject patch for marvin configGenerator.py
Date Fri, 22 Jun 2012 18:27:38 GMT
Thank you for applying my last patch.  I started using the tools/marvin/marvin directory.

I would like to add an command line option to choose between advanced or basic networking
configuration.  This would allow me or others to choose advanced networking without having
to edit the python code.

Thanks,
Jason

diff --git a/tools/marvin/marvin/configGenerator.py b/tools/marvin/marvin/configGenerator.py
index aec253b..1e84f90 100644
--- a/tools/marvin/marvin/configGenerator.py
+++ b/tools/marvin/marvin/configGenerator.py
@@ -389,8 +389,14 @@ def get_setup_config(file):
 if __name__ == "__main__":
     parser = OptionParser()
   
+    parser.add_option("-a", "--advanced", action="store_true", default=False, dest="advanced",
help="use advanced networking")
     parser.add_option("-o", "--output", action="store", default="./datacenterCfg", dest="output",
help="the path where the json config file generated, by default is ./datacenterCfg")
     
     (options, args) = parser.parse_args()
-    config = describe_setup_in_basic_mode()
+    
+    if options.advanced:
+        config = describe_setup_in_advanced_mode()
+    else:
+        config = describe_setup_in_basic_mode()
+        
     generate_setup_config(config, options.output)

Mime
View raw message