)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"4a1d2980b58830d191f83e76a721a537a1f14dfc","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added test case to execute code of listing instances with"},{"line_number":10,"context_line":"invalid group id."},{"line_number":11,"context_line":"Fixed pep8 failure"},{"line_number":12,"context_line":"Change-Id: I28942c03bbf4e6046ece7f7c472cb7c1897b332d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9a061dce_d8c68e48","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":18},"updated":"2016-04-10 05:20:36.000000000","message":"This patch doesn\u0027t seem to fix pep8 failure.","commit_id":"89081eaf30bc8678886552edae254a7cf2d7da8e"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"8995fd553f703cc719e007336378baedf8f5f93b","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Improved test coverage"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added test case to execute code of listing instances with"},{"line_number":10,"context_line":"invalid group id."},{"line_number":11,"context_line":"Fixed pep8 failure"},{"line_number":12,"context_line":"Change-Id: I28942c03bbf4e6046ece7f7c472cb7c1897b332d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7a1b2928_4b8f5e23","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":7},"updated":"2016-04-11 17:04:47.000000000","message":"invalid -\u003e nonexistent","commit_id":"c29b36fbdef2112ad8c269af106cf36f2163a19f"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"3e9fa68093e8f5e49b6baf8c6af86cb1ff2caaf7","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Improved test coverage"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added test case to execute code of listing instances with"},{"line_number":10,"context_line":"invalid group id."},{"line_number":11,"context_line":"Fixed pep8 failure"},{"line_number":12,"context_line":"Change-Id: I28942c03bbf4e6046ece7f7c472cb7c1897b332d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7a1b2928_28dca5e3","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":7},"in_reply_to":"7a1b2928_4b8f5e23","updated":"2016-04-11 18:00:34.000000000","message":"Done","commit_id":"c29b36fbdef2112ad8c269af106cf36f2163a19f"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"8995fd553f703cc719e007336378baedf8f5f93b","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added test case to execute code of listing instances with"},{"line_number":10,"context_line":"invalid group id."},{"line_number":11,"context_line":"Fixed pep8 failure"},{"line_number":12,"context_line":"Change-Id: I28942c03bbf4e6046ece7f7c472cb7c1897b332d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7a1b2928_6b87ba0f","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":18},"updated":"2016-04-11 17:04:47.000000000","message":"need to remove this line.","commit_id":"c29b36fbdef2112ad8c269af106cf36f2163a19f"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"3e9fa68093e8f5e49b6baf8c6af86cb1ff2caaf7","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added test case to execute code of listing instances with"},{"line_number":10,"context_line":"invalid group id."},{"line_number":11,"context_line":"Fixed pep8 failure"},{"line_number":12,"context_line":"Change-Id: I28942c03bbf4e6046ece7f7c472cb7c1897b332d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7a1b2928_e8d10dcf","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":18},"in_reply_to":"7a1b2928_6b87ba0f","updated":"2016-04-11 18:00:34.000000000","message":"Done","commit_id":"c29b36fbdef2112ad8c269af106cf36f2163a19f"}],"nova/tests/unit/api/openstack/compute/test_server_groups.py":[{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"bcdbaa34d2090d5f685e6a69ad92752700fe8c08","unresolved":false,"context_lines":[{"line_number":252,"context_line":"        for member in members:"},{"line_number":253,"context_line":"            self.assertIn(member, result_members)"},{"line_number":254,"context_line":""},{"line_number":255,"context_line":"    def test_invalid_group(self):"},{"line_number":256,"context_line":"        self.assertRaises(webob.exc.HTTPNotFound,"},{"line_number":257,"context_line":"                          self.controller.show, self.req, \u0027invalid\u0027)"},{"line_number":258,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9a061dce_1aa483a4","line":255,"range":{"start_line":255,"start_character":8,"end_line":255,"end_character":26},"updated":"2016-04-05 05:50:15.000000000","message":"IMO, this method\u0027s name is not proper.\n\u0027test_display_members_with_invalid_group\u0027 would be better.","commit_id":"7571f813b8c2ba2531884c620993d37240b31eb0"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"4a1d2980b58830d191f83e76a721a537a1f14dfc","unresolved":false,"context_lines":[{"line_number":252,"context_line":"        for member in members:"},{"line_number":253,"context_line":"            self.assertIn(member, result_members)"},{"line_number":254,"context_line":""},{"line_number":255,"context_line":"    def test_display_members_with_invalid_group(self):"},{"line_number":256,"context_line":"        self.assertRaises(webob.exc.HTTPNotFound,"},{"line_number":257,"context_line":"                          self.controller.show, self.req, \u0027invalid\u0027)"},{"line_number":258,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a061dce_f8c9d25b","line":255,"range":{"start_line":255,"start_character":34,"end_line":255,"end_character":41},"updated":"2016-04-10 05:20:36.000000000","message":"s/invalid/nonexistent/","commit_id":"89081eaf30bc8678886552edae254a7cf2d7da8e"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"4a1d2980b58830d191f83e76a721a537a1f14dfc","unresolved":false,"context_lines":[{"line_number":254,"context_line":""},{"line_number":255,"context_line":"    def test_display_members_with_invalid_group(self):"},{"line_number":256,"context_line":"        self.assertRaises(webob.exc.HTTPNotFound,"},{"line_number":257,"context_line":"                          self.controller.show, self.req, \u0027invalid\u0027)"},{"line_number":258,"context_line":""},{"line_number":259,"context_line":"    def test_display_active_members_only(self):"},{"line_number":260,"context_line":"        ctx \u003d context.RequestContext(\u0027fake_user\u0027, \u0027fake\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a061dce_98b416d2","line":257,"range":{"start_line":257,"start_character":58,"end_line":257,"end_character":67},"updated":"2016-04-10 05:20:36.000000000","message":"Server-group id should be uuid format as http://developer.openstack.org/api-ref-compute-v2.1.html#showServerGroup\n\nThis case should be NotFound case as the expected exception, so here should be a uuid format value instead of \u0027invalid\u0027.\n\nWe need to change here like\n\n from nova.tests import uuidsentinel\n\n ...\n\n self.assertRaises(webob.exc.HTTPNotFound,\n                   self.controller.show, self.req, uuidsentinel.group)","commit_id":"89081eaf30bc8678886552edae254a7cf2d7da8e"}]}