Remove old m18n.g and errno

locales/en.json

@@ -27,7 +27,6 @@
     "operation_interrupted": "Operation interrupted",
     "password": "Password",
     "pattern_not_match": "Does not match pattern",
-    "permission_denied": "Permission denied",
     "root_required": "You must be root to perform this action",
     "server_already_running": "A server is already running on that port",
     "success": "Success!",

moulinette/actionsmap.py

@@ -2,7 +2,6 @@
 
 import os
 import re
-import errno
 import logging
 import yaml
 import cPickle as pickle
@@ -187,8 +186,8 @@ class PatternParameter(_ExtraParameter):
             if msg == message:
                 msg = m18n.g(message)
 
-            raise MoulinetteError(m18n.g('invalid_argument',
+            raise MoulinetteError('invalid_argument',
-                                         argument=arg_name, error=msg))
+                                  argument=arg_name, error=msg)
         return arg_value
 
     @staticmethod
@@ -217,8 +216,8 @@ class RequiredParameter(_ExtraParameter):
         if required and (arg_value is None or arg_value == ''):
             logger.debug("argument '%s' is required",
                          arg_name)
-            raise MoulinetteError(m18n.g('argument_required',
+            raise MoulinetteError('argument_required',
-                                         argument=arg_name))
+                                  argument=arg_name)
         return arg_value
 
     @staticmethod
@@ -283,7 +282,7 @@ class ExtraArgumentParser(object):
                 except Exception as e:
                     logger.error("unable to validate extra parameter '%s' "
                                  "for argument '%s': %s", p, arg_name, e)
-                    raise MoulinetteError(m18n.g('error_see_log'))
+                    raise MoulinetteError('error_see_log')
 
         return parameters
 
@@ -499,7 +498,7 @@ class ActionsMap(object):
             except (AttributeError, ImportError):
                 logger.exception("unable to load function %s.%s",
                                  namespace, func_name)
-                raise MoulinetteError(m18n.g('error_see_log'))
+                raise MoulinetteError('error_see_log')
             else:
                 log_id = start_action_logging()
                 if logger.isEnabledFor(logging.DEBUG):

moulinette/authenticators/__init__.py

@@ -1,10 +1,8 @@
 # -*- coding: utf-8 -*-
 
-import errno
 import gnupg
 import logging
 
-from moulinette import m18n
 from moulinette.cache import open_cachefile
 from moulinette.core import MoulinetteError
 
@@ -97,7 +95,7 @@ class BaseAuthenticator(object):
             except TypeError:
                 logger.error("unable to extract token parts from '%s'", token)
                 if password is None:
-                    raise MoulinetteError(m18n.g('error_see_log'))
+                    raise MoulinetteError('error_see_log')
 
                 logger.info("session will not be stored")
                 store_session = False
@@ -114,7 +112,7 @@ class BaseAuthenticator(object):
         except:
             logger.exception("authentication (name: '%s', vendor: '%s') fails",
                              self.name, self.vendor)
-            raise MoulinetteError(m18n.g('unable_authenticate'))
+            raise MoulinetteError('unable_authenticate')
 
         # Store session
         if store_session:
@@ -149,7 +147,7 @@ class BaseAuthenticator(object):
                 enc_pwd = f.read()
         except IOError:
             logger.debug("unable to retrieve session", exc_info=1)
-            raise MoulinetteError(m18n.g('unable_retrieve_session'))
+            raise MoulinetteError('unable_retrieve_session')
         else:
             gpg = gnupg.GPG()
             gpg.encoding = 'utf-8'
@@ -158,5 +156,5 @@ class BaseAuthenticator(object):
             if decrypted.ok is not True:
                 logger.error("unable to decrypt password for the session: %s",
                              decrypted.status)
-                raise MoulinetteError(m18n.g('unable_retrieve_session'))
+                raise MoulinetteError('unable_retrieve_session')
             return decrypted.data

moulinette/authenticators/ldap.py

@@ -2,7 +2,6 @@
 
 # TODO: Use Python3 to remove this fix!
 from __future__ import absolute_import
-import errno
 import logging
 import random
 import string
@@ -10,7 +9,6 @@ import crypt
 import ldap
 import ldap.modlist as modlist
 
-from moulinette import m18n
 from moulinette.core import MoulinetteError
 from moulinette.authenticators import BaseAuthenticator
 
@@ -82,10 +80,10 @@ class Authenticator(BaseAuthenticator):
             else:
                 con.simple_bind_s()
         except ldap.INVALID_CREDENTIALS:
-            raise MoulinetteError(m18n.g('invalid_password'))
+            raise MoulinetteError('invalid_password')
         except ldap.SERVER_DOWN:
             logger.exception('unable to reach the server to authenticate')
-            raise MoulinetteError(169, m18n.g('ldap_server_down'))
+            raise MoulinetteError('ldap_server_down')
         else:
             self.con = con
             self._ensure_password_uses_strong_hash(password)
@@ -137,7 +135,7 @@ class Authenticator(BaseAuthenticator):
         except Exception as e:
             logger.exception("error during LDAP search operation with: base='%s', "
                              "filter='%s', attrs=%s and exception %s", base, filter, attrs, e)
-            raise MoulinetteError(169, m18n.g('ldap_operation_error'))
+            raise MoulinetteError('ldap_operation_error')
 
         result_list = []
         if not attrs or 'dn' not in attrs:
@@ -168,7 +166,7 @@ class Authenticator(BaseAuthenticator):
         except Exception as e:
             logger.exception("error during LDAP add operation with: rdn='%s', "
                              "attr_dict=%s and exception %s", rdn, attr_dict, e)
-            raise MoulinetteError(169, m18n.g('ldap_operation_error'))
+            raise MoulinetteError('ldap_operation_error')
         else:
             return True
 
@@ -188,7 +186,7 @@ class Authenticator(BaseAuthenticator):
             self.con.delete_s(dn)
         except Exception as e:
             logger.exception("error during LDAP delete operation with: rdn='%s' and exception %s", rdn, e)
-            raise MoulinetteError(169, m18n.g('ldap_operation_error'))
+            raise MoulinetteError('ldap_operation_error')
         else:
             return True
 
@@ -219,7 +217,7 @@ class Authenticator(BaseAuthenticator):
             logger.exception("error during LDAP update operation with: rdn='%s', "
                              "attr_dict=%s, new_rdn=%s and exception: %s", rdn, attr_dict,
                              new_rdn, e)
-            raise MoulinetteError(169, m18n.g('ldap_operation_error'))
+            raise MoulinetteError('ldap_operation_error')
         else:
             return True
 

moulinette/core.py

@@ -3,7 +3,6 @@
 import os
 import time
 import json
-import errno
 import logging
 import psutil
 
@@ -344,7 +343,7 @@ def init_interface(name, kwargs={}, actionsmap={}):
         mod = import_module('moulinette.interfaces.%s' % name)
     except ImportError:
         logger.exception("unable to load interface '%s'", name)
-        raise MoulinetteError(moulinette.m18n.g('error_see_log'))
+        raise MoulinetteError('error_see_log')
     else:
         try:
             # Retrieve interface classes
@@ -352,7 +351,7 @@ def init_interface(name, kwargs={}, actionsmap={}):
             interface = mod.Interface
         except AttributeError:
             logger.exception("unable to retrieve classes of interface '%s'", name)
-            raise MoulinetteError(moulinette.m18n.g('error_see_log'))
+            raise MoulinetteError('error_see_log')
 
     # Instantiate or retrieve ActionsMap
     if isinstance(actionsmap, dict):
@@ -361,7 +360,7 @@ def init_interface(name, kwargs={}, actionsmap={}):
         amap = actionsmap
     else:
         logger.error("invalid actionsmap value %r", actionsmap)
-        raise MoulinetteError(moulinette.m18n.g('error_see_log'))
+        raise MoulinetteError('error_see_log')
 
     return interface(amap, **kwargs)
 
@@ -382,7 +381,7 @@ def init_authenticator((vendor, name), kwargs={}):
         mod = import_module('moulinette.authenticators.%s' % vendor)
     except ImportError:
         logger.exception("unable to load authenticator vendor '%s'", vendor)
-        raise MoulinetteError(moulinette.m18n.g('error_see_log'))
+        raise MoulinetteError('error_see_log')
     else:
         return mod.Authenticator(name, **kwargs)
 
@@ -474,7 +473,7 @@ class MoulinetteLock(object):
                     break
 
             if self.timeout is not None and (time.time() - start_time) > self.timeout:
-                raise MoulinetteError(moulinette.m18n.g('instance_already_running'))
+                raise MoulinetteError('instance_already_running')
             # Wait before checking again
             time.sleep(self.interval)
 
@@ -497,10 +496,7 @@ class MoulinetteLock(object):
             with open(self._lockfile, 'w') as f:
                 f.write(str(os.getpid()))
         except IOError:
-            raise MoulinetteError(
+            raise MoulinetteError('root_required')
-                errno.EPERM, '%s. %s.'.format(
-                    moulinette.m18n.g('permission_denied'),
-                    moulinette.m18n.g('root_required')))
 
     def _lock_PIDs(self):
 

moulinette/interfaces/__init__.py

@@ -1,7 +1,6 @@
 # -*- coding: utf-8 -*-
 
 import re
-import errno
 import logging
 import argparse
 import copy
@@ -142,7 +141,7 @@ class BaseActionsMapParser(object):
         # Validate tid and namespace
         if not isinstance(tid, tuple) and \
                 (namespace is None or not hasattr(namespace, TO_RETURN_PROP)):
-            raise MoulinetteError(m18n.g('invalid_usage'))
+            raise MoulinetteError('invalid_usage')
         elif not tid:
             tid = GLOBAL_SECTION
 
@@ -158,7 +157,7 @@ class BaseActionsMapParser(object):
             # TODO: Catch errors
             auth = msignals.authenticate(cls(), **auth_conf)
             if not auth.is_authenticated:
-                raise MoulinetteError(m18n.g('authentication_required_long'))
+                raise MoulinetteError('authentication_required_long')
             if self.get_conf(tid, 'argument_auth') and \
                     self.get_conf(tid, 'authenticate') == 'all':
                 namespace.auth = auth
@@ -262,7 +261,7 @@ class BaseActionsMapParser(object):
             else:
                 logger.error("expecting 'all', 'False' or a list for "
                              "configuration 'authenticate', got %r", ifaces)
-                raise MoulinetteError(m18n.g('error_see_log'))
+                raise MoulinetteError('error_see_log')
 
         # -- 'authenticator'
         try:
@@ -277,7 +276,7 @@ class BaseActionsMapParser(object):
                 except KeyError:
                     logger.error("requesting profile '%s' which is undefined in "
                                  "global configuration of 'authenticator'", auth)
-                    raise MoulinetteError(m18n.g('error_see_log'))
+                    raise MoulinetteError('error_see_log')
             elif is_global and isinstance(auth, dict):
                 if len(auth) == 0:
                     logger.warning('no profile defined in global configuration '
@@ -300,7 +299,7 @@ class BaseActionsMapParser(object):
             else:
                 logger.error("expecting a dict of profile(s) or a profile name "
                              "for configuration 'authenticator', got %r", auth)
-                raise MoulinetteError(m18n.g('error_see_log'))
+                raise MoulinetteError('error_see_log')
 
         # -- 'argument_auth'
         try:
@@ -313,7 +312,7 @@ class BaseActionsMapParser(object):
             else:
                 logger.error("expecting a boolean for configuration "
                              "'argument_auth', got %r", arg_auth)
-                raise MoulinetteError(m18n.g('error_see_log'))
+                raise MoulinetteError('error_see_log')
 
         # -- 'lock'
         try:
@@ -326,7 +325,7 @@ class BaseActionsMapParser(object):
             else:
                 logger.error("expecting a boolean for configuration 'lock', "
                              "got %r", lock)
-                raise MoulinetteError(m18n.g('error_see_log'))
+                raise MoulinetteError('error_see_log')
 
         return conf
 
@@ -426,7 +425,7 @@ class _CallbackAction(argparse.Action):
         except:
             logger.exception("cannot get value from callback method "
                 "'{0}'".format(self.callback_method))
-            raise MoulinetteError(m18n.g('error_see_log'))
+            raise MoulinetteError('error_see_log')
         else:
             if value:
                 if self.callback_return:

moulinette/interfaces/api.py

@@ -189,7 +189,7 @@ class _HTTPArgumentParser(object):
 
     def _error(self, message):
         # TODO: Raise a proper exception
-        raise MoulinetteError(1, message)
+        raise MoulinetteError(message)
 
 
 class _ActionsMapPlugin(object):
@@ -660,7 +660,7 @@ class ActionsMapParser(BaseActionsMapParser):
             tid, parser = self._parsers[route]
         except KeyError:
             logger.error("no argument parser found for route '%s'", route)
-            raise MoulinetteError(m18n.g('error_see_log'))
+            raise MoulinetteError('error_see_log')
         ret = argparse.Namespace()
 
         # Perform authentication if needed
@@ -673,7 +673,7 @@ class ActionsMapParser(BaseActionsMapParser):
             # TODO: Catch errors
             auth = msignals.authenticate(klass(), **auth_conf)
             if not auth.is_authenticated:
-                raise MoulinetteError(m18n.g('authentication_required_long'))
+                raise MoulinetteError('authentication_required_long')
             if self.get_conf(tid, 'argument_auth') and \
                self.get_conf(tid, 'authenticate') == 'all':
                 ret.auth = auth
@@ -796,8 +796,8 @@ class Interface(BaseInterface):
             logger.exception("unable to start the server instance on %s:%d",
                              host, port)
             if e.args[0] == errno.EADDRINUSE:
-                raise MoulinetteError(m18n.g('server_already_running'))
+                raise MoulinetteError('server_already_running')
-            raise MoulinetteError(m18n.g('error_see_log'))
+            raise MoulinetteError('error_see_log')
 
     # Routes handlers
 

moulinette/interfaces/cli.py

@@ -2,7 +2,6 @@
 
 import os
 import sys
-import errno
 import getpass
 import locale
 import logging
@@ -362,7 +361,7 @@ class ActionsMapParser(BaseActionsMapParser):
             raise
         except:
             logger.exception("unable to parse arguments '%s'", ' '.join(args))
-            raise MoulinetteError(m18n.g('error_see_log'))
+            raise MoulinetteError('error_see_log')
         else:
             self.prepare_action_namespace(getattr(ret, '_tid', None), ret)
             self._parser.dequeue_callbacks(ret)
@@ -409,7 +408,7 @@ class Interface(BaseInterface):
 
         """
         if output_as and output_as not in ['json', 'plain', 'none']:
-            raise MoulinetteError(m18n.g('invalid_usage'))
+            raise MoulinetteError('invalid_usage')
 
         # auto-complete
         argcomplete.autocomplete(self.actionsmap.parser._parser)
@@ -422,7 +421,7 @@ class Interface(BaseInterface):
         try:
             ret = self.actionsmap.process(args, timeout=timeout)
         except (KeyboardInterrupt, EOFError):
-            raise MoulinetteError(m18n.g('operation_interrupted'))
+            raise MoulinetteError('operation_interrupted')
 
         if ret is None or output_as == 'none':
             return
@@ -472,7 +471,7 @@ class Interface(BaseInterface):
         if confirm:
             m = message[0].lower() + message[1:]
             if prompt(m18n.g('confirm', prompt=m)) != value:
-                raise MoulinetteError(m18n.g('values_mismatch'))
+                raise MoulinetteError('values_mismatch')
 
         return value
 

moulinette/utils/filesystem.py

@@ -23,16 +23,16 @@ def read_file(file_path):
 
     # Check file exists
     if not os.path.isfile(file_path):
-        raise MoulinetteError(m18n.g('file_not_exist', path=file_path))
+        raise MoulinetteError('file_not_exist', path=file_path)
 
     # Open file and read content
     try:
         with open(file_path, "r") as f:
             file_content = f.read()
     except IOError as e:
-        raise MoulinetteError(m18n.g('cannot_open_file', file=file_path, error=str(e)))
+        raise MoulinetteError('cannot_open_file', file=file_path, error=str(e))
     except Exception as e:
-        raise MoulinetteError(m18n.g('error_reading_file', file=file_path, error=str(e)))
+        raise MoulinetteError('error_reading_file', file=file_path, error=str(e))
 
     return file_content
 
@@ -52,7 +52,7 @@ def read_json(file_path):
     try:
         loaded_json = json.loads(file_content)
     except ValueError as e:
-        raise MoulinetteError(m18n.g('corrupted_json', ressource=file_path, error=str(e)))
+        raise MoulinetteError('corrupted_json', ressource=file_path, error=str(e))
 
     return loaded_json
 
@@ -72,7 +72,7 @@ def read_yaml(file_path):
     try:
         loaded_yaml = yaml.safe_load(file_content)
     except ValueError as e:
-        raise MoulinetteError(m18n.g('corrupted_yaml', ressource=file_path, error=str(e)))
+        raise MoulinetteError('corrupted_yaml', ressource=file_path, error=str(e))
 
     return loaded_yaml
 
@@ -102,9 +102,9 @@ def write_to_file(file_path, data, file_mode="w"):
         with open(file_path, file_mode) as f:
             f.write(data)
     except IOError as e:
-        raise MoulinetteError(m18n.g('cannot_write_file', file=file_path, error=str(e)))
+        raise MoulinetteError('cannot_write_file', file=file_path, error=str(e))
     except Exception as e:
-        raise MoulinetteError(m18n.g('error_writing_file', file=file_path, error=str(e)))
+        raise MoulinetteError('error_writing_file', file=file_path, error=str(e))
 
 
 def append_to_file(file_path, data):
@@ -139,9 +139,9 @@ def write_to_json(file_path, data):
         with open(file_path, "w") as f:
             json.dump(data, f)
     except IOError as e:
-        raise MoulinetteError(m18n.g('cannot_write_file', file=file_path, error=str(e)))
+        raise MoulinetteError('cannot_write_file', file=file_path, error=str(e))
     except Exception as e:
-        raise MoulinetteError(m18n.g('_error_writing_file', file=file_path, error=str(e)))
+        raise MoulinetteError('_error_writing_file', file=file_path, error=str(e))
 
 
 def mkdir(path, mode=0777, parents=False, uid=None, gid=None, force=False):
@@ -206,14 +206,14 @@ def chown(path, uid=None, gid=None, recursive=False):
         try:
             uid = getpwnam(uid).pw_uid
         except KeyError:
-            raise MoulinetteError(m18n.g('unknown_user', user=uid))
+            raise MoulinetteError('unknown_user', user=uid)
     elif uid is None:
         uid = -1
     if isinstance(gid, basestring):
         try:
             gid = grp.getgrnam(gid).gr_gid
         except KeyError:
-            raise MoulinetteError(m18n.g('unknown_group', group=gid))
+            raise MoulinetteError('unknown_group', group=gid)
     elif gid is None:
         gid = -1
 
@@ -226,7 +226,7 @@ def chown(path, uid=None, gid=None, recursive=False):
                 for f in files:
                     os.chown(os.path.join(root, f), uid, gid)
     except Exception as e:
-        raise MoulinetteError(m18n.g('error_changing_file_permissions', path=path, error=str(e)))
+        raise MoulinetteError('error_changing_file_permissions', path=path, error=str(e))
 
 
 def chmod(path, mode, fmode=None, recursive=False):
@@ -250,7 +250,7 @@ def chmod(path, mode, fmode=None, recursive=False):
                 for f in files:
                     os.chmod(os.path.join(root, f), fmode)
     except Exception as e:
-        raise MoulinetteError(m18n.g('error_changing_file_permissions', path=path, error=str(e)))
+        raise MoulinetteError('error_changing_file_permissions', path=path, error=str(e))
 
 
 def rm(path, recursive=False, force=False):
@@ -269,4 +269,4 @@ def rm(path, recursive=False, force=False):
             os.remove(path)
         except OSError as e:
             if not force:
-                raise MoulinetteError(m18n.g('error_removing', path=path, error=str(e)))
+                raise MoulinetteError('error_removing', path=path, error=str(e))

moulinette/utils/network.py

@@ -1,7 +1,5 @@
-import errno
 import json
 
-from moulinette import m18n
 from moulinette.core import MoulinetteError
 
 
@@ -25,22 +23,22 @@ def download_text(url, timeout=30, expected_status_code=200):
         r = requests.get(url, timeout=timeout)
     # Invalid URL
     except requests.exceptions.ConnectionError:
-        raise MoulinetteError(m18n.g('invalid_url', url=url))
+        raise MoulinetteError('invalid_url', url=url)
     # SSL exceptions
     except requests.exceptions.SSLError:
-        raise MoulinetteError(m18n.g('download_ssl_error', url=url))
+        raise MoulinetteError('download_ssl_error', url=url)
     # Timeout exceptions
     except requests.exceptions.Timeout:
-        raise MoulinetteError(m18n.g('download_timeout', url=url))
+        raise MoulinetteError('download_timeout', url=url)
     # Unknown stuff
     except Exception as e:
-        raise MoulinetteError(m18n.g('download_unknown_error',
+        raise MoulinetteError('download_unknown_error',
-                                     url=url, error=str(e)))
+                              url=url, error=str(e))
     # Assume error if status code is not 200 (OK)
     if expected_status_code is not None \
        and r.status_code != expected_status_code:
-        raise MoulinetteError(m18n.g('download_bad_status_code',
+        raise MoulinetteError('download_bad_status_code',
-                                     url=url, code=str(r.status_code)))
+                              url=url, code=str(r.status_code))
 
     return r.text
 
@@ -61,6 +59,6 @@ def download_json(url, timeout=30, expected_status_code=200):
     try:
         loaded_json = json.loads(text)
     except ValueError:
-        raise MoulinetteError(m18n.g('corrupted_json', ressource=url))
+        raise MoulinetteError('corrupted_json', ressource=url)
 
     return loaded_json