Commit Graph

12681 Commits

Author SHA1 Message Date
Kefu Chai
1f5169db36 Merge pull request #68297 from tchaikov/wip-feedback-without-tracker
mgr/feedback: fix flaky test_issue_tracker_create_with_invalid_key

Reviewed-by: Nizamudeen A <nia@redhat.com>
Reviewed-by: Afreen Misbah <afreen@ibm.com>
2026-04-17 13:34:51 +08:00
Casey Bodley
7a6816df67 Merge pull request #68289 from cbodley/wip-75945
qa/valgrind: generalize suppressions for gcc-14 MismatchedFree

Reviewed-by: Adam C. Emerson <aemerson@redhat.com>
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
2026-04-16 16:18:38 -04:00
Patrick Donnelly
fabaa7d359 Merge PR #67823 into main
* refs/pull/67823/head:
	qa: remove unused qa_scripts

Reviewed-by: Ilya Dryomov <idryomov@redhat.com>
Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
2026-04-16 12:29:26 -04:00
Patrick Donnelly
dd2b7661be Merge PR #67822 into main
* refs/pull/67822/head:
	qa: remove vestiges of ceph-deploy
	doc: remove references to ceph-deploy

Reviewed-by: Anthony D Atri <anthony.datri@gmail.com>
2026-04-16 12:28:52 -04:00
Kefu Chai
f9c14ff857 Merge pull request #68282 from tchaikov/wip-mgr-module-neg-exit-code
mgr/crash, mgr/status: return negative errno to fix CLI exit code

Reviewed-by: Dan Mick <dmick@ibm.com>
2026-04-16 13:13:05 +08:00
Casey Bodley
542f17513d qa/valgrind: generalize and group MismatchedFree suppressions
combine the various MismatchedFree suppressions into unconditional ones
for each function

Signed-off-by: Casey Bodley <cbodley@redhat.com>
2026-04-15 14:38:23 -04:00
Patrick Donnelly
49753d2e6d Merge PR #67032 into main
* refs/pull/67032/head:
	qa: add trivial cephfs-tool bench test
	doc/cephfs: add cephfs-tool documentation
	tools/cephfs: add new cephfs-tool

Reviewed-by: Venky Shankar <vshankar@redhat.com>
2026-04-15 09:11:34 -04:00
Venky Shankar
f140f1f5af Merge pull request #68351 from joscollin/wip-TestCorruptedSubvolumes-fix
qa: update yaml file for TestCorruptedSubvolumes

Reviewed-by: Rishabh Dave <ridave@redhat.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
2026-04-14 23:32:22 +05:30
Patrick Donnelly
acf75b9442 Merge PR #66294 into main
* refs/pull/66294/head:
	qa: enforce centos9 for test
	qa: rename distro
	qa/suites/fs/bugs: use centos9 for squid upgrade test
	qa: remove unused variables
	qa: use centos9 for fs suites using k-testing
	qa: update fs suite to rocky10
	qa: skip dashboard install due to dependency noise
	qa: only setup nat rules during bridge creation
	qa: correct wording of comment
	qa: use nft instead iptables
	qa: use py3 builtin ipaddress module

Reviewed-by: Venky Shankar <vshankar@redhat.com>
2026-04-14 11:46:54 -04:00
Venky Shankar
9f0cc8789d Merge pull request #62351 from vshankar/wip-revert-referent-inodes
mds: revert referent inode work

Reviewed-by: Patrick Donnelly <pdonnell@ibm.com>
2026-04-14 21:05:25 +05:30
Jos Collin
7d936833bd qa: update yaml file for TestCorruptedSubvolumes
Update yaml file to include TestCorruptedSubvolumes in qa tests.

Fixes: https://tracker.ceph.com/issues/76001
Signed-off-by: Jos Collin <jcollin@redhat.com>
2026-04-14 19:18:34 +05:30
Patrick Donnelly
d60ec8e0ac qa: remove vestiges of ceph-deploy
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
2026-04-14 09:03:58 -04:00
Patrick Donnelly
aff948588a qa: add trivial cephfs-tool bench test
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
2026-04-14 08:57:33 -04:00
Patrick Donnelly
4ebd6f62eb Merge PR #67555 into main
* refs/pull/67555/head:
	qa/tasks/quiescer: remove racy assertion

Reviewed-by: Rishabh Dave <ridave@redhat.com>
2026-04-14 08:18:07 -04:00
Patrick Donnelly
3e68392793 Merge PR #68360 into main
* refs/pull/68360/head:
	qa: ignore expected MON_DOWN

Reviewed-by: Venky Shankar <vshankar@redhat.com>
2026-04-14 08:17:28 -04:00
Jose Juan Palacios-Perez
5b7512ec1a Merge pull request #67268 from perezjosibm/wip-perezjos-tracker74444
qa/suites/crimson-rados: first version of cephfs/
2026-04-14 10:27:41 +01:00
Matan Breizman
16290acdc2 Merge pull request #67358 from knrt10/wip-crimson-rgw-sts
qa: Add rgw sts suites to crimson-rados

Reviewed-by: Matan Breizman <mbreizma@redhat.com>
2026-04-14 11:32:11 +03:00
Patrick Donnelly
815d1d73a7 qa: enforce centos9 for test
Avoids problem where rocky10 packages do not exist for squid.

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
2026-04-13 21:29:06 -04:00
Patrick Donnelly
9f2cac88d9 qa: rename distro
The kernel mount overrides for the distro have no effect if they are
applied before `supported-random-distro`.

Fixes:

    2026-04-13T19:06:13.603 INFO:teuthology.task.pexec:sudo dnf remove nvme-cli -y
    2026-04-13T19:06:13.603 INFO:teuthology.task.pexec:sudo dnf install nvmetcli nvme-cli -y
    2026-04-13T19:06:13.626 INFO:teuthology.task.pexec:Running commands on host ubuntu@trial005.front.sepia.ceph.com
    2026-04-13T19:06:13.627 INFO:teuthology.task.pexec:sudo dnf remove nvme-cli -y
    2026-04-13T19:06:13.627 INFO:teuthology.task.pexec:sudo dnf install nvmetcli nvme-cli -y
    2026-04-13T19:06:13.652 INFO:teuthology.orchestra.run.trial148.stderr:sudo: dnf: command not found
    2026-04-13T19:06:13.653 DEBUG:teuthology.orchestra.run:got remote process result: 1
    2026-04-13T19:06:13.654 ERROR:teuthology.run_tasks:Saw exception from tasks.
    Traceback (most recent call last):
      File "/home/teuthworker/src/git.ceph.com_teuthology_426ec63bc4a39bba882efb593125294667afc593/teuthology/run_tasks.py", line 105, in run_tasks
        manager = run_one_task(taskname, ctx=ctx, config=config)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      File "/home/teuthworker/src/git.ceph.com_teuthology_426ec63bc4a39bba882efb593125294667afc593/teuthology/run_tasks.py", line 83, in run_one_task
        return task(**kwargs)
               ^^^^^^^^^^^^^^
      File "/home/teuthworker/src/git.ceph.com_teuthology_426ec63bc4a39bba882efb593125294667afc593/teuthology/task/pexec.py", line 149, in task
        with parallel() as p:
      File "/home/teuthworker/src/git.ceph.com_teuthology_426ec63bc4a39bba882efb593125294667afc593/teuthology/parallel.py", line 84, in __exit__
        for result in self:
      File "/home/teuthworker/src/git.ceph.com_teuthology_426ec63bc4a39bba882efb593125294667afc593/teuthology/parallel.py", line 98, in __next__
        resurrect_traceback(result)
      File "/home/teuthworker/src/git.ceph.com_teuthology_426ec63bc4a39bba882efb593125294667afc593/teuthology/parallel.py", line 30, in resurrect_traceback
        raise exc.exc_info[1]
      File "/home/teuthworker/src/git.ceph.com_teuthology_426ec63bc4a39bba882efb593125294667afc593/teuthology/parallel.py", line 23, in capture_traceback
        return func(*args, **kwargs)
               ^^^^^^^^^^^^^^^^^^^^^
      File "/home/teuthworker/src/git.ceph.com_teuthology_426ec63bc4a39bba882efb593125294667afc593/teuthology/task/pexec.py", line 62, in _exec_host
        tor.wait([r])
      File "/home/teuthworker/src/git.ceph.com_teuthology_426ec63bc4a39bba882efb593125294667afc593/teuthology/orchestra/run.py", line 485, in wait
        proc.wait()
      File "/home/teuthworker/src/git.ceph.com_teuthology_426ec63bc4a39bba882efb593125294667afc593/teuthology/orchestra/run.py", line 161, in wait
        self._raise_for_status()
      File "/home/teuthworker/src/git.ceph.com_teuthology_426ec63bc4a39bba882efb593125294667afc593/teuthology/orchestra/run.py", line 181, in _raise_for_status
        raise CommandFailedError(
    teuthology.exceptions.CommandFailedError: Command failed on trial148 with status 1: 'TESTDIR=/home/ubuntu/cephtest bash -s'

which was done because these dnf commands were pulled from rocky10.yaml from the kclient overrides but ubuntu_latest was used for the random distro.

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
2026-04-13 21:27:19 -04:00
Patrick Donnelly
1e31a6deaf qa/suites/fs/bugs: use centos9 for squid upgrade test
To avoid missing package error for rocky10

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
2026-04-13 21:27:19 -04:00
Patrick Donnelly
d8cc2ea40c qa: remove unused variables
To make tox-qa happy.

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
2026-04-13 21:27:19 -04:00
Patrick Donnelly
5b2f5ee919 qa: use centos9 for fs suites using k-testing
A better approach would be to include centos9 OR rocky10 for
distribution choice. Then we can just filter out rocky10 when we're
testing the `testing` kernel but keep rocky10 coverage for other
testing.

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
2026-04-13 21:27:18 -04:00
Patrick Donnelly
6bf6fcb2b1 qa: update fs suite to rocky10
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
2026-04-13 21:27:18 -04:00
Patrick Donnelly
b81cca6517 qa: skip dashboard install due to dependency noise
2025-11-18T19:46:46.226 INFO:teuthology.orchestra.run.smithi008.stdout:/usr/bin/ceph: stderr Error ENOTSUP: Module 'alerts' is not enabled/loaded (required by command 'dashboard set-ssl-certificate'): use `ceph mgr module enable alerts` to enable it

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
2026-04-13 21:27:18 -04:00
Patrick Donnelly
6fa39caea7 qa: only setup nat rules during bridge creation
Currently the code recreates these NAT rules for every mount. This only
needs to be done once by the first mount.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
2026-04-13 21:27:18 -04:00
Patrick Donnelly
83ecfdfe8b qa: correct wording of comment
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
2026-04-13 21:27:18 -04:00
Patrick Donnelly
60b31e5021 qa: use nft instead iptables
rocky.10 does not support iptables with MASQUERADE targets. (Or maybe it
does with more prodding but it's easier to just switch to nft.)

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
2026-04-13 21:27:18 -04:00
Patrick Donnelly
5c511835c3 qa: use py3 builtin ipaddress module
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
2026-04-13 21:27:18 -04:00
Patrick Donnelly
7a042b0798 qa: ignore expected MON_DOWN
tasks.cephfs.test_misc.TestMisc.test_dump_inmemory_log_on_missed_internal_heartbeats
deliberately stops the monitors.

Fixes: https://tracker.ceph.com/issues/75992
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
2026-04-13 14:22:35 -04:00
Matan Breizman
82466e2244 Merge pull request #68342 from Matan-B/wip-matanb-objecter-branch
qa/tasks/admin_socket.py: replace git.ceph.com with GitHub raw URLs

Reviewed-by: Shraddha Agrawal <shraddhaag@ibm.com>
2026-04-13 19:20:40 +03:00
Sridhar Seshasayee
acab6731d4 Merge pull request #68048 from sseshasa/wip-fix-max-criteria-in-phase1
mgr/DaemonServer: Limit search for OSDs to upgrade within the crush bucket

Reviewed-by: Nitzan Mordechai <nmordech@redhat.com>
Reviewed-by: Kamoltat Sirivadhna <ksirivad@redhat.com>
2026-04-13 15:14:24 +05:30
Kefu Chai
f1f196bce9 Merge pull request #67622 from tchaikov/wip-qa-cephadm-ignore-failed-daemon
qa/cephadm: ignore transient CEPHADM_FAILED_DAEMON in smoke-singlehost

Reviewed-by: Laura Flores <lflores@redhat.com>
Reviewed-by: Adam King <adking@redhat.com>
2026-04-13 16:27:11 +08:00
Matan Breizman
05f80acd3d qa/tasks/admin_socket.py: replace git.ceph.com with GitHub raw URLs
admin_socket tests can still fail on non-default branches
due to git.ceph.com wget timeouts. Use raw URLs instead.

Fixes: https://tracker.ceph.com/issues/75969

Signed-off-by: Matan Breizman <mbreizma@redhat.com>
2026-04-12 17:41:30 +03:00
Redouane Kachach
e999950dcc Merge pull request #68288 from lumir-sliva/qa/fix-cephadm-release-mismatch
qa/cephadm: derive container image from cephadm release

Reviewed-by: Redouane Kachach <rkachach@ibm.com>
2026-04-10 23:15:28 +02:00
Redouane Kachach
e1e2a56a19 Merge pull request #68285 from rkachach/fix_issue_75944
qa: fixing target release for MDS upgrade to be umbrella

Reviewed-by: Adam King <adking@redhat.com>
2026-04-10 09:41:00 +02:00
Redouane Kachach
06b1c4fc03 Merge pull request #68281 from rkachach/fix_nvmeof_smoke_test
qa: fixing nvmeof orch smoke test

Reviewed-by: Adam King <adking@redhat.com>
2026-04-10 09:34:54 +02:00
Kefu Chai
69eff2d74b mgr/feedback: make tracker URL configurable and stop hitting live tracker in tests
The feedback module hard-coded tracker.ceph.com as the Ceph issue
tracker hostname. This meant the dashboard test
test_issue_tracker_create_with_invalid_key actually performed an
HTTPS POST against the live production tracker on every CI run.
Whenever tracker.ceph.com returned an unexpected response (5xx,
gateway error, transient failure), the test would fail on
completely unrelated pull requests.

Add a runtime module option 'mgr/feedback/tracker_url' that
controls which host the CephTrackerClient talks to. The default
keeps the existing behaviour (tracker.ceph.com). The dashboard
test now sets this option to an unreachable hostname in setUpClass
so create_issue fails fast with a ConnectionError instead of
depending on tracker.ceph.com being reachable.

Combined with the prior commit's error-handling fix, the test now
deterministically verifies that any tracker failure is surfaced as
HTTP 400, without requiring network access.

Signed-off-by: Kefu Chai <k.chai@proxmox.com>
2026-04-10 12:54:35 +08:00
Casey Bodley
c632fb95ea Merge pull request #68269 from kshtsk/wip-rgw-crypt-kmip
qa/rgw: Revive crypt kmip

Reviewed-by: Casey Bodley <cbodley@redhat.com>
2026-04-09 15:47:20 -04:00
Casey Bodley
1b6578e07f qa/valgrind: generalize suppressions for gcc-14 MismatchedFree
the rgw suite is showing a handful of these false positives, and i
really don't want to micromanage suppressions for every little thing
that shows up here

so i chose to generalize the existing suppressions from
https://tracker.ceph.com/issues/74604 so that a single suppression would
cover this entire class of MismatchedFree warnings

Fixes: https://tracker.ceph.com/issues/75945

Signed-off-by: Casey Bodley <cbodley@redhat.com>
2026-04-09 11:33:39 -04:00
Lumir Sliva
db3ba638e9 qa/cephadm: derive container image from cephadm release
test_cephadm.sh hardcodes IMAGE_DEFAULT to ceph:main, which breaks
every stable branch whenever main is renamed to a new release. The
mismatch check in cephadm correctly rejects the container because its
release name doesn't match cephadm's own release. This has recurred on
every release transition (squid→tentacle, quincy→reef) without a fix.

Instead of always pulling ceph:main, derive IMAGE_DEFAULT from the
installed cephadm's version output. On stable builds (release type
"stable"), use ceph:<release> so the container matches cephadm. On dev
builds (main branch), fall back to ceph:main as before. The IMAGE_DEFAULT
env var can still be set externally to override.

Fixes: https://tracker.ceph.com/issues/75821
Signed-off-by: Lumir Sliva <61183145+lumir-sliva@users.noreply.github.com>
2026-04-09 17:10:33 +02:00
Redouane Kachach
38affe97b8 qa: fixing target release for MDS upgrade to be umbrella
Fixes: https://tracker.ceph.com/issues/75944
Signed-off-by: Redouane Kachach <rkachach@ibm.com>
2026-04-09 15:58:20 +02:00
Kefu Chai
7041f6b6b3 mgr/crash, mgr/status: return negative errno to fix CLI exit code
The crash and status mgr modules return positive errno values (e.g.
errno.EINVAL) on error. However, the ceph CLI (src/ceph.in) only
treats negative return codes as errors:

    if ret < 0:
        final_ret = -ret

A positive return code falls through and the CLI exits with 0, masking
the error. This causes commands like "ceph crash post" and "ceph crash
info <bad-id>" to return success (rc=0) despite failing, with the
error message only appearing on stderr.

This was observed in the wild and worked around in a77b47eeeb, which added a
stderr != "" check to ceph-crash's post_crash() as a mitigation.

Fix the root cause by negating the errno values in both modules,
consistent with the HandleCommandResult convention
(retval: "E.g. 0 or -errno.EINVAL") and the 154 other call sites
across mgr modules that already use negative errno.

Also add error-path tests for the crash module to cover the three
affected commands (info, post, archive).

See-also: a77b47eeeb
Fixes: https://tracker.ceph.com/issues/75937
Signed-off-by: Kefu Chai <k.chai@proxmox.com>
2026-04-09 20:40:35 +08:00
Redouane Kachach
e2df25bd66 qa: fixing nvmeof orch smoke test
Adapting the orch apply cmd to use the new pool/group arguments

Signed-off-by: Redouane Kachach <rkachach@ibm.com>
Fixes: https://tracker.ceph.com/issues/75934
2026-04-09 13:51:50 +02:00
Redouane Kachach
93aa3cb449 Merge pull request #67953 from rkachach/fix_issue_raw_osd_teuthology
qa: wipe disks in case of raw option is provided

Reviewed-by: Guillaume Abrioux <gabrioux@ibm.com>
2026-04-09 12:53:43 +02:00
Vallari Agrawal
1e9a1717db Merge pull request #68266 from VallariAg/fix-nvmeof-upgrade-v20
qa: fix nvmeof upgrade from v20.2.0
2026-04-09 16:12:04 +05:30
Kyr Shatskyy
da7d3b2847 qa/tasks/pykmip: archive pykmip log after server down
Signed-off-by: Kyr Shatskyy <kyrylo.shatskyy@clyso.com>
2026-04-09 02:16:25 +02:00
Kyr Shatskyy
995fd24597 qa/tasks/pykmip: use OpenSSL names instead IANA
For OpenSSL 3.x which the IANA names might not be supported.

Signed-off-by: Kyr Shatskyy <kyrylo.shatskyy@clyso.com>
2026-04-09 02:16:25 +02:00
Kyr Shatskyy
4698fff408 qa/tasks/pykmip: drop py2 deps
We don't need no more python2 dependencies

Signed-off-by: Kyr Shatskyy <kyrylo.shatskyy@clyso.com>
2026-04-09 02:16:25 +02:00
Kyr Shatskyy
fd20467981 Revert "qa/rgw/crypt: disable failing kmip testing"
This reverts commit 5df7b44727.

Signed-off-by: Kyr Shatskyy <kyrylo.shatskyy@clyso.com>
2026-04-09 02:16:10 +02:00
Aishwarya Mathuria
424bacaf1b Merge pull request #68225 from Matan-B/wip-matanb-split-timeouts
qa/suites/crimson-rados/../pggrow: increase op_delay
2026-04-08 21:52:18 +05:30