rename main to _main to avoid fixing scripts

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

diff --git a/ceph_deploy/cli.py b/ceph_deploy/cli.py
index a6dfead8df42b95e2b204278ddf73aa0dadeb29c..2c7da8519903ef47fceab9dac0a11f03c89fc4b6 100644 (file)
--- a/ceph_deploy/cli.py
+++ b/ceph_deploy/cli.py
@@ -99,7 +99,7 @@ def get_parser():
 
 
 @catches((KeyboardInterrupt, RuntimeError, exc.DeployError,), handle_all=True)
-def main(args=None, namespace=None):
+def _main(args=None, namespace=None):
     parser = get_parser()
 
     if len(sys.argv) < 2:
@@ -147,3 +147,21 @@ def main(args=None, namespace=None):
     )
 
     return args.func(args)
+
+
+def main(args=None, namespace=None):
+    try:
+        _main(args=None, namespace=None)
+    finally:
+        # This block is crucial to avoid having issues with
+        # Python spitting non-sense thread exceptions. We have already
+        # handled what we could, so close stderr and stdout.
+        if not os.environ.get('CEPH_DEPLOY_TEST'):
+            try:
+                sys.stdout.close()
+            except:
+                pass
+            try:
+                sys.stderr.close()
+            except:
+                pass