Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DNM] quincy: qa: avoid explicit set to client mountpoint as "/" #51049

Closed
wants to merge 1 commit into from

Conversation

mchangir
Copy link
Contributor

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


backport of #49912
parent tracker: https://tracker.ceph.com/issues/56446

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

This causes self.cephfs_mntpt to set as "/" by default which
overrides the config in ceph.conf. `test_client_cache_size`
updates ceph.conf with:

        client mountpoint = /subdir

However, the ceph-fuse mount command has --client_mountpoint explicitly
set as "/", thereby causing the root of the file system to get mounted which
confuses the test.

Fixes: http://tracker.ceph.com/issues/56446
Introduced-by: bf83eaa
Signed-off-by: Venky Shankar <vshankar@redhat.com>
(cherry picked from commit 4322dcc)

Conflicts:
	qa/tasks/cephfs/fuse_mount.py
	qa/tasks/cephfs/mount.py
	- fixed 'main' and 'quincy' branch conflicts
@mchangir mchangir added this to the quincy milestone Apr 12, 2023
@mchangir mchangir added the tests label Apr 12, 2023
@github-actions github-actions bot added the cephfs Ceph File System label Apr 12, 2023
@vshankar
Copy link
Contributor

jenkins retest this please

@vshankar vshankar added DNM and removed needs-qa labels May 15, 2023
@vshankar vshankar changed the title quincy: qa: avoid explicit set to client mountpoint as "/" [DNM] quincy: qa: avoid explicit set to client mountpoint as "/" May 15, 2023
@vshankar
Copy link
Contributor

Commits pulled into #50974

@vshankar vshankar closed this Jul 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cephfs Ceph File System DNM tests
Projects
None yet
2 participants