)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f052cb6f9d40b5ebaaa594fbe7a473f8eb608239","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Allowed Address Pair API Tests Enhancements"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Updated Allowed Address pair tests to include"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"-Update of  allowed-address-pair attribute of Port"},{"line_number":12,"context_line":"  Create a port without allowed-address-pair"},{"line_number":13,"context_line":"  Set allowed-address-pair attribute of port using port_update"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"1ae5cdf2_0d479478","line":10,"updated":"2014-06-24 07:06:52.000000000","message":"Please update the commit msg also something like-\n\nAdd test to update \u0027allowed-address-pair\u0027 of port\n\nThis patch adds a new test cases for\n- Update of  \u0027allowed-address-pair\u0027 attribute of Port\n      Create a port without allowed-address-pair\t\n      Update \u0027allowed-address-pair\u0027 attribute of port using      \n      \u0027port_update\u0027 request.","commit_id":"b7e85e46386440f54b9fd35b0a03e7df12c82d18"},{"author":{"_account_id":11670,"name":"Alok Kumar Maurya","email":"alok.maurya88@gmail.com","username":"alokmaurya"},"change_message_id":"41d4d542c0770046ce375cc3d734453ccc6ad9b5","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Allowed Address Pair API Tests Enhancements"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Updated Allowed Address pair tests to include"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"-Update of  allowed-address-pair attribute of Port"},{"line_number":12,"context_line":"  Create a port without allowed-address-pair"},{"line_number":13,"context_line":"  Set allowed-address-pair attribute of port using port_update"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"1ae5cdf2_710c723c","line":10,"in_reply_to":"1ae5cdf2_0d479478","updated":"2014-07-05 22:00:29.000000000","message":"Thanks Ghanshyam Mann\tfor  review comments .  I have updated commit message , new message will look  like\n\nAdd test to update \u0027allowed-address-pair\u0027 attribute of port\n\n-Create a port without allowed-address-pair\n-Set allowed-address-pair attribute of port using port_update\n-Validate that allowed-address-pair attribute of port is updated successfully","commit_id":"b7e85e46386440f54b9fd35b0a03e7df12c82d18"}],"tempest/api/network/test_allowed_address_pair.py":[{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"558c9ef52f09e680887c74660342be478c534ebc","unresolved":false,"context_lines":[{"line_number":70,"context_line":"        self.assertTrue(port, msg)"},{"line_number":71,"context_line":"        self._confirm_allowed_address_pair(port[0], self.ip_address)"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"        # Create second port without allowed address pair"},{"line_number":74,"context_line":"        resp, body \u003d self.client.create_port(network_id\u003dself.network[\u0027id\u0027])"},{"line_number":75,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"},{"line_number":76,"context_line":"        port_id \u003d body[\u0027port\u0027][\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":4,"id":"1ae5cdf2_1c01886a","line":73,"updated":"2014-06-24 04:36:39.000000000","message":"IMO, we need a new test method for this test.\n\ne.g.\n\ntest_update_port_with_address_pair","commit_id":"e2f35af5936dbd6d2398e84d53f7c3a4a0817141"},{"author":{"_account_id":11670,"name":"Alok Kumar Maurya","email":"alok.maurya88@gmail.com","username":"alokmaurya"},"change_message_id":"3d1072de1959fa630470fed2d8fadff2e28ac541","unresolved":false,"context_lines":[{"line_number":70,"context_line":"        self.assertTrue(port, msg)"},{"line_number":71,"context_line":"        self._confirm_allowed_address_pair(port[0], self.ip_address)"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"        # Create second port without allowed address pair"},{"line_number":74,"context_line":"        resp, body \u003d self.client.create_port(network_id\u003dself.network[\u0027id\u0027])"},{"line_number":75,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"},{"line_number":76,"context_line":"        port_id \u003d body[\u0027port\u0027][\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":4,"id":"1ae5cdf2_17ea43ac","line":73,"in_reply_to":"1ae5cdf2_1c01886a","updated":"2014-06-24 06:50:51.000000000","message":"Thanks Zhi Kun Liu  for   reviewing the test  ,  I have added a new method  for  update  test\n\ndef test_update_port_with_address_pair(self):","commit_id":"e2f35af5936dbd6d2398e84d53f7c3a4a0817141"},{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"558c9ef52f09e680887c74660342be478c534ebc","unresolved":false,"context_lines":[{"line_number":82,"context_line":"        ports \u003d body[\u0027ports\u0027]"},{"line_number":83,"context_line":"        port \u003d [p for p in ports if p[\u0027id\u0027] \u003d\u003d port_id]"},{"line_number":84,"context_line":"        msg \u003d \u0027Created port not found in list of ports returned by Neutron\u0027"},{"line_number":85,"context_line":"        self.assertTrue(port, msg)"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"        # Update alloed address pair attribute of second port"},{"line_number":88,"context_line":"        resp, body \u003d self.client.update_port(port_id,"}],"source_content_type":"text/x-python","patch_set":4,"id":"1ae5cdf2_3cf56484","line":85,"updated":"2014-06-24 04:36:39.000000000","message":"To be simple, you can use self.client.get_port(port_id) to confirm port creation.","commit_id":"e2f35af5936dbd6d2398e84d53f7c3a4a0817141"},{"author":{"_account_id":11670,"name":"Alok Kumar Maurya","email":"alok.maurya88@gmail.com","username":"alokmaurya"},"change_message_id":"3d1072de1959fa630470fed2d8fadff2e28ac541","unresolved":false,"context_lines":[{"line_number":82,"context_line":"        ports \u003d body[\u0027ports\u0027]"},{"line_number":83,"context_line":"        port \u003d [p for p in ports if p[\u0027id\u0027] \u003d\u003d port_id]"},{"line_number":84,"context_line":"        msg \u003d \u0027Created port not found in list of ports returned by Neutron\u0027"},{"line_number":85,"context_line":"        self.assertTrue(port, msg)"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"        # Update alloed address pair attribute of second port"},{"line_number":88,"context_line":"        resp, body \u003d self.client.update_port(port_id,"}],"source_content_type":"text/x-python","patch_set":4,"id":"1ae5cdf2_57f20b9c","line":85,"in_reply_to":"1ae5cdf2_3cf56484","updated":"2014-06-24 06:50:51.000000000","message":"Used self.client.show_port  , to verify  that port is created.\n  # Confirm  port is created\n    resp, body \u003d self.client.show_port(port_id)\n    self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])","commit_id":"e2f35af5936dbd6d2398e84d53f7c3a4a0817141"}]}