Exit as failure when tap is running or not enabled (#546)

transferwise · Oct 2, 2020 · dfb8544 · dfb8544
1 parent 64a19fe
commit dfb8544
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/pipelinewise/cli/pipelinewise.py b/pipelinewise/cli/pipelinewise.py
@@ -898,14 +898,14 @@ def run_tap(self):
 
         # Run only if tap enabled
         if not self.tap.get('enabled', False):
-            self.logger.info('Tap %s is not enabled. Do nothing and exit normally.', self.tap['name'])
-            sys.exit(0)
+            self.logger.info('Tap %s is not enabled.', self.tap['name'])
+            sys.exit(1)
 
         # Run only if not running
         tap_status = self.detect_tap_status(target_id, tap_id)
         if tap_status['currentStatus'] == 'running':
-            self.logger.info('Tap %s is currently running. Do nothing and exit normally.', self.tap['name'])
-            sys.exit(0)
+            self.logger.info('Tap %s is currently running.', self.tap['name'])
+            sys.exit(1)
 
         # Generate and run the command to run the tap directly
         tap_config = self.tap['files']['config']
@@ -1051,8 +1051,8 @@ def sync_tables(self):
 
         # Run only if tap enabled
         if not self.tap.get('enabled', False):
-            self.logger.info('Tap %s is not enabled. Do nothing and exit normally.', self.tap['name'])
-            sys.exit(0)
+            self.logger.info('Tap %s is not enabled.', self.tap['name'])
+            sys.exit(1)
 
         # Run only if tap not running
         tap_status = self.detect_tap_status(target_id, tap_id)