Project

General

Profile

Actions

QA Run #72654

open

wip-jcollin-testing-20251007.040314-squid

Added by Jos Collin 7 months ago. Updated about 1 month ago.

Status:
QA Needs Approval
Priority:
Normal

Description

  • PR #64205 -- squid: mgr/vol: include group name in subvolume's pool namespace name
  • PR #63222 -- squid: mgr/vol: make "snapshot getpath" cmd work with v1 and legacy
  • PR #63068 -- squid: mgr/vol: don't delete user-created pool in "volume create" command
  • PR #63023 -- squid: pybind/mgr/volumes: fix typo in casesensitive vxattr
  • PR #62918 -- squid: mgr/vol: add command to get snapshot path

Related issues 6 (6 open0 closed)

Blocks CephFS - Backport #71854: squid: mgr/volumes: subvolume create with --namespace-isolation doesn't isolate the namespace for subvolume of same name across groupsQA TestingRishabh DaveActions
Blocks CephFS - Backport #71276: squid: mgr/vol: add "snapshot getpath" cmd for subvol v1QA TestingRishabh DaveActions
Blocks CephFS - Backport #71150: squid: mgr/vol: volume create cmd deletes user-created pool if one of the pool passed is absentQA TestingRishabh DaveActions
Blocks CephFS - Backport #71117: squid: pybind/mgr/volumes: charmap not preserved via subvol cloneQA TestingPatrick DonnellyActions
Blocks CephFS - Backport #71130: squid: pybind/mgr/volumes: make casesensitive name uniform in interfaceQA TestingPatrick DonnellyActions
Blocks CephFS - Backport #71018: squid: mgr/vol: add a command to get the path of snapshotsQA TestingRishabh DaveActions
Actions #1

Updated by Jos Collin 7 months ago

  • Related to Backport #71854: squid: mgr/volumes: subvolume create with --namespace-isolation doesn't isolate the namespace for subvolume of same name across groups added
  • Related to Backport #71276: squid: mgr/vol: add "snapshot getpath" cmd for subvol v1 added
  • Related to Backport #71150: squid: mgr/vol: volume create cmd deletes user-created pool if one of the pool passed is absent added
  • Related to Backport #71117: squid: pybind/mgr/volumes: charmap not preserved via subvol clone added
  • Related to Backport #71130: squid: pybind/mgr/volumes: make casesensitive name uniform in interface added
Actions #2

Updated by Jos Collin 7 months ago

  • Assignee changed from Jos Collin to Christopher Hoffman

@Christopher Hoffman Please review.

Actions #3

Updated by Jos Collin 7 months ago

  • Status changed from QA Testing to QA Needs Approval
Actions #5

Updated by Christopher Hoffman 5 months ago

  • Status changed from QA Needs Approval to QA Needs Rerun/Rebuilt
  • Assignee changed from Christopher Hoffman to Jos Collin

There's three failures related to getpath here:
Test failure: test_subvolume_clone_in_progress_getpath (tasks.cephfs.test_volumes.TestSubvolumeSnapshotClones)

https://pulpito.ceph.com/jcollin-2025-08-21_00:49:41-fs:volumes-wip-jcollin-testing-20250820.084115-squid-distro-default-smithi/8456023

https://pulpito.ceph.com/jcollin-2025-08-21_00:49:41-fs:volumes-wip-jcollin-testing-20250820.084115-squid-distro-default-smithi/8456028

Test failure: test_subvolume_snapshot_protect_unprotect_sanity (tasks.cephfs.test_volumes.TestSubvolumeSnapshots)

https://pulpito.ceph.com/jcollin-2025-08-21_00:49:41-fs:volumes-wip-jcollin-testing-20250820.084115-squid-distro-default-smithi/8456031
https://pulpito.ceph.com/jcollin-2025-08-21_00:49:41-fs:volumes-wip-jcollin-testing-20250820.084115-squid-distro-default-smithi/8456026

Test failure: test_subvolume_no_upgrade_v1_sanity (tasks.cephfs.test_volumes.TestMisc)
https://pulpito.ceph.com/jcollin-2025-08-21_00:49:41-fs:volumes-wip-jcollin-testing-20250820.084115-squid-distro-default-smithi/8456026

https://pulpito.ceph.com/jcollin-2025-08-21_00:49:41-fs:volumes-wip-jcollin-testing-20250820.084115-squid-distro-default-smithi/8456030
https://pulpito.ceph.com/jcollin-2025-08-21_00:49:41-fs:volumes-wip-jcollin-testing-20250820.084115-squid-distro-default-smithi/8456035

The error is:

2025-08-21T01:34:50.032 INFO:tasks.cephfs_test_runner:Traceback (most recent call last):
2025-08-21T01:34:50.032 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/github.com_ceph_ceph-c_f65c5c680f3bf0b270c5461ed05d1b7c553fdfec/qa/tasks/cephfs/test_volumes.py", line 5462, in test_subvolume_snapshot_protect_unprotect_sanity
2025-08-21T01:34:50.032 INFO:tasks.cephfs_test_runner:    self._verify_clone(subvolume, snapshot, clone)
2025-08-21T01:34:50.032 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/github.com_ceph_ceph-c_f65c5c680f3bf0b270c5461ed05d1b7c553fdfec/qa/tasks/cephfs/test_volumes.py", line 143, in _verify_clone
2025-08-21T01:34:50.032 INFO:tasks.cephfs_test_runner:    path1 = self._get_subvolume_snapshot_path(subvolume, snapshot, source_group)
2025-08-21T01:34:50.032 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/github.com_ceph_ceph-c_f65c5c680f3bf0b270c5461ed05d1b7c553fdfec/qa/tasks/cephfs/test_volumes.py", line 79, in _get_subvolume_snapshot_path
2025-08-21T01:34:50.032 INFO:tasks.cephfs_test_runner:    cephfs_snap_path = self.get_ceph_cmd_stdout(cmd).strip()
2025-08-21T01:34:50.033 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/github.com_ceph_ceph-c_f65c5c680f3bf0b270c5461ed05d1b7c553fdfec/qa/tasks/ceph_test_case.py", line 55, in get_ceph_cmd_stdout
2025-08-21T01:34:50.033 INFO:tasks.cephfs_test_runner:    return self.run_ceph_cmd(**kwargs).stdout.getvalue()
2025-08-21T01:34:50.033 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/github.com_ceph_ceph-c_f65c5c680f3bf0b270c5461ed05d1b7c553fdfec/qa/tasks/ceph_test_case.py", line 30, in run_ceph_cmd
2025-08-21T01:34:50.033 INFO:tasks.cephfs_test_runner:    return self.mon_manager.run_cluster_cmd(**kwargs)
2025-08-21T01:34:50.033 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/github.com_ceph_ceph-c_f65c5c680f3bf0b270c5461ed05d1b7c553fdfec/qa/tasks/ceph_manager.py", line 1635, in run_cluster_cmd
2025-08-21T01:34:50.033 INFO:tasks.cephfs_test_runner:    return self.controller.run(**kwargs)
2025-08-21T01:34:50.033 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/git.ceph.com_teuthology_c9986ad400a1e25fcd55bdfe499ed247e51ac465/teuthology/orchestra/remote.py", line 575, in run
2025-08-21T01:34:50.033 INFO:tasks.cephfs_test_runner:    r = self._runner(client=self.ssh, name=self.shortname, **kwargs)
2025-08-21T01:34:50.033 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/git.ceph.com_teuthology_c9986ad400a1e25fcd55bdfe499ed247e51ac465/teuthology/orchestra/run.py", line 461, in run
2025-08-21T01:34:50.033 INFO:tasks.cephfs_test_runner:    r.wait()
2025-08-21T01:34:50.033 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/git.ceph.com_teuthology_c9986ad400a1e25fcd55bdfe499ed247e51ac465/teuthology/orchestra/run.py", line 161, in wait
2025-08-21T01:34:50.033 INFO:tasks.cephfs_test_runner:    self._raise_for_status()
2025-08-21T01:34:50.033 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/git.ceph.com_teuthology_c9986ad400a1e25fcd55bdfe499ed247e51ac465/teuthology/orchestra/run.py", line 181, in _raise_for_status
2025-08-21T01:34:50.033 INFO:tasks.cephfs_test_runner:    raise CommandFailedError(
2025-08-21T01:34:50.034 INFO:tasks.cephfs_test_runner:teuthology.exceptions.CommandFailedError: Command failed on smithi105 with status 22: 'sudo adjust-ulimits ceph-coverage /home/ubuntu/cephtest/archive/coverage timeout 120 ceph --cluster ceph fs subvolume snapshot getpath cephfs subvol18610 subvol_snap29600'

This is because the commit adding "fs subvolume snapshot getpath" does not exist in this branch. It's part of another qa test run and yet to be merged

QA tracker here: https://tracker.ceph.com/issues/73331
Specific backport: https://tracker.ceph.com/issues/71018

@Jos Collin Can you pull in this backport and rerun? I'd recommend all of the original 15 jobs as the size is minimal.

Actions #6

Updated by Jos Collin 5 months ago

  • Tags (freeform) set to wip-jcollin-testing-squid2
Actions #8

Updated by Jos Collin 5 months ago

  • Blocks Backport #71018: squid: mgr/vol: add a command to get the path of snapshots added
Actions #9

Updated by Jos Collin 5 months ago

  • Subject changed from wip-jcollin-testing-20250820.084115-squid to wip-jcollin-testing-20251007.040314-squid
  • Description updated (diff)
  • Shaman Build changed from wip-jcollin-testing-20250820.084115-squid to wip-jcollin-testing-20251007.040314-squid
  • QA Runs changed from wip-jcollin-testing-20250820.084115-squid to wip-jcollin-testing-20251007.040314-squid
  • Git Branch changed from ceph/ceph-ci/commits/testing/wip-jcollin-testing-20250820.084115-squid to ceph/ceph-ci/commits/testing/wip-jcollin-testing-20251007.040314-squid
Actions #10

Updated by Jos Collin 5 months ago

  • Related to deleted (Backport #71854: squid: mgr/volumes: subvolume create with --namespace-isolation doesn't isolate the namespace for subvolume of same name across groups)
Actions #11

Updated by Jos Collin 5 months ago

  • Related to deleted (Backport #71276: squid: mgr/vol: add "snapshot getpath" cmd for subvol v1)
Actions #12

Updated by Jos Collin 5 months ago

  • Related to deleted (Backport #71150: squid: mgr/vol: volume create cmd deletes user-created pool if one of the pool passed is absent)
Actions #13

Updated by Jos Collin 5 months ago

  • Related to deleted (Backport #71117: squid: pybind/mgr/volumes: charmap not preserved via subvol clone)
Actions #14

Updated by Jos Collin 5 months ago

  • Related to deleted (Backport #71130: squid: pybind/mgr/volumes: make casesensitive name uniform in interface)
Actions #15

Updated by Jos Collin 5 months ago

  • Blocks deleted (Backport #71018: squid: mgr/vol: add a command to get the path of snapshots)
Actions #16

Updated by Jos Collin 5 months ago

  • Blocks Backport #71854: squid: mgr/volumes: subvolume create with --namespace-isolation doesn't isolate the namespace for subvolume of same name across groups added
  • Blocks Backport #71276: squid: mgr/vol: add "snapshot getpath" cmd for subvol v1 added
  • Blocks Backport #71150: squid: mgr/vol: volume create cmd deletes user-created pool if one of the pool passed is absent added
  • Blocks Backport #71117: squid: pybind/mgr/volumes: charmap not preserved via subvol clone added
  • Blocks Backport #71130: squid: pybind/mgr/volumes: make casesensitive name uniform in interface added
  • Blocks Backport #71018: squid: mgr/vol: add a command to get the path of snapshots added
Actions #17

Updated by Jos Collin 5 months ago

  • Status changed from QA Needs Rerun/Rebuilt to QA Building

@Rishabh Dave
Please take care of this. The tests failed because you've created multiple backport PRs for related fixes.
If the fixes are related, please add the commits to one backport PR. Thus everything goes together.

@Christopher Hoffman I've pulled https://github.com/ceph/ceph/pull/62918 and gave a rebuild.

Actions #18

Updated by Jos Collin 5 months ago

  • Status changed from QA Building to QA Testing
Actions #19

Updated by Jos Collin 5 months ago

  • Status changed from QA Testing to QA Needs Approval

@Christopher Hoffman This is ready for review now.

Actions #20

Updated by Jos Collin 5 months ago

  • Assignee changed from Jos Collin to Christopher Hoffman
Actions #21

Updated by Jos Collin about 1 month ago

  • Status changed from QA Needs Approval to QA Building
  • Assignee changed from Christopher Hoffman to Jos Collin

Suites:
- fs:volumes

Actions #22

Updated by Jos Collin about 1 month ago

  • Status changed from QA Building to QA Needs Approval
  • Assignee changed from Jos Collin to Christopher Hoffman
Actions

Also available in: Atom PDF