]> git-server-git.apps.pok.os.sepia.ceph.com Git - teuthology.git/commit
task/install: extra_system_packages at the top level can be a list
authorIlya Dryomov <idryomov@gmail.com>
Tue, 4 Nov 2025 12:02:46 +0000 (13:02 +0100)
committerIlya Dryomov <idryomov@gmail.com>
Tue, 4 Nov 2025 12:15:54 +0000 (13:15 +0100)
commit46012fc980da2a3772b31c964a8f20c83fd102aa
tree6ca598c4644f6fcc0d9e6c0dedc79f29b57b5019
parent196839919f79cf9ec6de99cb8141f08ecfc93f3a
task/install: extra_system_packages at the top level can be a list

Commit fc5a3197f22e ("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>
teuthology/task/install/__init__.py
teuthology/test/task/test_install.py