squid: qa: Add multifs root_squash testcase#65802
Closed
joscollin wants to merge 3 commits intoceph:squidfrom
Closed
squid: qa: Add multifs root_squash testcase#65802joscollin wants to merge 3 commits intoceph:squidfrom
joscollin wants to merge 3 commits intoceph:squidfrom
Conversation
Fixes: https://tracker.ceph.com/issues/64641 Signed-off-by: Kotresh HR <khiremat@redhat.com> (cherry picked from commit 1fda8ed) Conflicts: qa/tasks/cephfs/test_admin.py - Few of the other tests got added in between.
Root squash is valid for kclient too, Kotresh ran test recently fo it against main branch. Therefore it is safe to remove. ceph#56846 (comment) Signed-off-by: Rishabh Dave <ridave@redhat.com> (cherry picked from commit 422d98c)
test_multifs_single_path_rootsquash was never run with vstart_runner.py or with teuthology and is therefore full of bugs. Fix it to make sure it runs fine. Introduced-by: 1fda8ed Fixes: https://tracker.ceph.com/issues/65246 Signed-off-by: Rishabh Dave <ridave@redhat.com> (cherry picked from commit d9752a6)
Member
Author
|
This PR is under test in https://tracker.ceph.com/issues/73450. |
rishabh-d-dave
requested changes
Oct 14, 2025
| self.assertTrue(self.mount_a.is_blocked()) | ||
|
|
||
|
|
||
| def test_multifs_single_path_rootsquash(self): |
Contributor
There was a problem hiding this comment.
This test failed in QA -
2025-10-10T21:21:46.080 INFO:tasks.cephfs_test_runner:======================================================================
2025-10-10T21:21:46.080 INFO:tasks.cephfs_test_runner:ERROR: test_multifs_single_path_rootsquash (tasks.cephfs.test_admin.TestFsAuthorize)
2025-10-10T21:21:46.080 INFO:tasks.cephfs_test_runner:Test root_squash with multi fs
2025-10-10T21:21:46.080 INFO:tasks.cephfs_test_runner:----------------------------------------------------------------------
2025-10-10T21:21:46.081 INFO:tasks.cephfs_test_runner:Traceback (most recent call last):
2025-10-10T21:21:46.081 INFO:tasks.cephfs_test_runner: File "/home/teuthworker/src/github.com_ceph_ceph-c_104ea276496eeef39137386037044178d6f67f79/qa/tasks/cephfs/test_admin.py", line 1985, in tearDown
2025-10-10T21:21:46.081 INFO:tasks.cephfs_test_runner: self.mount_a.umount_wait()
2025-10-10T21:21:46.081 INFO:tasks.cephfs_test_runner: File "/home/teuthworker/src/github.com_ceph_ceph-c_104ea276496eeef39137386037044178d6f67f79/qa/tasks/cephfs/kernel_mount.py", line 191, in umount_wait
2025-10-10T21:21:46.081 INFO:tasks.cephfs_test_runner: self.umount(force)
2025-10-10T21:21:46.081 INFO:tasks.cephfs_test_runner: File "/home/teuthworker/src/github.com_ceph_ceph-c_104ea276496eeef39137386037044178d6f67f79/qa/tasks/cephfs/kernel_mount.py", line 170, in umount
2025-10-10T21:21:46.081 INFO:tasks.cephfs_test_runner: raise e
2025-10-10T21:21:46.081 INFO:tasks.cephfs_test_runner: File "/home/teuthworker/src/github.com_ceph_ceph-c_104ea276496eeef39137386037044178d6f67f79/qa/tasks/cephfs/kernel_mount.py", line 163, in umount
2025-10-10T21:21:46.081 INFO:tasks.cephfs_test_runner: self.client_remote.run(args=cmd, timeout=UMOUNT_TIMEOUT, omit_sudo=False)
2025-10-10T21:21:46.081 INFO:tasks.cephfs_test_runner: File "/home/teuthworker/src/git.ceph.com_teuthology_78c036dc9ad59cb33807dc7f21fda50de2f348d2/teuthology/orchestra/remote.py", line 575, in run
2025-10-10T21:21:46.081 INFO:tasks.cephfs_test_runner: r = self._runner(client=self.ssh, name=self.shortname, **kwargs)
2025-10-10T21:21:46.081 INFO:tasks.cephfs_test_runner: File "/home/teuthworker/src/git.ceph.com_teuthology_78c036dc9ad59cb33807dc7f21fda50de2f348d2/teuthology/orchestra/run.py", line 461, in run
2025-10-10T21:21:46.081 INFO:tasks.cephfs_test_runner: r.wait()
2025-10-10T21:21:46.081 INFO:tasks.cephfs_test_runner: File "/home/teuthworker/src/git.ceph.com_teuthology_78c036dc9ad59cb33807dc7f21fda50de2f348d2/teuthology/orchestra/run.py", line 161, in wait
2025-10-10T21:21:46.081 INFO:tasks.cephfs_test_runner: self._raise_for_status()
2025-10-10T21:21:46.082 INFO:tasks.cephfs_test_runner: File "/home/teuthworker/src/git.ceph.com_teuthology_78c036dc9ad59cb33807dc7f21fda50de2f348d2/teuthology/orchestra/run.py", line 181, in _raise_for_status
2025-10-10T21:21:46.082 INFO:tasks.cephfs_test_runner: raise CommandFailedError(
2025-10-10T21:21:46.082 INFO:tasks.cephfs_test_runner:teuthology.exceptions.CommandFailedError: Command failed on smithi071 with status 32: 'sudo umount /home/ubuntu/cephtest/mnt.0'
joscollin
commented
Oct 16, 2025
Member
Author
joscollin
left a comment
There was a problem hiding this comment.
QA failed:
https://tracker.ceph.com/issues/73565
Member
Author
|
@kotreshhr QA Failed for this backport. Please check. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
backport tracker: https://tracker.ceph.com/issues/64939
backport of #55833
parent tracker: https://tracker.ceph.com/issues/64641
backport tracker: https://tracker.ceph.com/issues/65907
backport of #56846
parent tracker: https://tracker.ceph.com/issues/65246
Both the trackers need to be backported as the #55833 has a bug.
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
xbetween the brackets:[x]. Spaces and capitalization matter when checking off items this way.Checklist
Show available Jenkins commands
jenkins test classic perfJenkins Job | Jenkins Job Definitionjenkins test crimson perfJenkins Job | Jenkins Job Definitionjenkins test signedJenkins Job | Jenkins Job Definitionjenkins test make checkJenkins Job | Jenkins Job Definitionjenkins test make check arm64Jenkins Job | Jenkins Job Definitionjenkins test submodulesJenkins Job | Jenkins Job Definitionjenkins test dashboardJenkins Job | Jenkins Job Definitionjenkins test dashboard cephadmJenkins Job | Jenkins Job Definitionjenkins test apiJenkins Job | Jenkins Job Definitionjenkins test docsReadTheDocs | Github Workflow Definitionjenkins test ceph-volume allJenkins Jobs | Jenkins Jobs Definitionjenkins test windowsJenkins Job | Jenkins Job Definitionjenkins test rook e2eJenkins Job | Jenkins Job Definition