)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"b000c85bddf2c67eaed060c655cdeb2ca1a50e15","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Code dedup in tests for server._action_rebuild"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Externed common parts to setUp()"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"bp nova-tests-code-duplication"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"AAAAP3%2F%2Fm74%3D","line":9,"updated":"2013-07-15 21:46:12.000000000","message":"I would like a little more detail on this…","commit_id":"9f12f10756298714e313175318bad2a93563a528"}],"nova/tests/api/openstack/compute/plugins/v3/test_servers.py":[{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"725aea651a9cc477d36bb10559541dc4d54ed6a8","unresolved":false,"context_lines":[{"line_number":146,"context_line":"class ControllerTest(test.TestCase):"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"    def setUp(self):"},{"line_number":149,"context_line":"        super(ControllerTest, self).setUp()"},{"line_number":150,"context_line":"        self.flags(verbose\u003dTrue, use_ipv6\u003dFalse)"},{"line_number":151,"context_line":"        fakes.stub_out_rate_limiting(self.stubs)"},{"line_number":152,"context_line":"        fakes.stub_out_key_pair_funcs(self.stubs)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPn%2F%2Feig%3D","line":149,"updated":"2013-07-01 03:38:35.000000000","message":"this should be changed not here, but in previous patch\n\nas it is already fixed, so it means that this should be just rebased.","commit_id":"75a7b9139b618b4d3ed804a66ef972ebbaaa65d1"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"b000c85bddf2c67eaed060c655cdeb2ca1a50e15","unresolved":false,"context_lines":[{"line_number":1372,"context_line":"        self.server_delete_called \u003d False"},{"line_number":1373,"context_line":""},{"line_number":1374,"context_line":"        self.stubs.Set(db, \u0027instance_get_by_uuid\u0027,"},{"line_number":1375,"context_line":"            fakes.fake_instance_get(vm_state\u003dvm_states.ACTIVE,"},{"line_number":1376,"context_line":"                                    task_state\u003dtask_states.RESIZE_PREP))"},{"line_number":1377,"context_line":""},{"line_number":1378,"context_line":"        def instance_destroy_mock(*args, **kwargs):"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAP3%2F%2Fm7w%3D","line":1375,"updated":"2013-07-15 21:46:12.000000000","message":"This deindent doesn\u0027t seem to be related to your patch…","commit_id":"9f12f10756298714e313175318bad2a93563a528"}],"nova/tests/api/openstack/compute/test_servers.py":[{"author":{"_account_id":7763,"name":"Alexei Kornienko","email":"akornienko@mirantis.com","username":"akornienko"},"change_message_id":"8e8ae55517198be93fa6520313091a472db5b602","unresolved":false,"context_lines":[{"line_number":1365,"context_line":"class ServersControllerRebuildInstanceTest(ControllerTest):"},{"line_number":1366,"context_line":"    image_uuid \u003d \u002776fa36fc-c930-4bf3-8c8a-ea2a2420deb6\u0027"},{"line_number":1367,"context_line":"    image_href \u003d \u0027http://localhost/v2/fake/images/%s\u0027 % image_uuid"},{"line_number":1368,"context_line":"    body \u003d {"},{"line_number":1369,"context_line":"        \u0027rebuild\u0027: {"},{"line_number":1370,"context_line":"            \u0027name\u0027: \u0027new_name\u0027,"},{"line_number":1371,"context_line":"            \u0027imageRef\u0027: image_href,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPn%2F%2Fnxw%3D","line":1368,"updated":"2013-06-26 15:20:34.000000000","message":"Having this variables static means that they should be read only and you modify them in several tests.\nPlease create this variables in setUp method so we can be sure that tests are independent from each other.","commit_id":"858ba654b7111d968ba727d9bab3c94d23291e8a"},{"author":{"_account_id":7763,"name":"Alexei Kornienko","email":"akornienko@mirantis.com","username":"akornienko"},"change_message_id":"4db6c61d03cb74f804f64c4111cbedf209cb2a40","unresolved":false,"context_lines":[{"line_number":1453,"context_line":"        self.body[\u0027rebuild\u0027][\u0027accessIPv4\u0027] \u003d \u0027bad_format\u0027"},{"line_number":1454,"context_line":"        self.body[\u0027rebuild\u0027][\u0027accessIPv6\u0027] \u003d \u0027fead::1234\u0027"},{"line_number":1455,"context_line":"        self.body[\u0027rebuild\u0027][\u0027metadata\u0027][\u0027hello\u0027] \u003d \u0027world\u0027"},{"line_number":1456,"context_line":"        self.req.body \u003d jsonutils.dumps(self.body)"},{"line_number":1457,"context_line":"        self.assertRaises(webob.exc.HTTPBadRequest,"},{"line_number":1458,"context_line":"                          self.controller._action_rebuild,"},{"line_number":1459,"context_line":"                          self.req, FAKE_UUID, self.body)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2FlrQ%3D","line":1456,"updated":"2013-06-27 08:07:24.000000000","message":"You make changes to static variable. By this you can make tests depend on each other. Please make this variable non static","commit_id":"bab38d3625ca5c46bdff6043cf3405c46a35864a"}]}