{"action":"created","issue":{"url":"https://api.github.com/repos/ceph/ceph/issues/6884","labels_url":"https://api.github.com/repos/ceph/ceph/issues/6884/labels{/name}","comments_url":"https://api.github.com/repos/ceph/ceph/issues/6884/comments","events_url":"https://api.github.com/repos/ceph/ceph/issues/6884/events","html_url":"https://github.com/ceph/ceph/pull/6884","id":121433791,"number":6884,"title":"tests: fix failure for osd-scrub-snap.sh","user":{"login":"mslovy","id":9523883,"avatar_url":"https://avatars.githubusercontent.com/u/9523883?v=3","gravatar_id":"","url":"https://api.github.com/users/mslovy","html_url":"https://github.com/mslovy","followers_url":"https://api.github.com/users/mslovy/followers","following_url":"https://api.github.com/users/mslovy/following{/other_user}","gists_url":"https://api.github.com/users/mslovy/gists{/gist_id}","starred_url":"https://api.github.com/users/mslovy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mslovy/subscriptions","organizations_url":"https://api.github.com/users/mslovy/orgs","repos_url":"https://api.github.com/users/mslovy/repos","events_url":"https://api.github.com/users/mslovy/events{/privacy}","received_events_url":"https://api.github.com/users/mslovy/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-12-10T09:04:17Z","updated_at":"2015-12-10T09:39:36Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ceph/ceph/pulls/6884","html_url":"https://github.com/ceph/ceph/pull/6884","diff_url":"https://github.com/ceph/ceph/pull/6884.diff","patch_url":"https://github.com/ceph/ceph/pull/6884.patch"},"body":"Reason to get a false negative\r\nonly one osd, when calling kill_daemons $dir KILL osd, which means kill -9\r\nshutdown() would not be called to notify monitor so that it remain active+clean and osd up until monitor got a mon_report timeout.\r\nThus wait_for_clean lose it function and when we call ceph pg scrub, we got an EAGAIN error and monitor will ignore the scrub message because of osd not up\r\n\r\nSigned-off-by: Ning Yao <zay11022@gmail.com>"},"comment":{"url":"https://api.github.com/repos/ceph/ceph/issues/comments/163553807","html_url":"https://github.com/ceph/ceph/pull/6884#issuecomment-163553807","issue_url":"https://api.github.com/repos/ceph/ceph/issues/6884","id":163553807,"user":{"login":"mslovy","id":9523883,"avatar_url":"https://avatars.githubusercontent.com/u/9523883?v=3","gravatar_id":"","url":"https://api.github.com/users/mslovy","html_url":"https://github.com/mslovy","followers_url":"https://api.github.com/users/mslovy/followers","following_url":"https://api.github.com/users/mslovy/following{/other_user}","gists_url":"https://api.github.com/users/mslovy/gists{/gist_id}","starred_url":"https://api.github.com/users/mslovy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mslovy/subscriptions","organizations_url":"https://api.github.com/users/mslovy/orgs","repos_url":"https://api.github.com/users/mslovy/repos","events_url":"https://api.github.com/users/mslovy/events{/privacy}","received_events_url":"https://api.github.com/users/mslovy/received_events","type":"User","site_admin":false},"created_at":"2015-12-10T09:39:36Z","updated_at":"2015-12-10T09:39:36Z","body":"@dachary  ping,  using TERM instead of KILL may not actually kill the process under exception cases. Under normal case, it works. I think if the process cannot TERM normally, it actually exists some issues so that a test failure should be reported. \r\nThus, for what purpose  we use KILL signal in all test modules?\r\nIf this is needed to prevent something, sleep seconds (like sleep 5) before wait_for_clean may be a better choice"}}