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: introduce rbd-mirror thrasher to existing tests #21541

Merged
merged 2 commits into from Apr 25, 2018

Conversation

dillaman
Copy link

No description provided.

@tchaikov
Copy link
Contributor

retest this please

@trociny
Copy link
Contributor

trociny commented Apr 23, 2018

@dillaman not sure it is problem with the test or it reveals a real problem: I am observing "stop/start/restart mirror via admin socket" test fails after stopping an image replayer via admin socket.

In [1,2] it stopped the replayer on one instance, but when checking the status via admin socket another instance kept reporting it was running there. In [3] the status check via admin socket passed (no instance reported it running) but the status in mirror pool directory was reporting it 'up+replaying'.

[1] http://qa-proxy.ceph.com/teuthology/trociny-2018-04-21_06:50:46-rbd-wip-mgolub-testing-distro-basic-smithi/2425070/teuthology.log
[2] http://qa-proxy.ceph.com/teuthology/trociny-2018-04-21_18:49:24-rbd-wip-mgolub-testing-distro-basic-smithi/2425388/teuthology.log
[3] http://qa-proxy.ceph.com/teuthology/trociny-2018-04-21_06:50:46-rbd-wip-mgolub-testing-distro-basic-smithi/2425083/teuthology.log

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
Fixes: http://tracker.ceph.com/issues/18753
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
@dillaman
Copy link
Author

Tweaked it a little bit into two classes of tests: (1) daemons managed by teuthology for thrashed tests (which will skip anything related to stop/starting services, and (2) daemons managed by the workunits.

http://pulpito.ceph.com/jdillaman-2018-04-23_15:35:34-rbd-master-distro-basic-smithi/
http://pulpito.ceph.com/jdillaman-2018-04-23_16:17:33-rbd-master-distro-basic-smithi/

Copy link
Contributor

@trociny trociny left a comment

Choose a reason for hiding this comment

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

LGTM

@trociny
Copy link
Contributor

trociny commented Apr 24, 2018

@dillaman It looks like the thrasher was able to trigger a rbd-mirror daemon crash:
http://pulpito.ceph.com/trociny-2018-04-24_08:17:18-rbd-wip-mgolub-testing-distro-basic-smithi/2434398/

@trociny trociny merged commit 808503f into ceph:master Apr 25, 2018
@dillaman dillaman deleted the wip-18753 branch April 25, 2018 13:09
@dillaman
Copy link
Author

@trociny ... just to close the loop, the crash was caused by a bug in the Objecter: #21649

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants