)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c2dbcd3063b63e6cb5f1089ba482278a2eeca36e","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Missing response status code update"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds checking for missing response status code."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I4bc5aa756508b533919c283ce18f641fec50b30e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAXH%2F%2FsF8%3D","line":10,"updated":"2014-04-18 07:50:48.000000000","message":"mention the file or test folder for what all you are changing which help reviewers.","commit_id":"425d2550a87de90be0c45b8be1489395b6b63a4c"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"64a16108a442f430ec28d27972a2af55a5b0d8a8","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Missing response status code update"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds checking for missing response status code."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I4bc5aa756508b533919c283ce18f641fec50b30e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAXH%2F%2FrsM%3D","line":10,"in_reply_to":"AAAAXH%2F%2FsF8%3D","updated":"2014-04-18 09:11:53.000000000","message":"Done","commit_id":"425d2550a87de90be0c45b8be1489395b6b63a4c"},{"author":{"_account_id":7882,"name":"Yuiko Takada Mori","email":"yuiko-mori@nec.com","username":"YuikoTakada"},"change_message_id":"cb7989b66809689121bd0a69adaa1b4201d52f4c","unresolved":false,"context_lines":[{"line_number":9,"context_line":"In following test files response status code is not being checked -"},{"line_number":10,"context_line":"1. test_api_discovery.py"},{"line_number":11,"context_line":"2. test_chassis.py"},{"line_number":12,"context_line":"3. test_nodes.py"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This patch adds checking for missing response status code in above."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAXH%2F%2Fg08%3D","line":12,"updated":"2014-04-21 00:38:39.000000000","message":"Should add 4. test_ports.py","commit_id":"30f6c454cde058c5d3ddad0a635ea91318407504"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"91ffd048b2530c1d1461d9dd905dec4414c9ab26","unresolved":false,"context_lines":[{"line_number":9,"context_line":"In following test files response status code is not being checked -"},{"line_number":10,"context_line":"1. test_api_discovery.py"},{"line_number":11,"context_line":"2. test_chassis.py"},{"line_number":12,"context_line":"3. test_nodes.py"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This patch adds checking for missing response status code in above."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAXH%2F%2FgFg%3D","line":12,"in_reply_to":"AAAAXH%2F%2Fg08%3D","updated":"2014-04-21 07:32:46.000000000","message":"Thanks for suggestion, I\u0027ll do it in next patch.","commit_id":"30f6c454cde058c5d3ddad0a635ea91318407504"}],"tempest/api/baremetal/test_chassis.py":[{"author":{"_account_id":8824,"name":"Sergey Murashov","email":"smurashov@mirantis.com","username":"SergeyMurashov"},"change_message_id":"06e7d8a7cfc602384a0787f22e84749d8716e5e2","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        descr \u003d data_utils.rand_name(\u0027test-chassis-\u0027)"},{"line_number":38,"context_line":"        resp, chassis \u003d self.create_chassis(description\u003ddescr)"},{"line_number":39,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"},{"line_number":40,"context_line":"        self.assertEqual(chassis[\u0027description\u0027], descr)"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":43,"context_line":"    def test_create_chassis_unicode_description(self):"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_e93d57de","line":40,"updated":"2014-06-05 12:46:45.000000000","message":"ussually self.assertEqual(expected, actual)","commit_id":"37bcf8e6f49978c104ac2fcaf47d9c378111cac0"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"ed9bbceaef7a95b6cc37091f337d27692cd6c0d5","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        descr \u003d data_utils.rand_name(\u0027test-chassis-\u0027)"},{"line_number":38,"context_line":"        resp, chassis \u003d self.create_chassis(description\u003ddescr)"},{"line_number":39,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"},{"line_number":40,"context_line":"        self.assertEqual(chassis[\u0027description\u0027], descr)"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":43,"context_line":"    def test_create_chassis_unicode_description(self):"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_95084b29","line":40,"in_reply_to":"1ae5cdf2_e93d57de","updated":"2014-06-06 04:25:19.000000000","message":"As per comments of Matthew Treinish in  https://review.openstack.org/#/c/88524/ \n(Expected, Actual)  \u003d\u003d (Actual, Expected)","commit_id":"37bcf8e6f49978c104ac2fcaf47d9c378111cac0"},{"author":{"_account_id":8824,"name":"Sergey Murashov","email":"smurashov@mirantis.com","username":"SergeyMurashov"},"change_message_id":"06e7d8a7cfc602384a0787f22e84749d8716e5e2","unresolved":false,"context_lines":[{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":65,"context_line":"    def test_delete_chassis(self):"},{"line_number":66,"context_line":"        resp, body \u003d self.create_chassis()"},{"line_number":67,"context_line":"        uuid \u003d body[\u0027uuid\u0027]"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        resp \u003d self.delete_chassis(uuid)"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_04442a48","line":66,"updated":"2014-06-05 12:46:45.000000000","message":"unused variable \"resp\"","commit_id":"37bcf8e6f49978c104ac2fcaf47d9c378111cac0"},{"author":{"_account_id":7293,"name":"Elena Ezhova","email":"ezhovaelena19@gmail.com","username":"elena"},"change_message_id":"a4bde2e3ef4cb84f82462ec76247f4c7f034abad","unresolved":false,"context_lines":[{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":65,"context_line":"    def test_delete_chassis(self):"},{"line_number":66,"context_line":"        resp, body \u003d self.create_chassis()"},{"line_number":67,"context_line":"        uuid \u003d body[\u0027uuid\u0027]"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        resp \u003d self.delete_chassis(uuid)"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_fe598b35","line":66,"updated":"2014-06-05 11:41:27.000000000","message":"why don\u0027t you check the response status here?","commit_id":"37bcf8e6f49978c104ac2fcaf47d9c378111cac0"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"ed9bbceaef7a95b6cc37091f337d27692cd6c0d5","unresolved":false,"context_lines":[{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":65,"context_line":"    def test_delete_chassis(self):"},{"line_number":66,"context_line":"        resp, body \u003d self.create_chassis()"},{"line_number":67,"context_line":"        uuid \u003d body[\u0027uuid\u0027]"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        resp \u003d self.delete_chassis(uuid)"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_d5157340","line":66,"in_reply_to":"1ae5cdf2_04442a48","updated":"2014-06-06 04:25:19.000000000","message":"Done","commit_id":"37bcf8e6f49978c104ac2fcaf47d9c378111cac0"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"ed9bbceaef7a95b6cc37091f337d27692cd6c0d5","unresolved":false,"context_lines":[{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":65,"context_line":"    def test_delete_chassis(self):"},{"line_number":66,"context_line":"        resp, body \u003d self.create_chassis()"},{"line_number":67,"context_line":"        uuid \u003d body[\u0027uuid\u0027]"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        resp \u003d self.delete_chassis(uuid)"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_b50b8722","line":66,"in_reply_to":"1ae5cdf2_fe598b35","updated":"2014-06-06 04:25:19.000000000","message":"Done","commit_id":"37bcf8e6f49978c104ac2fcaf47d9c378111cac0"},{"author":{"_account_id":7293,"name":"Elena Ezhova","email":"ezhovaelena19@gmail.com","username":"elena"},"change_message_id":"a4bde2e3ef4cb84f82462ec76247f4c7f034abad","unresolved":false,"context_lines":[{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":74,"context_line":"    def test_update_chassis(self):"},{"line_number":75,"context_line":"        resp, body \u003d self.create_chassis()"},{"line_number":76,"context_line":"        uuid \u003d body[\u0027uuid\u0027]"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        new_description \u003d data_utils.rand_name(\u0027new-description-\u0027)"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_3e3f73b5","line":75,"updated":"2014-06-05 11:41:27.000000000","message":"ditto","commit_id":"37bcf8e6f49978c104ac2fcaf47d9c378111cac0"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"ed9bbceaef7a95b6cc37091f337d27692cd6c0d5","unresolved":false,"context_lines":[{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":74,"context_line":"    def test_update_chassis(self):"},{"line_number":75,"context_line":"        resp, body \u003d self.create_chassis()"},{"line_number":76,"context_line":"        uuid \u003d body[\u0027uuid\u0027]"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        new_description \u003d data_utils.rand_name(\u0027new-description-\u0027)"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_f510af4f","line":75,"in_reply_to":"1ae5cdf2_3e3f73b5","updated":"2014-06-06 04:25:19.000000000","message":"Done","commit_id":"37bcf8e6f49978c104ac2fcaf47d9c378111cac0"},{"author":{"_account_id":8824,"name":"Sergey Murashov","email":"smurashov@mirantis.com","username":"SergeyMurashov"},"change_message_id":"06e7d8a7cfc602384a0787f22e84749d8716e5e2","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        uuid \u003d body[\u0027uuid\u0027]"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        new_description \u003d data_utils.rand_name(\u0027new-description-\u0027)"},{"line_number":79,"context_line":"        resp, body \u003d (self.client.update_chassis(uuid,"},{"line_number":80,"context_line":"                      description\u003dnew_description))"},{"line_number":81,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":82,"context_line":"        resp, chassis \u003d self.client.show_chassis(uuid)"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_246686ed","line":79,"updated":"2014-06-05 12:46:45.000000000","message":"unused variable body","commit_id":"37bcf8e6f49978c104ac2fcaf47d9c378111cac0"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"ed9bbceaef7a95b6cc37091f337d27692cd6c0d5","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        uuid \u003d body[\u0027uuid\u0027]"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        new_description \u003d data_utils.rand_name(\u0027new-description-\u0027)"},{"line_number":79,"context_line":"        resp, body \u003d (self.client.update_chassis(uuid,"},{"line_number":80,"context_line":"                      description\u003dnew_description))"},{"line_number":81,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":82,"context_line":"        resp, chassis \u003d self.client.show_chassis(uuid)"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_151c3b69","line":79,"in_reply_to":"1ae5cdf2_246686ed","updated":"2014-06-06 04:25:19.000000000","message":"Done","commit_id":"37bcf8e6f49978c104ac2fcaf47d9c378111cac0"}],"tempest/api/baremetal/test_nodes.py":[{"author":{"_account_id":7882,"name":"Yuiko Takada Mori","email":"yuiko-mori@nec.com","username":"YuikoTakada"},"change_message_id":"cb7989b66809689121bd0a69adaa1b4201d52f4c","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        body \u003d self.create_node(self.chassis[\u0027uuid\u0027], **params)"},{"line_number":36,"context_line":"        node \u003d body[\u0027node\u0027]"},{"line_number":37,"context_line":"        resp \u003d body[\u0027response\u0027]"},{"line_number":38,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"        for key in params:"},{"line_number":41,"context_line":"            self.assertEqual(node[\u0027properties\u0027][key], params[key])"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXH%2F%2Fg00%3D","line":38,"updated":"2014-04-21 00:38:39.000000000","message":"Why enclose with single quotes?","commit_id":"30f6c454cde058c5d3ddad0a635ea91318407504"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"91ffd048b2530c1d1461d9dd905dec4414c9ab26","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        body \u003d self.create_node(self.chassis[\u0027uuid\u0027], **params)"},{"line_number":36,"context_line":"        node \u003d body[\u0027node\u0027]"},{"line_number":37,"context_line":"        resp \u003d body[\u0027response\u0027]"},{"line_number":38,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"        for key in params:"},{"line_number":41,"context_line":"            self.assertEqual(node[\u0027properties\u0027][key], params[key])"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXH%2F%2FgD4%3D","line":38,"in_reply_to":"AAAAXH%2F%2Fg00%3D","updated":"2014-04-21 07:32:46.000000000","message":"@Takada\nit is because type(resp[\u0027status\u0027]) is \u0027str\u0027\nIf I would had been using resp.status then type be \u0027int\u0027.\nSo here quotes are required.","commit_id":"30f6c454cde058c5d3ddad0a635ea91318407504"},{"author":{"_account_id":7293,"name":"Elena Ezhova","email":"ezhovaelena19@gmail.com","username":"elena"},"change_message_id":"a4bde2e3ef4cb84f82462ec76247f4c7f034abad","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"        resp, body \u003d self.client.update_node(node[\u0027uuid\u0027], properties\u003dnew_p)"},{"line_number":86,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":87,"context_line":"        resp, node \u003d self.client.show_node(node[\u0027uuid\u0027])"},{"line_number":88,"context_line":"        self._assertExpected(new_p, node[\u0027properties\u0027])"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_fe434b10","line":87,"updated":"2014-06-05 11:41:27.000000000","message":"please add a status check here too.","commit_id":"37bcf8e6f49978c104ac2fcaf47d9c378111cac0"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"ed9bbceaef7a95b6cc37091f337d27692cd6c0d5","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"        resp, body \u003d self.client.update_node(node[\u0027uuid\u0027], properties\u003dnew_p)"},{"line_number":86,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":87,"context_line":"        resp, node \u003d self.client.show_node(node[\u0027uuid\u0027])"},{"line_number":88,"context_line":"        self._assertExpected(new_p, node[\u0027properties\u0027])"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_d5df3367","line":87,"in_reply_to":"1ae5cdf2_fe434b10","updated":"2014-06-06 04:25:19.000000000","message":"Done","commit_id":"37bcf8e6f49978c104ac2fcaf47d9c378111cac0"}]}