Merge pull request #148 from timvaillancourt/review_fixes_1.0.2

Review fixes 1.0.2

Author: timvaillancourt

mongodb_consistent_backup/Common/Config.py

@@ -51,7 +51,7 @@ def makeParser(self):
         parser.add_argument("-v", "--verbose", dest="verbose", help="Verbose output", default=False, action="store_true")
         parser.add_argument("-H", "--host", dest="host", help="MongoDB Hostname, IP address or '<replset>/<host:port>,<host:port>,..' URI (default: localhost)", default="localhost", type=str)
         parser.add_argument("-P", "--port", dest="port", help="MongoDB Port (default: 27017)", default=27017, type=int)
-        parser.add_argument("-u", "--username", dest="username", help="MongoDB Authentication Username (for optional auth)", type=str)
+        parser.add_argument("-u", "--user", "--username", dest="username", help="MongoDB Authentication Username (for optional auth)", type=str)
         parser.add_argument("-p", "--password", dest="password", help="MongoDB Authentication Password (for optional auth)", type=str)
         parser.add_argument("-a", "--authdb", dest="authdb", help="MongoDB Auth Database (for optional auth - default: admin)", default='admin', type=str)
         parser.add_argument("-L", "--log-dir", dest="log_dir", help="Path to write log files to (default: disabled)", default='', type=str)

mongodb_consistent_backup/Logger.py

@@ -18,6 +18,7 @@ def __init__(self, config, backup_time):
         if self.config.log_dir is not '':
             self.do_file_log = True
             if not os.path.isdir(self.config.log_dir):
+                print "WARNING: Creating logging directory: %s" % self.config.log_dir
                 os.mkdir(self.config.log_dir)
 
         self.log_format = '[%(asctime)s] [%(levelname)s] [%(processName)s] [%(module)s:%(funcName)s:%(lineno)d] %(message)s'