)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"6171df9ae2e080e6033257df02e24a331862f655","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This commit takes the testing for list_extension from the volumes"},{"line_number":10,"context_line":"extensions client and uses it for the compute testing of"},{"line_number":11,"context_line":"list_extensions. This also fixes a skip decorator to use use"},{"line_number":12,"context_line":"requires_ext instead of calling is_extension_enabled directly."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Id08f39e09df0aea0d3c8f507f421c82ddf32900f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"AAAATn%2F8z%2Fc%3D","line":11,"updated":"2013-12-20 02:38:35.000000000","message":"Should a \u0027use\u0027 be deleted at the end of this line?","commit_id":"5684ea6ffee8cc0d71c6a793a1d4d029d1043b20"}],"tempest/api/compute/test_extensions.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"315f0441fbafb5cb4860399517d76f031df055b1","unresolved":false,"context_lines":[{"line_number":32,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":33,"context_line":"        ext \u003d self.config.compute_feature_enabled.api_extensions[0]"},{"line_number":34,"context_line":"        if ext \u003d\u003d \u0027all\u0027:"},{"line_number":35,"context_line":"            self.assertIn(\u0027Hosts\u0027, map(lambda x: x[\u0027name\u0027], extensions))"},{"line_number":36,"context_line":"        elif ext:"},{"line_number":37,"context_line":"            self.assertIn(ext, map(lambda x: x[\u0027name\u0027], extensions))"},{"line_number":38,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAUH%2F%2F9GM%3D","line":35,"updated":"2014-01-06 09:02:37.000000000","message":"Can we always use \u0027Hosts\u0027 extension? The volume test may be the same though.","commit_id":"5684ea6ffee8cc0d71c6a793a1d4d029d1043b20"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"8df313aa01785f4508a9332bc5896ade4cf49f81","unresolved":false,"context_lines":[{"line_number":32,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":33,"context_line":"        ext \u003d self.config.compute_feature_enabled.api_extensions[0]"},{"line_number":34,"context_line":"        if ext \u003d\u003d \u0027all\u0027:"},{"line_number":35,"context_line":"            self.assertIn(\u0027Hosts\u0027, map(lambda x: x[\u0027name\u0027], extensions))"},{"line_number":36,"context_line":"        elif ext:"},{"line_number":37,"context_line":"            self.assertIn(ext, map(lambda x: x[\u0027name\u0027], extensions))"},{"line_number":38,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAUH%2F%2F768%3D","line":35,"in_reply_to":"AAAAUH%2F%2F9GM%3D","updated":"2014-01-06 15:52:08.000000000","message":"Here we can assume that because the if statement above checks to see what extensions are enabled. If the config option has a single entry in the list, \u0027all\u0027, that means all extensions are supposed to be enabled on nova. \n\nI just chose Hosts here because I wanted an extension that problem wasn\u0027t ever going to be removed. (also one that wasn\u0027t many characters)","commit_id":"5684ea6ffee8cc0d71c6a793a1d4d029d1043b20"},{"author":{"_account_id":6983,"name":"YangLei","email":"iamhappg@aliyun.com","username":"YangLei"},"change_message_id":"0b447622db2ca36ef7a14e066a3a38a1892495ca","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        else:"},{"line_number":39,"context_line":"            raise self.skipException(\u0027There are not any extensions configured\u0027)"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    @test.requires_ext(extension\u003d\u0027os-consoles\u0027, service\u003d\u0027compute\u0027)"},{"line_number":42,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":43,"context_line":"    def test_get_extension(self):"},{"line_number":44,"context_line":"        # get the specified extensions"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAATn%2F80EQ%3D","line":41,"updated":"2013-12-20 02:07:51.000000000","message":"Do you plan to add the \"@test.requires_ext(extension\u003d\u0027os-consoles\u0027, service\u003d\u0027compute\u0027)\" in \ntempest/api/compute/v3/test_extensions.py\n?","commit_id":"5684ea6ffee8cc0d71c6a793a1d4d029d1043b20"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"0b32d2d71661b48352807236c7ac2b613a87c298","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        else:"},{"line_number":39,"context_line":"            raise self.skipException(\u0027There are not any extensions configured\u0027)"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    @test.requires_ext(extension\u003d\u0027os-consoles\u0027, service\u003d\u0027compute\u0027)"},{"line_number":42,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":43,"context_line":"    def test_get_extension(self):"},{"line_number":44,"context_line":"        # get the specified extensions"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAATn%2F8zrE%3D","line":41,"in_reply_to":"AAAATn%2F80EQ%3D","updated":"2014-01-03 21:23:43.000000000","message":"No, because  if you look at the v3 version of this test it is not running the GET on os-consoles, but instead servers. Servers is part of the core api in v3, so there is no need to specify that the test requires the extension.","commit_id":"5684ea6ffee8cc0d71c6a793a1d4d029d1043b20"}],"tempest/api/compute/v3/test_extensions.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"315f0441fbafb5cb4860399517d76f031df055b1","unresolved":false,"context_lines":[{"line_number":32,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":33,"context_line":"        ext \u003d self.config.compute_feature_enabled.api_v3_extensions[0]"},{"line_number":34,"context_line":"        if ext \u003d\u003d \u0027all\u0027:"},{"line_number":35,"context_line":"            self.assertIn(\u0027Hosts\u0027, map(lambda x: x[\u0027name\u0027], extensions))"},{"line_number":36,"context_line":"        elif ext:"},{"line_number":37,"context_line":"            self.assertIn(ext, map(lambda x: x[\u0027name\u0027], extensions))"},{"line_number":38,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAUH%2F%2F9GE%3D","line":35,"updated":"2014-01-06 09:02:37.000000000","message":"I have a same question as the v2 test_extenstions.py.","commit_id":"5684ea6ffee8cc0d71c6a793a1d4d029d1043b20"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"8df313aa01785f4508a9332bc5896ade4cf49f81","unresolved":false,"context_lines":[{"line_number":32,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":33,"context_line":"        ext \u003d self.config.compute_feature_enabled.api_v3_extensions[0]"},{"line_number":34,"context_line":"        if ext \u003d\u003d \u0027all\u0027:"},{"line_number":35,"context_line":"            self.assertIn(\u0027Hosts\u0027, map(lambda x: x[\u0027name\u0027], extensions))"},{"line_number":36,"context_line":"        elif ext:"},{"line_number":37,"context_line":"            self.assertIn(ext, map(lambda x: x[\u0027name\u0027], extensions))"},{"line_number":38,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAUH%2F%2F7v8%3D","line":35,"in_reply_to":"AAAAUH%2F%2F9GE%3D","updated":"2014-01-06 15:52:08.000000000","message":"Same as on v2 the config check ensures that Hosts is available.","commit_id":"5684ea6ffee8cc0d71c6a793a1d4d029d1043b20"}]}