]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Remove unused methods from cinder.utils
authorVladislav Kuzmin <vkuzmin@mirantis.com>
Thu, 22 Aug 2013 12:15:45 +0000 (16:15 +0400)
committerVladislav Kuzmin <vkuzmin@mirantis.com>
Thu, 22 Aug 2013 12:43:46 +0000 (16:43 +0400)
These methods were left over from the split from Nova and are unused still.

Remove methods:
fetchfile()
trycmd()
generate_uid()
last_octet()
get_my_linklocal()
parse_mailmap()
str_dict_replace()
flatten_dict()

Change-Id: I5d41dec66246814bb1edb6f4993d5307587a00f9

cinder/utils.py

index 08d1ff57aa4fde0cafa035155b3fe61ba22e1609..51db310e89e9f511f923fb2424ea40ff62297f1d 100644 (file)
@@ -136,11 +136,6 @@ def check_exclusive_options(**kwargs):
         raise exception.InvalidInput(reason=msg)
 
 
-def fetchfile(url, target):
-    LOG.debug(_('Fetching %s') % url)
-    execute('curl', '--fail', url, '-o', target)
-
-
 def execute(*cmd, **kwargs):
     """Convenience wrapper around oslo's execute() method."""
     if 'run_as_root' in kwargs and not 'root_helper' in kwargs:
@@ -159,24 +154,6 @@ def execute(*cmd, **kwargs):
     return (stdout, stderr)
 
 
-def trycmd(*args, **kwargs):
-    """Convenience wrapper around oslo's trycmd() method."""
-    if 'run_as_root' in kwargs and not 'root_helper' in kwargs:
-        kwargs['root_helper'] = get_root_helper()
-    try:
-        (stdout, stderr) = processutils.trycmd(*args, **kwargs)
-    except processutils.ProcessExecutionError as ex:
-        raise exception.ProcessExecutionError(
-            exit_code=ex.exit_code,
-            stderr=ex.stderr,
-            stdout=ex.stdout,
-            cmd=ex.cmd,
-            description=ex.description)
-    except processutils.UnknownArgumentError as ex:
-        raise exception.Error(ex.message)
-    return (stdout, stderr)
-
-
 def check_ssh_injection(cmd_list):
     ssh_injection_pattern = ['`', '$', '|', '||', ';', '&', '&&', '>', '>>',
                              '<']
@@ -295,12 +272,6 @@ def debug(arg):
     return arg
 
 
-def generate_uid(topic, size=8):
-    characters = '01234567890abcdefghijklmnopqrstuvwxyz'
-    choices = [random.choice(characters) for x in xrange(size)]
-    return '%s-%s' % (topic, ''.join(choices))
-
-
 # Default symbols to use for passwords. Avoids visually confusing characters.
 # ~6 bits per symbol
 DEFAULT_PASSWORD_SYMBOLS = ('23456789',  # Removed: 0,1
@@ -438,45 +409,6 @@ def generate_username(length=20, symbolgroups=DEFAULT_PASSWORD_SYMBOLS):
     return generate_password(length, symbolgroups)
 
 
-def last_octet(address):
-    return int(address.split('.')[-1])
-
-
-def get_my_linklocal(interface):
-    try:
-        if_str = execute('ip', '-f', 'inet6', '-o', 'addr', 'show', interface)
-        condition = '\s+inet6\s+([0-9a-f:]+)/\d+\s+scope\s+link'
-        links = [re.search(condition, x) for x in if_str[0].split('\n')]
-        address = [w.group(1) for w in links if w is not None]
-        if address[0] is not None:
-            return address[0]
-        else:
-            raise exception.Error(_('Link Local address is not found.:%s')
-                                  % if_str)
-    except Exception as ex:
-        raise exception.Error(_("Couldn't get Link Local IP of %(interface)s"
-                                " :%(ex)s") %
-                              {'interface': interface, 'ex': ex, })
-
-
-def parse_mailmap(mailmap='.mailmap'):
-    mapping = {}
-    if os.path.exists(mailmap):
-        fp = open(mailmap, 'r')
-        for l in fp:
-            l = l.strip()
-            if not l.startswith('#') and ' ' in l:
-                canonical_email, alias = l.split(' ')
-                mapping[alias.lower()] = canonical_email.lower()
-    return mapping
-
-
-def str_dict_replace(s, mapping):
-    for s1, s2 in mapping.iteritems():
-        s = s.replace(s1, s2)
-    return s
-
-
 class LazyPluggable(object):
     """A pluggable backend loaded lazily based on some value."""
 
@@ -680,17 +612,6 @@ def get_from_path(items, path):
         return get_from_path(results, remainder)
 
 
-def flatten_dict(dict_, flattened=None):
-    """Recursively flatten a nested dictionary."""
-    flattened = flattened or {}
-    for key, value in dict_.iteritems():
-        if hasattr(value, 'iteritems'):
-            flatten_dict(value, flattened)
-        else:
-            flattened[key] = value
-    return flattened
-
-
 def map_dict_keys(dict_, key_map):
     """Return a dict in which the dictionaries keys are mapped to new keys."""
     mapped = {}