)]}'
{"tempest/api/compute/admin/test_hosts.py":[{"author":{"_account_id":8625,"name":"Anju Tiwari","username":"Anju5","inactive":true},"change_message_id":"66248f3bc629f5760328220b61f159c0d0e6f732","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        super(HostsAdminTestJSON, cls).setUpClass()"},{"line_number":35,"context_line":"        cls.client \u003d cls.os_adm.hosts_client"},{"line_number":36,"context_line":"        cls.non_admin_client \u003d cls.os.hosts_client"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def _get_host_name(self):"},{"line_number":39,"context_line":"        resp, hosts \u003d self.client.list_hosts()"},{"line_number":40,"context_line":"        self.assertEqual(200, resp.status)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FFug%3D","line":37,"updated":"2013-10-04 15:50:45.000000000","message":"Please mark @attr(type\u003d\u0027gate\u0027) here","commit_id":"59522cc6e2ecb8c4480a02821a9f271b1fe41c8b"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"f54e9694f2d8004247f5a5f3f97d5021ed815e4e","unresolved":false,"context_lines":[{"line_number":114,"context_line":"    def test_update_host(self):"},{"line_number":115,"context_line":"        hostname \u003d self._get_host_name()"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"        # Because libvirt driver doesn\u0027t implement both set_host_enabled and"},{"line_number":118,"context_line":"        # host_maintenance_mode, so an exception should be raised."},{"line_number":119,"context_line":"        self.assertRaises(exceptions.ComputeFault,"},{"line_number":120,"context_line":"                          self.client.update_host,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAS3%2F%2FEJ4%3D","line":117,"updated":"2013-10-05 02:26:09.000000000","message":"By other driver such as xen, vmware, this test case may be succeeded.\nSo I think this test case should be skip until libvirt driver implemented, or take care the xen/vmware case.","commit_id":"fdc29ecf887a5b21f888546e47873bb83cb7c2bd"},{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"8fcd9eeb4989e1795608b970f7890f96c501fb2e","unresolved":false,"context_lines":[{"line_number":114,"context_line":"    def test_update_host(self):"},{"line_number":115,"context_line":"        hostname \u003d self._get_host_name()"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"        # Because libvirt driver doesn\u0027t implement both set_host_enabled and"},{"line_number":118,"context_line":"        # host_maintenance_mode, so an exception should be raised."},{"line_number":119,"context_line":"        self.assertRaises(exceptions.ComputeFault,"},{"line_number":120,"context_line":"                          self.client.update_host,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAS3%2F%2FD4w%3D","line":117,"in_reply_to":"AAAAS3%2F%2FD54%3D","updated":"2013-10-05 09:47:13.000000000","message":"I just not very sure. I have asked Giulio if the usage is right.","commit_id":"fdc29ecf887a5b21f888546e47873bb83cb7c2bd"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"a7ad0cbab4a7375cd38e78bc336aaa5df6a58c79","unresolved":false,"context_lines":[{"line_number":114,"context_line":"    def test_update_host(self):"},{"line_number":115,"context_line":"        hostname \u003d self._get_host_name()"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"        # Because libvirt driver doesn\u0027t implement both set_host_enabled and"},{"line_number":118,"context_line":"        # host_maintenance_mode, so an exception should be raised."},{"line_number":119,"context_line":"        self.assertRaises(exceptions.ComputeFault,"},{"line_number":120,"context_line":"                          self.client.update_host,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAS3%2F%2FD54%3D","line":117,"in_reply_to":"AAAAS3%2F%2FD7U%3D","updated":"2013-10-05 09:19:49.000000000","message":"You already wrote the code to skip in Patch Set 4.\nDid I miss something?","commit_id":"fdc29ecf887a5b21f888546e47873bb83cb7c2bd"},{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"311ca9ac3855330ddc36e21c2bd5bac2fd75c59c","unresolved":false,"context_lines":[{"line_number":114,"context_line":"    def test_update_host(self):"},{"line_number":115,"context_line":"        hostname \u003d self._get_host_name()"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"        # Because libvirt driver doesn\u0027t implement both set_host_enabled and"},{"line_number":118,"context_line":"        # host_maintenance_mode, so an exception should be raised."},{"line_number":119,"context_line":"        self.assertRaises(exceptions.ComputeFault,"},{"line_number":120,"context_line":"                          self.client.update_host,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAS3%2F%2FD7U%3D","line":117,"in_reply_to":"AAAAS3%2F%2FEJ4%3D","updated":"2013-10-05 08:30:20.000000000","message":"hi, Masayuki, how could I handle the code to skip?","commit_id":"fdc29ecf887a5b21f888546e47873bb83cb7c2bd"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"765f6e689a27ad43f2fad60137be48eae3f2ace8","unresolved":false,"context_lines":[{"line_number":114,"context_line":""},{"line_number":115,"context_line":"    @testtools.skip("},{"line_number":116,"context_line":"        \"Skipped until libvirt driver has implemented set_host_enabled and \""},{"line_number":117,"context_line":"        \"host_maintenance_mode\")"},{"line_number":118,"context_line":"    @attr(type\u003d\u0027gate\u0027)"},{"line_number":119,"context_line":"    def test_update_host(self):"},{"line_number":120,"context_line":"        hostname \u003d self._get_host_name()"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATH%2F%2F%2B0Q%3D","line":117,"updated":"2013-10-07 01:49:54.000000000","message":"Do we have a bug report of the OpenStack launchpad already for this?\nif yes, please write the report number on the above with the decorator @skip_because().","commit_id":"e5520a4077b22eeac9eeef0897df5e2ff9420dec"},{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"f9fe1b2a09c4fed71062bf350a28db72fd0ab159","unresolved":false,"context_lines":[{"line_number":114,"context_line":""},{"line_number":115,"context_line":"    @testtools.skip("},{"line_number":116,"context_line":"        \"Skipped until libvirt driver has implemented set_host_enabled and \""},{"line_number":117,"context_line":"        \"host_maintenance_mode\")"},{"line_number":118,"context_line":"    @attr(type\u003d\u0027gate\u0027)"},{"line_number":119,"context_line":"    def test_update_host(self):"},{"line_number":120,"context_line":"        hostname \u003d self._get_host_name()"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATH%2F%2F%2Bv0%3D","line":117,"in_reply_to":"AAAATH%2F%2F%2B0Q%3D","updated":"2013-10-07 02:59:38.000000000","message":"Thanks, Ken\u0027ichi. I\u0027m afraid I haven\u0027t seen there is a  related bug for this, and it seems the \u0027@skip_because\u0027 hasn\u0027t been merged.","commit_id":"e5520a4077b22eeac9eeef0897df5e2ff9420dec"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"437819c561b56707ac89356c3b4710ccd0025956","unresolved":false,"context_lines":[{"line_number":114,"context_line":""},{"line_number":115,"context_line":"    @testtools.skip("},{"line_number":116,"context_line":"        \"Skipped until libvirt driver has implemented set_host_enabled and \""},{"line_number":117,"context_line":"        \"host_maintenance_mode\")"},{"line_number":118,"context_line":"    @attr(type\u003d\u0027gate\u0027)"},{"line_number":119,"context_line":"    def test_update_host(self):"},{"line_number":120,"context_line":"        hostname \u003d self._get_host_name()"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATH%2F%2F%2Bus%3D","line":117,"in_reply_to":"AAAATH%2F%2F%2Bv0%3D","updated":"2013-10-07 03:07:44.000000000","message":"OK, please report it if we need.\n\nand @skip_because has been merged already. (If53f2ef81d6bddbce284267216254b467046855f)\nCould you update your tempest repository?","commit_id":"e5520a4077b22eeac9eeef0897df5e2ff9420dec"}],"tempest/api/compute/admin/test_hosts_negative.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"ae1046328d9a72f30d9dba1b6454e9e84e8c9b3a","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        hostname \u003d hosts[0][\u0027host_name\u0027]"},{"line_number":42,"context_line":"        return hostname"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    @attr(type\u003d\u0027negative\u0027)"},{"line_number":45,"context_line":"    def test_list_hosts_with_non_existent_zone(self):"},{"line_number":46,"context_line":"        params \u003d {\u0027zone\u0027: \u0027xxx\u0027}"},{"line_number":47,"context_line":"        resp, hosts \u003d self.client.list_hosts(params)"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAATH%2F%2FhlQ%3D","line":44,"updated":"2013-10-15 00:38:34.000000000","message":"Why don\u0027t you add \u0027gate\u0027?","commit_id":"34302d89af5cb075a23fd44e81a42ad8f5936e36"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"ae1046328d9a72f30d9dba1b6454e9e84e8c9b3a","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        return hostname"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    @attr(type\u003d\u0027negative\u0027)"},{"line_number":45,"context_line":"    def test_list_hosts_with_non_existent_zone(self):"},{"line_number":46,"context_line":"        params \u003d {\u0027zone\u0027: \u0027xxx\u0027}"},{"line_number":47,"context_line":"        resp, hosts \u003d self.client.list_hosts(params)"},{"line_number":48,"context_line":"        self.assertEqual(0, len(hosts))"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAATH%2F%2FhlU%3D","line":45,"updated":"2013-10-15 00:38:34.000000000","message":"There are some \"non_existent_\" and \"nonexistent_\" test cases, but they are the same test purpose. So the part of their names should be the same to \"non_existent_\" or \"nonexistent_\".","commit_id":"34302d89af5cb075a23fd44e81a42ad8f5936e36"},{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"dab32096330359c4b482d54134564709e99a9d82","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        return hostname"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    @attr(type\u003d\u0027negative\u0027)"},{"line_number":45,"context_line":"    def test_list_hosts_with_non_existent_zone(self):"},{"line_number":46,"context_line":"        params \u003d {\u0027zone\u0027: \u0027xxx\u0027}"},{"line_number":47,"context_line":"        resp, hosts \u003d self.client.list_hosts(params)"},{"line_number":48,"context_line":"        self.assertEqual(0, len(hosts))"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAATH%2F%2FhLw%3D","line":45,"in_reply_to":"AAAATH%2F%2FhlU%3D","updated":"2013-10-15 04:05:30.000000000","message":"Thanks Ken\u0027ichi, good point! I\u0027ll fix them.","commit_id":"34302d89af5cb075a23fd44e81a42ad8f5936e36"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"ae1046328d9a72f30d9dba1b6454e9e84e8c9b3a","unresolved":false,"context_lines":[{"line_number":53,"context_line":"        self.assertRaises(exceptions.Unauthorized,"},{"line_number":54,"context_line":"                          self.non_admin_client.list_hosts)"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    @attr(type\u003d\u0027negative\u0027)"},{"line_number":57,"context_line":"    def test_show_host_detail_with_nonexist_hostname(self):"},{"line_number":58,"context_line":"        nonexitent_hostname \u003d data_utils.rand_name(\u0027rand_hostname\u0027)"},{"line_number":59,"context_line":"        self.assertRaises(exceptions.NotFound,"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAATH%2F%2FhlM%3D","line":56,"updated":"2013-10-15 00:38:34.000000000","message":"ditto","commit_id":"34302d89af5cb075a23fd44e81a42ad8f5936e36"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"57b8cb3c489be261f9133ca369747421f34d1f72","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        return hostname"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":45,"context_line":"    def test_list_hosts_with_nonexist_zone(self):"},{"line_number":46,"context_line":"        params \u003d {\u0027zone\u0027: \u0027xxx\u0027}"},{"line_number":47,"context_line":"        resp, hosts \u003d self.client.list_hosts(params)"},{"line_number":48,"context_line":"        self.assertEqual(0, len(hosts))"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAATH%2F%2FhKc%3D","line":45,"updated":"2013-10-15 04:33:26.000000000","message":"This test case does not contain assertRaises() call.\nand the similar test case test_list_hosts_with_a_blank_zone() is in positive tests. Should this test case be in negative ?","commit_id":"e7783f3a9a574b4befbbdb530616d75712f4c64b"},{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"bb9358c76d7de152b06327efefbb52dbeca8e32c","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        return hostname"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":45,"context_line":"    def test_list_hosts_with_nonexist_zone(self):"},{"line_number":46,"context_line":"        params \u003d {\u0027zone\u0027: \u0027xxx\u0027}"},{"line_number":47,"context_line":"        resp, hosts \u003d self.client.list_hosts(params)"},{"line_number":48,"context_line":"        self.assertEqual(0, len(hosts))"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAATH%2F%2FgJc%3D","line":45,"in_reply_to":"AAAATH%2F%2FhKc%3D","updated":"2013-10-15 10:47:45.000000000","message":"a very keen eye! Thanks!","commit_id":"e7783f3a9a574b4befbbdb530616d75712f4c64b"}]}