Skip to content

Commit

Permalink
qa/cephfs: override testing kernel with -k option
Browse files Browse the repository at this point in the history
Normally, the fs suite overrides the kernel branch whenever the kernel client
is used, according to the matrix of configs. This prevents easily testing a new
kernel with the -k option to teuthology-suite. So, using the base config passed
to the script, detect if an alternate testing kernel is desired and use that
instead.

The scheduler still needs to do some work:

    $ teuthology-suite ... --verbose -k wip-pdonnell-i66704 --filter k-testing
    ...
    2024-10-18 00:29:43,875.875 DEBUG:teuthology.suite.merge:base kernel {'branch': 'wip-pdonnell-i66704', 'kdb': 1, 'sha1': '745cacd8f31e50d7f3b6039bbd8c9a8dfc07bf03', 'flavor': 'default'}
    2024-10-18 00:29:43,875.875 DEBUG:teuthology.suite.merge:overriding testing kernel with {'branch': 'wip-pdonnell-i66704', 'kdb': 1, 'sha1': '745cacd8f31e50d7f3b6039bbd8c9a8dfc07bf03', 'flavor': 'default'}

vs.

    $ teuthology-suite ... --verbose --filter k-testing
    ...
    2024-10-17 20:04:52,265.265 DEBUG:teuthology.suite.merge:base kernel {'branch': 'distro', 'kdb': 1, 'sha1': 'distro'}

Which will only select jobs using the "testing" kernel.

See-also: ceph/teuthology#2008
Fixes: https://tracker.ceph.com/issues/68603
Signed-off-by: Patrick Donnelly <[email protected]>
  • Loading branch information
Patrick Donnelly authored and kulahe3 committed Dec 10, 2024
1 parent b7ab7f7 commit d0c2eca
Show file tree
Hide file tree
Showing 8 changed files with 38 additions and 0 deletions.
23 changes: 23 additions & 0 deletions qa/cephfs/begin/3-kernel.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
# When the --kernel option is given to teuthology-suite, the kernel is set for
# all nodes (also, the kernel is "distro" when the --kernel option is not set).
# We don't generally want to use a custom kernel for all tests, so unset it.
# The k-testing.yaml will set it, if given, for only the client nodes.
#
# Allow overriding this by using a branch ending in "-all".

teuthology:
postmerge:
- |
local branch = yaml.kernel.branch
if branch and not yaml.kernel.branch:find "-all$" then
log.debug("removing default kernel specification: %s", yaml.kernel)
py_attrgetter(yaml.kernel).pop('branch', nil)
py_attrgetter(yaml.kernel).pop('deb', nil)
py_attrgetter(yaml.kernel).pop('flavor', nil)
py_attrgetter(yaml.kernel).pop('kdb', nil)
py_attrgetter(yaml.kernel).pop('koji', nil)
py_attrgetter(yaml.kernel).pop('koji_task', nil)
py_attrgetter(yaml.kernel).pop('rpm', nil)
py_attrgetter(yaml.kernel).pop('sha1', nil)
py_attrgetter(yaml.kernel).pop('tag', nil)
end
Original file line number Diff line number Diff line change
@@ -1,3 +1,12 @@
teuthology:
premerge: |
log.debug("base kernel %s", base_config.kernel)
local kernel = base_config.kernel
if kernel.branch ~= "distro" then
log.debug("overriding testing kernel with %s", kernel)
yaml_fragment.kernel.client = kernel
end
kernel:
client:
branch: testing
Expand Down
1 change: 1 addition & 0 deletions qa/suites/fs/upgrade/mds_upgrade_sequence/kernel.yaml
1 change: 1 addition & 0 deletions qa/suites/fs/upgrade/nofs/kernel.yaml
1 change: 1 addition & 0 deletions qa/suites/fs/upgrade/upgraded_client/kernel.yaml
1 change: 1 addition & 0 deletions qa/suites/fs/workload/begin/3-kernel.yaml

0 comments on commit d0c2eca

Please sign in to comment.