Changeset 40898 for branches/ipp-132_automate_jira_conf_backups/backups/backup_atlassian_applications.py

Timestamp:

Oct 3, 2019, 12:46:09 PM (7 years ago)

Author:

fairlamb

Message:

separated out erros into it's own class and moved files to be under one module...not sure if history saved

File:

• branches/ipp-132_automate_jira_conf_backups/backups/backup_atlassian_applications.py (modified) (2 diffs)

branches/ipp-132_automate_jira_conf_backups/backups/backup_atlassian_applications.py

@@ -9 +9 @@
 import sys
 
-# note that this replies on sym link to the 'trunk/tools/utils/python' folder existing
-# in this directory with the name 'utils'. This is not an elegant solution, but
-# a restructure would be needed
-# Ideally, the utils should be added to the path.
-import subprocess_utils as sub_utils
-
+import backups.utils.subprocess_utils as sub_utils
+from backups.utils.errors import ConfigError, SubprocessError
 
 DEFAULT_CONFIG_FILE = path.join(path.dirname(__file__), 'atlassian_backups.config')
-
-
-class ConfigError(Exception):
-    """Error in loading the config file"""
-
-    def __init__(self, config_item, message):
-        self.config_item = config_item
-        self.message = message
 
 
@@ -284 +272 @@
             decoded_stderr = stde.decode()
             if decoded_stderr.casefold().find('error'.casefold()) > -1:
-                raise sub_utils.SubprocessError()
+                raise SubprocessError()
 
         # move the dump file to the correct locations