Skip to content

quincy: qa/cephfs: fix test_single_path_authorize_on_nonalphanumeric_fsname#58559

Closed
rishabh-d-dave wants to merge 1 commit intoceph:quincyfrom
rishabh-d-dave:wip-66930-quincy
Closed

quincy: qa/cephfs: fix test_single_path_authorize_on_nonalphanumeric_fsname#58559
rishabh-d-dave wants to merge 1 commit intoceph:quincyfrom
rishabh-d-dave:wip-66930-quincy

Conversation

@rishabh-d-dave
Copy link
Contributor

backport tracker: https://tracker.ceph.com/issues/66930


backport of #58311
parent tracker: https://tracker.ceph.com/issues/66077

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

This test deletes the CephFS already present on the cluster at the very
beginning and unmounts the first client beforehand. But it leaves the
second client mounted on this deleted CephFS that doesn't exist for the
rest of the test. And then at the very end of this test it attempts to
remount the second client (during tearDown()) which hangs and causes
test runner to crash.

Unmount the second client beforehand to prevent the bug and delete
mount_b object to avoid confusion for the readers in future about
whether or not 2nd mountpoint exists.

Fixes: https://tracker.ceph.com/issues/66077
Signed-off-by: Rishabh Dave <ridave@redhat.com>
(cherry picked from commit 2130ec8)
@rishabh-d-dave rishabh-d-dave added this to the quincy milestone Jul 12, 2024
@github-actions github-actions bot added the cephfs Ceph File System label Jul 12, 2024
@rishabh-d-dave rishabh-d-dave requested a review from a team July 12, 2024 14:50
@rishabh-d-dave
Copy link
Contributor Author

jenkins test make check

1 similar comment
@lxbsz
Copy link
Member

lxbsz commented Aug 2, 2024

jenkins test make check

@lxbsz
Copy link
Member

lxbsz commented Aug 19, 2024

Copy link
Contributor

@vshankar vshankar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rishabh-d-dave
Copy link
Contributor Author

jenkins test make check

@rishabh-d-dave
Copy link
Contributor Author

jenkins test api

@rishabh-d-dave
Copy link
Contributor Author

self.mount_b is absent in the test because CLIENTS_REQUIRED is set 1 for this test class on Quincy branch. So this patch is not required.

@rishabh-d-dave rishabh-d-dave deleted the wip-66930-quincy branch October 24, 2024 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cephfs Ceph File System tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants