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

qa: synchronize kcephfs suites with fs/multimds #19856

Merged
merged 1 commit into from Feb 1, 2018
Merged

Conversation

batrick
Copy link
Member

@batrick batrick commented Jan 9, 2018

Fixes: http://tracker.ceph.com/issues/22627

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

@batrick
Copy link
Member Author

batrick commented Jan 9, 2018

@ukernel qa/cephfs/clusters/1-mds-4-client.yaml places 2 clients on the same machine, do you know if that would cause a problem for the kclient testing?

Also, the qa/cephfs/clusters/1-mds-*-client.yaml now colocate 1 client with the osds/mdss to reduce the number of machines required for testing. Do you forsee problems with that? I ask because I've been told at least OSD full testing can result in weird deadlocks when the kernel client is also on the same machine.

@ukernel
Copy link
Contributor

ukernel commented Jan 10, 2018

kernel mount and OSDs on the same machine can cause memory allocation deadlock. (OSD allocates memory from kernel. To satisfy the memory allocation, kernel flushes dirty data in cephfs and sends requests to the OSD)

For multiple kernel mounts on the same machine, I think the main problem is that killing kernel mount requires rebooting machine. If a test kills a mount, all mounts on that machine disappear.

Putting kernel mount and ceph daemons on the same machine has similar issue.

I think only tests in qa/tasks/cephfs kill mount. Maybe we can give these tests a separate cluster config

@batrick
Copy link
Member Author

batrick commented Jan 10, 2018

Great, thanks for explaining!

Fixes: http://tracker.ceph.com/issues/22627

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

@ukernel ukernel left a comment

Choose a reason for hiding this comment

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

LGTM

@batrick batrick merged commit 3689c0c into ceph:master Feb 1, 2018
batrick added a commit that referenced this pull request Feb 1, 2018
* refs/pull/19856/head:
	qa: synchronize kcephfs suites with fs/multimds

Reviewed-by: Zheng Yan <zyan@redhat.com>
@batrick batrick deleted the i22627 branch February 1, 2018 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants