{"issue":{"state":"open","pull_request":{"diff_url":"https://github.com/autotest/autotest/pull/60.diff","html_url":"https://github.com/autotest/autotest/pull/60","patch_url":"https://github.com/autotest/autotest/pull/60.patch"},"user":{"avatar_url":"https://secure.gravatar.com/avatar/e69d88e28c8a4b9c5ebee4b927322ceb?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"e69d88e28c8a4b9c5ebee4b927322ceb","url":"https://api.github.com/users/zhouqt","login":"zhouqt","id":35799},"milestone":null,"labels":[],"assignee":null,"updated_at":"2011-11-11T14:45:54Z","comments":6,"closed_at":null,"url":"https://api.github.com/repos/autotest/autotest/issues/60","body":"Some scripts use HumanMonitor command to get guest's status, which makes these test fail when using QMPMonitor. This patchset will use monitor.verify_status() method instead, this method will work in both HumanMonitor and QMPMonitor.","html_url":"https://github.com/autotest/autotest/issues/60","title":"[KVM Test] Get guest status with verify_status method of monitor","number":60,"id":2162031,"created_at":"2011-11-07T12:15:49Z"},"action":"created","comment":{"user":{"avatar_url":"https://secure.gravatar.com/avatar/743e19d84263b1b4b4d8d904daa392f1?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"743e19d84263b1b4b4d8d904daa392f1","url":"https://api.github.com/users/lmr","login":"lmr","id":296807},"updated_at":"2011-11-11T14:45:54Z","url":"https://api.github.com/repos/autotest/autotest/issues/comments/2708757","body":"@zhouqt: Ok, now that I've finally took the time to read all the patches and think about things, it occurred to me that all our methods implemented so far that are named verify_foo return an exception in case condition foo is not satisfied. Although your patch looks fine from a coding perspective, I've modified it and sent pull request #69. Would you please take a look at it? I'd need to test it a bit, but that's the idea. What do you think?","id":2708757,"created_at":"2011-11-11T14:45:54Z"}}