Merge "Log the traceback in the kolla libraries"

ansible/library/bslurp.py

@@ -109,6 +109,7 @@ Distribute a to file many host without compression; Change permissions on dest:
 import base64
 import hashlib
 import os
+import traceback
 import zlib
 
 
@@ -180,8 +181,9 @@ def main():
             copy_to_host(module)
         else:
             copy_from_host(module)
-    except Exception as e:
-        module.exit_json(failed=True, changed=True, msg=repr(e))
+    except Exception:
+        module.exit_json(failed=True, changed=True,
+                         msg=repr(traceback.format_exc()))
 
 
 # import module snippets

ansible/library/kolla_docker.py

@@ -188,6 +188,7 @@ EXAMPLES = '''
 '''
 
 import os
+import traceback
 
 import docker
 
@@ -728,8 +729,9 @@ def main():
         # meaningful data, we need to refactor all methods to return dicts.
         result = bool(getattr(dw, module.params.get('action'))())
         module.exit_json(changed=dw.changed, result=result)
-    except Exception as e:
-        module.exit_json(failed=True, changed=True, msg=repr(e))
+    except Exception:
+        module.exit_json(failed=True, changed=True,
+                         msg=repr(traceback.format_exc()))
 
 # import module snippets
 from ansible.module_utils.basic import *  # noqa

docker/kolla-toolbox/kolla_keystone_service.py

@@ -22,6 +22,8 @@
 # in upstream shade we will be able to use more of the shade module. Until then
 # if we want to be 'stable' we really need to be using it as a passthrough
 
+import traceback
+
 import shade
 
 
@@ -74,8 +76,9 @@ def main():
                 region=endpoint_region)
 
         module.exit_json(changed=changed)
-    except Exception as e:
-        module.exit_json(failed=True, changed=True, msg=e)
+    except Exception:
+        module.exit_json(failed=True, changed=True,
+                         msg=repr(traceback.format_exc()))
 
 # import module snippets
 from ansible.module_utils.basic import *  # noqa

docker/kolla-toolbox/kolla_keystone_user.py

@@ -22,6 +22,8 @@
 # in upstream shade we will be able to use more of the shade module. Until then
 # if we want to be 'stable' we really need to be using it as a passthrough
 
+import traceback
+
 import shade
 
 
@@ -78,8 +80,9 @@ def main():
                                               project=project)
 
         module.exit_json(changed=changed)
-    except Exception as e:
-        module.exit_json(failed=True, changed=True, msg=e)
+    except Exception:
+        module.exit_json(failed=True, changed=True,
+                         msg=repr(traceback.format_exc()))
 
 # import module snippets
 from ansible.module_utils.basic import *  # noqa

docker/kolla-toolbox/kolla_sanity.py

@@ -22,6 +22,8 @@
 # in upstream shade we will be able to use more of the shade module. Until then
 # if we want to be 'stable' we really need to be using it as a passthrough
 
+import traceback
+
 import shade
 
 
@@ -61,8 +63,9 @@ def main():
         getattr(SanityChecks, module.params.pop("service"))(cloud)
 
         module.exit_json(changed=changed)
-    except Exception as e:
-        module.exit_json(failed=True, changed=True, msg=e)
+    except Exception:
+        module.exit_json(failed=True, changed=True,
+                         msg=repr(traceback.format_exc()))
 
 # import module snippets
 from ansible.module_utils.basic import *  # noqa

docker/kolla-toolbox/kolla_zookeeper.py

@@ -15,6 +15,7 @@
 #    under the License.
 
 import contextlib
+import traceback
 
 import kazoo.client
 import kazoo.exceptions
@@ -56,8 +57,9 @@ def main():
                 changed = True
 
         module.exit_json(changed=changed)
-    except Exception as e:
-        module.exit_json(failed=True, changed=True, msg=e)
+    except Exception:
+        module.exit_json(failed=True, changed=True,
+                         msg=repr(traceback.format_exc()))
 
 
 # import module snippets