CE-575 Fix Python error when raising an Exception with traceback.

PR URL: https://www.github.com/delphix/couchbase-plugin/pull/62

Author: satishk1536

src/operations/discovery.py

@@ -8,7 +8,6 @@
 """
 ##############################################################################
 import logging
-import sys
 
 from controller import helper_lib
 from generated.definitions import RepositoryDefinition
@@ -59,7 +58,8 @@ def find_repos(source_connection):
 
         return repositories
     except RepositoryDiscoveryError as err:
-        raise err.to_user_error()(None).with_traceback(sys.exc_info()[2])
+        logger.exception(err)
+        raise err.to_user_error()
     except Exception as err:
         logger.debug("find_repos: Caught unexpected exception!" + str(err))
         raise
@@ -102,7 +102,8 @@ def find_source(source_connection, repository):
             # # Couchbase supports only 1 instance on server so that instance
             # on host should be managed by delphix
     except SourceConfigDiscoveryError as err:
-        raise err.to_user_error()(None).with_traceback(sys.exc_info()[2])
+        logger.exception(err)
+        raise err.to_user_error()
     except Exception as err:
         logger.debug("find_source: Caught unexpected exception!" + str(err))
         raise

src/operations/linked.py

@@ -6,7 +6,6 @@
 #############################################################################
 
 import logging
-import sys
 
 import db_commands
 from controller import helper_lib
@@ -42,7 +41,8 @@ def resync(staged_source, repository, source_config, input_parameters):
             )
 
         logger.debug("Completed resynchronization")
-    except UserError:
+    except UserError as ex_obj:
+        logger.exception(ex_obj)
         raise
 
     except Exception as ex_obj:
@@ -56,14 +56,13 @@ def resync(staged_source, repository, source_config, input_parameters):
             link_cbbkpmgr.unmount_file_system_in_error_case(
                 staged_source, repository, source_config
             )
+        logger.exception(ex_obj)
         if (
             isinstance(ex_obj, PluginException)
             or isinstance(ex_obj, DatabaseException)
             or isinstance(ex_obj, GenericUserError)
         ):
-            raise ex_obj.to_user_error()(None).with_traceback(
-                sys.exc_info()[2],
-            )
+            raise ex_obj.to_user_error()
         raise
 
 
@@ -88,14 +87,13 @@ def pre_snapshot(staged_source, repository, source_config, input_parameters):
             link_cbbkpmgr.unmount_file_system_in_error_case(
                 staged_source, repository, source_config
             )
+        logger.exception(ex_obj)
         if (
             isinstance(ex_obj, PluginException)
             or isinstance(ex_obj, DatabaseException)
             or isinstance(ex_obj, GenericUserError)
         ):
-            raise ex_obj.to_user_error()(None).with_traceback(
-                sys.exc_info()[2],
-            )
+            raise ex_obj.to_user_error()
         raise
 
 

src/operations/virtual.py

@@ -10,7 +10,6 @@
 
 import json
 import logging
-import sys
 import time
 
 from controller import helper_lib
@@ -465,10 +464,9 @@ def vdb_start(virtual_source, repository, source_config):
                     ),
                 )
                 addnode.start_couchbase()
-    except Exception:
-        raise CouchbaseServicesError(" Start").to_user_error()(
-            None
-        ).with_traceback(sys.exc_info()[2])
+    except Exception as ex_obj:
+        logger.exception(ex_obj)
+        raise CouchbaseServicesError(" Start").to_user_error()
 
 
 def vdb_stop(virtual_source, repository, source_config):