task/install: extra_system_packages at the top level can be a list

Commit fc5a319 ("task/install: allow both list and dicts for
extra_system_packages") introduced a regression for job definitions
where extra_system_packages at the top level (i.e. on the install
task itself) is a list.  For example

  overrides:
    install:
      ceph:
        flavor: default
        sha1: b7d57fefbec68934ae5e6e7d7c48ac09182397a5
      extra_system_packages:
      - pv

produces AssertionError on

  assert isinstance(b, dict)

in deep_merge() when it's called from _override_extra_system_packages().

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>

Author: idryomov

teuthology/task/install/__init__.py

@@ -427,6 +427,9 @@ def _override_extra_system_packages(config, project, install_overrides):
             e = dict(deb=copy.deepcopy(extra), rpm=copy.deepcopy(extra))
         elif isinstance(extra, dict):
             e = copy.deepcopy(extra)
+
+        if isinstance(extra_overrides, list):
+            extra_overrides = dict(deb=extra_overrides, rpm=extra_overrides)
         elif isinstance(extra, str):
             e = dict(deb=[extra], rpm=[extra])
 

teuthology/test/task/test_install.py

@@ -362,6 +362,28 @@ def test_rh_install_pkgs(self, m_get_pkg_version):
                     'rpm': ['alpha', 'xerxes', 'yellow'],
                 }
             ],
+            [
+                {
+                    'tasks': [ { 'install': { 'clean': True, }, }, ],
+                    'overrides': {
+                        'install': {
+                            'ceph': {
+                                 'extra_system_packages': {
+                                     'deb': [],
+                                     'rpm': ['xerxes', 'yellow'],
+                                 },
+                                 'flavor': 'default',
+                                 'sha1': '0123456789abcdef0123456789abcdef01234567',
+                             },
+                            'extra_system_packages': ['alpha'],
+                        },
+                    },
+                },
+                {
+                    'deb': ['alpha'],
+                    'rpm': ['xerxes', 'yellow', 'alpha'],
+                }
+            ],
             [
                 {
                     'tasks': [ { 'install': { 'clean': True, }, }, ],
@@ -383,6 +405,24 @@ def test_rh_install_pkgs(self, m_get_pkg_version):
                     'rpm': ['xerxes', 'yellow'],
                 }
             ],
+            [
+                {
+                    'tasks': [ { 'install': { 'clean': True, }, }, ],
+                    'overrides': {
+                        'install': {
+                            'ceph': {
+                                 'flavor': 'default',
+                                 'sha1': '0123456789abcdef0123456789abcdef01234567',
+                             },
+                            'extra_system_packages': ['xerxes', 'yellow'],
+                        },
+                    },
+                },
+                {
+                    'deb': ['xerxes', 'yellow'],
+                    'rpm': ['xerxes', 'yellow'],
+                }
+            ],
             [
                 {
                     'tasks': [ { 'install': { 'clean': True, }, }, ],