qa: cephfs suite changes for rocky by batrick · Pull Request #66294 · ceph/ceph

batrick · 2025-11-17T18:15:52Z

Contribution Guidelines

To sign and title your commits, please refer to Submitting Patches to Ceph.
If you are submitting a fix for a stable branch (e.g. "quincy"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.
When filling out the below checklist, you may click boxes directly in the GitHub web UI. When entering or editing the entire PR message in the GitHub web UI editor, you may also select a checklist item by adding an x between the brackets: [x]. Spaces and capitalization matter when checking off items this way.

Checklist

Tracker (select at least one)
- References tracker ticket
- Very recent bug; references commit where it was introduced
- New feature (ticket optional)
- Doc update (no ticket needed)
- Code cleanup (no ticket needed)
Component impact
- Affects Dashboard, opened tracker ticket
- Affects Orchestrator, opened tracker ticket
- No impact that needs to be tracked
Documentation (select at least one)
- Updates relevant documentation
- No doc update is appropriate
Tests (select at least one)
- Includes unit test(s)
- Includes integration test(s)
- Includes bug reproducer
- No tests

Show available Jenkins commands

jenkins test classic perf Jenkins Job | Jenkins Job Definition
jenkins test crimson perf Jenkins Job | Jenkins Job Definition
jenkins test signed Jenkins Job | Jenkins Job Definition
jenkins test make check Jenkins Job | Jenkins Job Definition
jenkins test make check arm64 Jenkins Job | Jenkins Job Definition
jenkins test submodules Jenkins Job | Jenkins Job Definition
jenkins test dashboard Jenkins Job | Jenkins Job Definition
jenkins test dashboard cephadm Jenkins Job | Jenkins Job Definition
jenkins test api Jenkins Job | Jenkins Job Definition
jenkins test docs ReadTheDocs | Github Workflow Definition
jenkins test ceph-volume all Jenkins Jobs | Jenkins Jobs Definition
jenkins test windows Jenkins Job | Jenkins Job Definition
jenkins test rook e2e Jenkins Job | Jenkins Job Definition

You must only issue one Jenkins command per-comment. Jenkins does not understand
comments with more than one command.

batrick · 2025-11-17T18:27:37Z

This PR is under test in https://tracker.ceph.com/issues/73883.

batrick · 2025-11-17T18:27:57Z

This PR is under test in https://tracker.ceph.com/issues/73884.

batrick · 2025-11-17T19:43:36Z

This PR is under test in https://tracker.ceph.com/issues/73885.

batrick · 2025-11-17T20:42:23Z

This PR is under test in https://tracker.ceph.com/issues/73888.

batrick · 2025-11-17T20:42:49Z

This PR is under test in https://tracker.ceph.com/issues/73889.

* refs/pull/66294/head: qa: use nft instead iptables qa: use py3 builtin ipaddress module

batrick · 2025-12-09T14:11:15Z

This PR is under test in https://tracker.ceph.com/issues/74161.

batrick · 2026-02-07T21:46:12Z

This PR is under test in https://tracker.ceph.com/issues/74811.

* refs/pull/66294/head: qa: update fs:upgrade to N-2 release for U qa: update fs suite to rocky10 qa: skip dashboard install due to dependency noise qa/suites/fs: use rocky-10 with cephadm qa: use nft instead iptables qa: use py3 builtin ipaddress module

batrick · 2026-02-23T15:02:23Z

(trivial rebase)

batrick · 2026-03-06T17:50:41Z

This PR is under test in https://tracker.ceph.com/issues/75377.

vshankar · 2026-03-09T07:25:13Z

qa/tasks/cephfs/mount.py

+
+            # Ensure filter table exists. Ignore error if it already does.
+            sudo nft add table ip filter > /dev/null 2>&1 || true
+            sudo nft add chain ip filter forward {{ type filter hook forward priority 0 \; }} > /dev/null 2>&1 || true


I am not 100% sure, but maybe this bit of change is causing clients (at least fuse) on ubuntu hosts to not have a successful mount. Tracker: https://tracker.ceph.com/issues/75355 (comment https://tracker.ceph.com/issues/75355#note-6)

E.g.: /a/yuriw-2026-03-05_16:00:50-fs-wip-rocky10-branch-of-the-day-2026-03-04-1772633736-distro-default-trial/89336

Possibly kernel clients too. See: https://tracker.ceph.com/issues/75357

Thanks, I believe I've addressed this: corrected the saddr netmask (use {self.ceph_brx_net}) and iptables for ubuntu 22.04.

Fantastic! I will have a look at run tests.

qa/tasks/cephfs/mount.py

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>

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>

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>

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>

Skip installing dashboard due to dependency noise (alerts module). Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>

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>

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>

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>

Signed-off-by: Venky Shankar <vshankar@redhat.com>

github-actions bot added cephfs Ceph File System tests labels Nov 17, 2025

batrick added wip-pdonnell-testing wip-pdonnell-testing2 labels Nov 17, 2025

batrick added a commit to batrick/ceph that referenced this pull request Nov 18, 2025

Merge PR ceph#66294 into wip-pdonnell-testing-20251117.204235-debug

cf76906

* refs/pull/66294/head: qa: use nft instead iptables qa: use py3 builtin ipaddress module

batrick force-pushed the qa-cephfs-rocky branch from 963eef1 to 144b0fe Compare November 19, 2025 17:26

batrick requested a review from vshankar December 6, 2025 00:29

vshankar self-assigned this Dec 8, 2025

batrick force-pushed the qa-cephfs-rocky branch from 144b0fe to af24a9c Compare February 2, 2026 14:18

batrick mentioned this pull request Feb 7, 2026

qa/distros: add rocky 10.0 as supported distro/container host #66055

Open

batrick force-pushed the qa-cephfs-rocky branch from af24a9c to 1222975 Compare February 7, 2026 21:45

batrick force-pushed the qa-cephfs-rocky branch from 1222975 to b708696 Compare February 8, 2026 18:29

yaarith added the rocky10 label Feb 12, 2026

vshankar requested a review from a team February 17, 2026 15:58

batrick force-pushed the qa-cephfs-rocky branch from b708696 to 4a9ea3b Compare February 23, 2026 15:02

vshankar added wip-vshankar-testing and removed wip-vshankar-testing labels Mar 3, 2026

batrick force-pushed the qa-cephfs-rocky branch from 4a9ea3b to 944d069 Compare March 4, 2026 13:30

batrick added ubuntu-24.04 and removed wip-pdonnell-testing wip-pdonnell-testing2 labels Mar 6, 2026

vshankar reviewed Mar 9, 2026

View reviewed changes

batrick force-pushed the qa-cephfs-rocky branch from 944d069 to 24b2a3e Compare March 11, 2026 06:18

ljflores reviewed Mar 17, 2026

View reviewed changes

qa/tasks/cephfs/mount.py Outdated Show resolved Hide resolved

batrick and others added 10 commits March 18, 2026 12:51

qa: use py3 builtin ipaddress module

d56ea8c

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>

qa: use nft instead iptables

cefab4c

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>

qa: correct wording of comment

d495c09

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>

qa: only setup nat rules during bridge creation

ac49ff7

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>

qa/suites/fs: use rocky-10 with cephadm

c5a64c9

Skip installing dashboard due to dependency noise (alerts module). Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>

qa: update fs suite to rocky10

113392b

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>

qa: update fs:upgrade to N-2 release for U

abcf4b9

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>

qa/cephfs: add missing ceph-osd-{classic,crinsom} in fs:upgrade

c2f2855

Signed-off-by: Venky Shankar <vshankar@redhat.com>

vshankar force-pushed the qa-cephfs-rocky branch from 24b2a3e to c2f2855 Compare March 18, 2026 07:22

Conversation

batrick commented Nov 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Contribution Guidelines

Checklist

Uh oh!

batrick commented Nov 17, 2025

Uh oh!

batrick commented Nov 17, 2025

Uh oh!

batrick commented Nov 17, 2025

Uh oh!

batrick commented Nov 17, 2025

Uh oh!

batrick commented Nov 17, 2025

Uh oh!

batrick commented Dec 9, 2025

Uh oh!

batrick commented Feb 7, 2026

Uh oh!

batrick commented Feb 23, 2026

Uh oh!

batrick commented Mar 6, 2026

Uh oh!

vshankar Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

vshankar Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

batrick Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

vshankar Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

batrick commented Nov 17, 2025 •

edited

Loading