Merge lp:~corey.bryant/charms/trusty/swift-storage/mitaka-ch-sync into lp:~openstack-charmers-archive/charms/trusty/swift-storage/next

[Corey Bryant]

[Corey Bryant (corey.bryant)]

I have some notes in the charm-helper mp that may be of interest: https://code.launchpad.net/~corey.bryant/charm-helpers/swift-mitaka/+merge/282337

[uosci-testing-bot (uosci-testing-bot)]

charm_lint_check #17253 swift-storage-next for corey.bryant mp282494
    LINT OK: passed

Build: http://10.245.162.77:8080/job/charm_lint_check/17253/

[uosci-testing-bot (uosci-testing-bot)]

charm_unit_test #16118 swift-storage-next for corey.bryant mp282494
    UNIT OK: passed

Build: http://10.245.162.77:8080/job/charm_unit_test/16118/

[uosci-testing-bot (uosci-testing-bot)]

charm_amulet_test #8760 swift-storage-next for corey.bryant mp282494
    AMULET OK: passed

Build: http://10.245.162.77:8080/job/charm_amulet_test/8760/

=== modified file 'charmhelpers/contrib/openstack/utils.py'
--- charmhelpers/contrib/openstack/utils.py	2016-01-08 02:38:26 +0000
+++ charmhelpers/contrib/openstack/utils.py	2016-01-13 19:15:29 +0000
@@ -103,29 +103,18 @@
     ('2016.1', 'mitaka'),
 ])
 
-# The ugly duckling
+# The ugly duckling - must list releases oldest to newest
 SWIFT_CODENAMES = OrderedDict([
-    ('1.4.3', 'diablo'),
-    ('1.4.8', 'essex'),
-    ('1.7.4', 'folsom'),
-    ('1.8.0', 'grizzly'),
-    ('1.7.7', 'grizzly'),
-    ('1.7.6', 'grizzly'),
-    ('1.10.0', 'havana'),
-    ('1.9.1', 'havana'),
-    ('1.9.0', 'havana'),
-    ('1.13.1', 'icehouse'),
-    ('1.13.0', 'icehouse'),
-    ('1.12.0', 'icehouse'),
-    ('1.11.0', 'icehouse'),
-    ('2.0.0', 'juno'),
-    ('2.1.0', 'juno'),
-    ('2.2.0', 'juno'),
-    ('2.2.1', 'kilo'),
-    ('2.2.2', 'kilo'),
-    ('2.3.0', 'liberty'),
-    ('2.4.0', 'liberty'),
-    ('2.5.0', 'liberty'),
+    ('diablo',   ['1.4.3']),
+    ('essex',    ['1.4.8']),
+    ('folsom',   ['1.7.4']),
+    ('grizzly',  ['1.7.6', '1.7.7', '1.8.0']),
+    ('havana',   ['1.9.0', '1.9.1', '1.10.0']),
+    ('icehouse', ['1.11.0', '1.12.0', '1.13.0', '1.13.1']),
+    ('juno',     ['2.0.0', '2.1.0', '2.2.0']),
+    ('kilo',     ['2.2.1', '2.2.2']),
+    ('liberty',  ['2.3.0', '2.4.0', '2.5.0']),
+    ('mitaka',   ['2.5.0']),
 ])
 
 # >= Liberty version->codename mapping
@@ -227,6 +216,33 @@
     error_out(e)
 
 
+def get_os_version_codename_swift(codename):
+    '''Determine OpenStack version number of swift from codename.'''
+    for k, v in six.iteritems(SWIFT_CODENAMES):
+        if k == codename:
+            return v[-1]
+    e = 'Could not derive swift version for '\
+        'codename: %s' % codename
+    error_out(e)
+
+
+def get_swift_codename(version):
+    '''Determine OpenStack codename that corresponds to swift version.'''
+    codenames = [k for k, v in six.iteritems(SWIFT_CODENAMES) if version in v]
+    if len(codenames) > 1:
+        # If more than one release codename contains this version we determine
+        # the actual codename based on the highest available install source.
+        for codename in reversed(codenames):
+            releases = UBUNTU_OPENSTACK_RELEASE
+            release = [k for k, v in six.iteritems(releases) if codename in v]
+            ret = subprocess.check_output(['apt-cache', 'policy', 'swift'])
+            if codename in ret or release[0] in ret:
+                return codename
+    elif len(codenames) == 1:
+        return codenames[0]
+    return None
+
+
 def get_os_codename_package(package, fatal=True):
     '''Derive OpenStack release codename from an installed package.'''
     import apt_pkg as apt
@@ -270,7 +286,7 @@
         # < Liberty co-ordinated project versions
         try:
             if 'swift' in pkg.name:
-                return SWIFT_CODENAMES[vers]
+                return get_swift_codename(vers)
             else:
                 return OPENSTACK_CODENAMES[vers]
         except KeyError:
@@ -289,12 +305,14 @@
 
     if 'swift' in pkg:
         vers_map = SWIFT_CODENAMES
+        for cname, version in six.iteritems(vers_map):
+            if cname == codename:
+                return version[-1]
     else:
         vers_map = OPENSTACK_CODENAMES
-
-    for version, cname in six.iteritems(vers_map):
-        if cname == codename:
-            return version
+        for version, cname in six.iteritems(vers_map):
+            if cname == codename:
+                return version
     # e = "Could not determine OpenStack version for package: %s" % pkg
     # error_out(e)
 
@@ -460,11 +478,16 @@
     cur_vers = get_os_version_package(package)
     if "swift" in package:
         codename = get_os_codename_install_source(src)
-        available_vers = get_os_version_codename(codename, SWIFT_CODENAMES)
+        avail_vers = get_os_version_codename_swift(codename)
     else:
-        available_vers = get_os_version_install_source(src)
+        avail_vers = get_os_version_install_source(src)
     apt.init()
-    return apt.version_compare(available_vers, cur_vers) == 1
+    if "swift" in package:
+        major_cur_vers = cur_vers.split('.', 1)[0]
+        major_avail_vers = avail_vers.split('.', 1)[0]
+        major_diff = apt.version_compare(major_avail_vers, major_cur_vers)
+        return avail_vers > cur_vers and (major_diff == 1 or major_diff == 0)
+    return apt.version_compare(avail_vers, cur_vers) == 1
 
 
 def ensure_block_device(block_device):
 
=== modified file 'charmhelpers/fetch/giturl.py'
--- charmhelpers/fetch/giturl.py	2016-01-08 02:38:26 +0000
+++ charmhelpers/fetch/giturl.py	2016-01-13 19:15:29 +0000
@@ -49,8 +49,8 @@
             cmd = ['git', '-C', dest, 'pull', source, branch]
         else:
             cmd = ['git', 'clone', source, dest, '--branch', branch]
-        if depth:
-            cmd.extend(['--depth', depth])
+            if depth:
+                cmd.extend(['--depth', depth])
         check_call(cmd)
 
     def install(self, source, branch="master", dest=None, depth=None):