[ARVADOS] updated: 954b101b542b59ba76885c123a64b9fbf55b3c41

Git user git at public.curoverse.com
Mon Feb 27 07:42:19 EST 2017


Summary of changes:
 sdk/python/arvados/commands/put.py | 3 +++
 1 file changed, 3 insertions(+)

       via  954b101b542b59ba76885c123a64b9fbf55b3c41 (commit)
      from  d2a202effce8975313f5fc92fd5dfbb72bd56034 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 954b101b542b59ba76885c123a64b9fbf55b3c41
Author: Lucas Di Pentima <lucas at curoverse.com>
Date:   Mon Feb 27 09:41:17 2017 -0300

    11002: Added note explaining why we're expecting a SystemExit to catch a SIGINT (KeyboardInterrupt)

diff --git a/sdk/python/arvados/commands/put.py b/sdk/python/arvados/commands/put.py
index a6c7d14..32d5fef 100644
--- a/sdk/python/arvados/commands/put.py
+++ b/sdk/python/arvados/commands/put.py
@@ -453,6 +453,9 @@ class ArvPutUploadJob(object):
         except (SystemExit, Exception) as e:
             self._checkpoint_before_quit = False
             # Log stack trace only when Ctrl-C isn't pressed (SIGINT)
+            # Note: We're expecting SystemExit instead of KeyboardInterrupt because
+            #   we have a custom signal handler in place that raises SystemExit with
+            #   the catched signal's code.
             if not isinstance(e, SystemExit) or e.code != -2:
                 self.logger.warning("Abnormal termination:\n{}".format(traceback.format_exc(e)))
             raise

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list