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)

Reply via email to