)]}'
{"tempest/api/compute/admin/test_hypervisor.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"bb94542823c7d6a5dad45a654510266a15d8a9c9","unresolved":false,"context_lines":[{"line_number":95,"context_line":"            # because hypervisors might be disabled, this loops looking"},{"line_number":96,"context_line":"            # for any good hit."},{"line_number":97,"context_line":"            resp, uptime \u003d self.client.get_hypervisor_uptime(hyper[\u0027id\u0027])"},{"line_number":98,"context_line":"            self.assertEqual(200, resp.status)"},{"line_number":99,"context_line":"            if len(uptime) \u003e 0:"},{"line_number":100,"context_line":"                has_valid_uptime \u003d True"},{"line_number":101,"context_line":"                break"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F8%2BjQ%3D","line":98,"updated":"2013-12-18 07:13:42.000000000","message":"I think this assert fails if the hypervisor is disabled.\nIn my environment, the response code is \u0027400\u0027 when the nova-compute is down.\nDo I miss something?","commit_id":"fcd0889b25491605ef7e83c47bf4b29cb9d2a22d"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"73b6a80cc4d40f6e78b02c4183c71c4a352f7197","unresolved":false,"context_lines":[{"line_number":95,"context_line":"            # because hypervisors might be disabled, this loops looking"},{"line_number":96,"context_line":"            # for any good hit."},{"line_number":97,"context_line":"            resp, uptime \u003d self.client.get_hypervisor_uptime(hyper[\u0027id\u0027])"},{"line_number":98,"context_line":"            self.assertEqual(200, resp.status)"},{"line_number":99,"context_line":"            if len(uptime) \u003e 0:"},{"line_number":100,"context_line":"                has_valid_uptime \u003d True"},{"line_number":101,"context_line":"                break"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F89nE%3D","line":98,"in_reply_to":"AAAATn%2F8%2BjQ%3D","updated":"2013-12-18 11:00:40.000000000","message":"no you did not, I did this wrong :)","commit_id":"fcd0889b25491605ef7e83c47bf4b29cb9d2a22d"},{"author":{"_account_id":5586,"name":"Ivan-Zhu","email":"zhubo198511@gmail.com","username":"ivan-zhu"},"change_message_id":"a78d93606a3d90426a0918b56bccfd252281c13e","unresolved":false,"context_lines":[{"line_number":94,"context_line":"        for hyper in hypers:"},{"line_number":95,"context_line":"            # because hypervisors might be disabled, this loops looking"},{"line_number":96,"context_line":"            # for any good hit."},{"line_number":97,"context_line":"            resp, uptime \u003d self.client.get_hypervisor_uptime(hyper[\u0027id\u0027])"},{"line_number":98,"context_line":"            if (resp.status \u003d\u003d 200) and (len(uptime) \u003e 0):"},{"line_number":99,"context_line":"                has_valid_uptime \u003d True"},{"line_number":100,"context_line":"                break"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F883Q%3D","line":97,"updated":"2013-12-18 13:18:48.000000000","message":"It will still raise exception in get_hypervisor_uptime, when the nova-compute is down.\n{u\u0027badRequest\u0027: {u\u0027message\u0027: u\u0027Compute service of xxx is unavailable at this time.\u0027, u\u0027code\u0027: 400}}","commit_id":"bf900a4da74b426e6963e53a02aac7adad0fc18c"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"4393d5517755e462ab0a9ebd0519266c72702870","unresolved":false,"context_lines":[{"line_number":94,"context_line":"        for hyper in hypers:"},{"line_number":95,"context_line":"            # because hypervisors might be disabled, this loops looking"},{"line_number":96,"context_line":"            # for any good hit."},{"line_number":97,"context_line":"            resp, uptime \u003d self.client.get_hypervisor_uptime(hyper[\u0027id\u0027])"},{"line_number":98,"context_line":"            if (resp.status \u003d\u003d 200) and (len(uptime) \u003e 0):"},{"line_number":99,"context_line":"                has_valid_uptime \u003d True"},{"line_number":100,"context_line":"                break"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F83sI%3D","line":97,"in_reply_to":"AAAATn%2F883Q%3D","updated":"2013-12-19 11:13:10.000000000","message":"good point","commit_id":"bf900a4da74b426e6963e53a02aac7adad0fc18c"}]}