watcher: Wait for child processes by default
authorMichael Hanselmann <hansmi@google.com>
Mon, 22 Aug 2011 15:44:11 +0000 (17:44 +0200)
committerMichael Hanselmann <hansmi@google.com>
Mon, 22 Aug 2011 15:46:22 +0000 (17:46 +0200)
This patch retains the behaviour of ganeti-watcher in previous Ganeti
versions.

Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com>

lib/watcher/__init__.py

index a305fe4..323283b 100644 (file)
@@ -363,8 +363,12 @@ def ParseOptions():
                           " 6 hours)")
   parser.add_option("--ignore-pause", dest="ignore_pause", default=False,
                     action="store_true", help="Ignore cluster pause setting")
-  parser.add_option("--wait-children", dest="wait_children", default=False,
+  parser.add_option("--wait-children", dest="wait_children",
                     action="store_true", help="Wait for child processes")
+  parser.add_option("--no-wait-children", dest="wait_children",
+                    action="store_false", help="Don't wait for child processes")
+  # See optparse documentation for why default values are not set by options
+  parser.set_defaults(wait_children=True)
   options, args = parser.parse_args()
   options.job_age = cli.ParseTimespec(options.job_age)