)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"fe64c92e6d9efe36d0b596619e09aa3390ad4f5d","unresolved":false,"context_lines":[{"line_number":10,"context_line":"to the internal functionality."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ieff9ebd2d28f7ad1bf8d68ba79b0dac40d6df82b"},{"line_number":13,"context_line":"Closes-Bug: #1243685"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATX%2F%2F%2BKE%3D","line":13,"updated":"2013-10-23 14:02:17.000000000","message":"I think the Change-Id line needs to be the last line","commit_id":"ec7f0840d5ece31231d983362c8a0f2637f9613d"}],"nova/api/openstack/compute/plugins/v3/servers.py":[{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"8d75aef70a8df4e26f93c57dd22cdf553d06f387","unresolved":false,"context_lines":[{"line_number":546,"context_line":"        sort_dir \u003d search_opts.pop(\u0027sort_dir\u0027) if \u0027sort_dir\u0027 in search_opts \\"},{"line_number":547,"context_line":"                                                    else \u0027desc\u0027"},{"line_number":548,"context_line":"        sort_key \u003d search_opts.pop(\u0027sort_key\u0027) if \u0027sort_key\u0027 in search_opts \\"},{"line_number":549,"context_line":"                                                    else \u0027created_at\u0027"},{"line_number":550,"context_line":""},{"line_number":551,"context_line":"        limit, marker \u003d common.get_limit_and_marker(req)"},{"line_number":552,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F%2Bya0%3D","line":549,"updated":"2013-11-20 10:50:28.000000000","message":"How about use \"search_opts.pop(\u0027sort_dir, \u0027desc\u0027)\" directly?","commit_id":"6fe622780a02cc6b0d2ede35b26676338def3727"}],"nova/tests/api/openstack/compute/plugins/v3/test_servers.py":[{"author":{"_account_id":7040,"name":"GuoHui LIu","email":"liuguohui@gmail.com","username":"guohliu"},"change_message_id":"085a51e1b34110b7627771d8e0cfad91bfcd9f9e","unresolved":false,"context_lines":[{"line_number":661,"context_line":""},{"line_number":662,"context_line":"        self.stubs.Set(compute_api.API, \u0027get_all\u0027, fake_get_all)"},{"line_number":663,"context_line":""},{"line_number":664,"context_line":"        req \u003d fakes.HTTPRequest.\\"},{"line_number":665,"context_line":"            blank(\u0027/fake/servers?sort_key\u003dcreated_at\u0026sort_dir\u003dasc\u0027)"},{"line_number":666,"context_line":"        servers \u003d self.controller.detail(req)[\u0027servers\u0027]"},{"line_number":667,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2Fz%2BE%3D","line":664,"updated":"2013-10-28 16:46:12.000000000","message":"for v3 test, we should use HTTPRequestV3.","commit_id":"c367955f5f31b07cafdc9bd9484057d978634297"},{"author":{"_account_id":7040,"name":"GuoHui LIu","email":"liuguohui@gmail.com","username":"guohliu"},"change_message_id":"085a51e1b34110b7627771d8e0cfad91bfcd9f9e","unresolved":false,"context_lines":[{"line_number":694,"context_line":""},{"line_number":695,"context_line":"        self.stubs.Set(compute_api.API, \u0027get_all\u0027, fake_get_all)"},{"line_number":696,"context_line":""},{"line_number":697,"context_line":"        req \u003d fakes.HTTPRequest.\\"},{"line_number":698,"context_line":"            blank(\u0027/fake/servers?sort_key\u003dcreated_at\u0026sort_dir\u003ddesc\u0027)"},{"line_number":699,"context_line":"        servers \u003d self.controller.detail(req)[\u0027servers\u0027]"},{"line_number":700,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2Fz98%3D","line":697,"updated":"2013-10-28 16:46:12.000000000","message":"ditto","commit_id":"c367955f5f31b07cafdc9bd9484057d978634297"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"7ffa60b1607175168123b24267e0b02d2ba6ede8","unresolved":false,"context_lines":[{"line_number":662,"context_line":"        self.stubs.Set(compute_api.API, \u0027get_all\u0027, fake_get_all)"},{"line_number":663,"context_line":""},{"line_number":664,"context_line":"        req \u003d fakes.HTTPRequestV3.\\"},{"line_number":665,"context_line":"            blank(\u0027/fake/servers?sort_key\u003dcreated_at\u0026sort_dir\u003dasc\u0027)"},{"line_number":666,"context_line":"        servers \u003d self.controller.detail(req)[\u0027servers\u0027]"},{"line_number":667,"context_line":""},{"line_number":668,"context_line":"        self.assertTrue(len(servers) \u003d\u003d 5)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FkjM%3D","line":665,"updated":"2013-11-01 05:19:01.000000000","message":"Rather than use \\ to split the line, please break on the parantheses, eg:\n\nreq \u003d fakes.HTTPRequestV3.blank(\n     \u0027/servers?sort_key\u003dcreated_at\u0026sort_dir\u003dasc\u0027)\n\nAlso not that for V3 you do not need the tenant id prefix for the url","commit_id":"9e50ea3186371d66357f7b420d07599f20c4916f"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"7ffa60b1607175168123b24267e0b02d2ba6ede8","unresolved":false,"context_lines":[{"line_number":695,"context_line":"        self.stubs.Set(compute_api.API, \u0027get_all\u0027, fake_get_all)"},{"line_number":696,"context_line":""},{"line_number":697,"context_line":"        req \u003d fakes.HTTPRequestV3.\\"},{"line_number":698,"context_line":"            blank(\u0027/fake/servers?sort_key\u003dcreated_at\u0026sort_dir\u003ddesc\u0027)"},{"line_number":699,"context_line":"        servers \u003d self.controller.detail(req)[\u0027servers\u0027]"},{"line_number":700,"context_line":""},{"line_number":701,"context_line":"        self.assertTrue(len(servers) \u003d\u003d 5)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FkjI%3D","line":698,"updated":"2013-11-01 05:19:01.000000000","message":"same issue as above with the linebreak","commit_id":"9e50ea3186371d66357f7b420d07599f20c4916f"}],"nova/tests/api/openstack/compute/test_servers.py":[{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"7ffa60b1607175168123b24267e0b02d2ba6ede8","unresolved":false,"context_lines":[{"line_number":642,"context_line":"        self.stubs.Set(compute_api.API, \u0027get_all\u0027, fake_get_all)"},{"line_number":643,"context_line":""},{"line_number":644,"context_line":"        req \u003d fakes.HTTPRequest.\\"},{"line_number":645,"context_line":"            blank(\u0027/fake/servers?sort_key\u003dcreated_at\u0026sort_dir\u003dasc\u0027)"},{"line_number":646,"context_line":"        servers \u003d self.controller.detail(req)[\u0027servers\u0027]"},{"line_number":647,"context_line":""},{"line_number":648,"context_line":"        self.assertTrue(len(servers) \u003d\u003d 5)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FkjE%3D","line":645,"updated":"2013-11-01 05:19:01.000000000","message":"please break lines on parantheses as mentioned in in the v3 version of this test","commit_id":"9e50ea3186371d66357f7b420d07599f20c4916f"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"7ffa60b1607175168123b24267e0b02d2ba6ede8","unresolved":false,"context_lines":[{"line_number":675,"context_line":"        self.stubs.Set(compute_api.API, \u0027get_all\u0027, fake_get_all)"},{"line_number":676,"context_line":""},{"line_number":677,"context_line":"        req \u003d fakes.HTTPRequest.\\"},{"line_number":678,"context_line":"            blank(\u0027/fake/servers?sort_key\u003dcreated_at\u0026sort_dir\u003ddesc\u0027)"},{"line_number":679,"context_line":"        servers \u003d self.controller.detail(req)[\u0027servers\u0027]"},{"line_number":680,"context_line":""},{"line_number":681,"context_line":"        self.assertTrue(len(servers) \u003d\u003d 5)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FkjA%3D","line":678,"updated":"2013-11-01 05:19:01.000000000","message":"same line break issue","commit_id":"9e50ea3186371d66357f7b420d07599f20c4916f"}]}