)]}'
{"tempest/api/network/test_load_balancer.py":[{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"73a2c833e3fa7505132825ef7f2a2c13c40f0bd7","unresolved":false,"context_lines":[{"line_number":134,"context_line":"    @attr(type\u003d\u0027smoke\u0027)"},{"line_number":135,"context_line":"    def test_create_update_delete_member(self):"},{"line_number":136,"context_line":"        # Creates a member"},{"line_number":137,"context_line":"        resp, body \u003d self.client.create_member(\"10.0.9.47\", 80,"},{"line_number":138,"context_line":"                                               self.pool[\u0027id\u0027])"},{"line_number":139,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"},{"line_number":140,"context_line":"        member \u003d body[\u0027member\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2Bsks%3D","line":137,"updated":"2013-11-21 12:30:52.000000000","message":"It\u0027s better to generate a random ip instead of hard code to avoid conflict.","commit_id":"abbe563c1a7e76c84ac23f97c08e688aba90bb1d"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"37e8ce77756ccfc008930973b9846ff88c8efb67","unresolved":false,"context_lines":[{"line_number":134,"context_line":"    @attr(type\u003d\u0027smoke\u0027)"},{"line_number":135,"context_line":"    def test_create_update_delete_member(self):"},{"line_number":136,"context_line":"        # Creates a member"},{"line_number":137,"context_line":"        resp, body \u003d self.client.create_member(\"10.0.9.47\", 80,"},{"line_number":138,"context_line":"                                               self.pool[\u0027id\u0027])"},{"line_number":139,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"},{"line_number":140,"context_line":"        member \u003d body[\u0027member\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2B0Fc%3D","line":137,"updated":"2013-11-19 19:40:33.000000000","message":"can you point to other place where 9.46 was reused?","commit_id":"abbe563c1a7e76c84ac23f97c08e688aba90bb1d"},{"author":{"_account_id":6624,"name":"Stephen Gran","email":"sgran@debian.org","username":"sgran"},"change_message_id":"b017505040b61bdaa2f75f69252aff9861de516f","unresolved":false,"context_lines":[{"line_number":134,"context_line":"    @attr(type\u003d\u0027smoke\u0027)"},{"line_number":135,"context_line":"    def test_create_update_delete_member(self):"},{"line_number":136,"context_line":"        # Creates a member"},{"line_number":137,"context_line":"        resp, body \u003d self.client.create_member(\"10.0.9.47\", 80,"},{"line_number":138,"context_line":"                                               self.pool[\u0027id\u0027])"},{"line_number":139,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"},{"line_number":140,"context_line":"        member \u003d body[\u0027member\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2B0DA%3D","line":137,"in_reply_to":"AAAATn%2F%2B0Fc%3D","updated":"2013-11-19 19:51:32.000000000","message":"tempest/api/network/base.py:        resp, body \u003d cls.client.create_member(\"10.0.9.46\",","commit_id":"abbe563c1a7e76c84ac23f97c08e688aba90bb1d"},{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"d385e8b38f23cdaf09b29785bfdd58eeba121833","unresolved":false,"context_lines":[{"line_number":134,"context_line":"    @attr(type\u003d\u0027smoke\u0027)"},{"line_number":135,"context_line":"    def test_create_update_delete_member(self):"},{"line_number":136,"context_line":"        # Creates a member"},{"line_number":137,"context_line":"        resp, body \u003d self.client.create_member(\"10.0.9.47\", 80,"},{"line_number":138,"context_line":"                                               self.pool[\u0027id\u0027])"},{"line_number":139,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"},{"line_number":140,"context_line":"        member \u003d body[\u0027member\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2Bpas%3D","line":137,"in_reply_to":"AAAATn%2F%2BsYM%3D","updated":"2013-11-22 02:01:17.000000000","message":"Sure..","commit_id":"abbe563c1a7e76c84ac23f97c08e688aba90bb1d"},{"author":{"_account_id":6624,"name":"Stephen Gran","email":"sgran@debian.org","username":"sgran"},"change_message_id":"a41b3afe51cfabee4a530b167ec4af75fd350bc3","unresolved":false,"context_lines":[{"line_number":134,"context_line":"    @attr(type\u003d\u0027smoke\u0027)"},{"line_number":135,"context_line":"    def test_create_update_delete_member(self):"},{"line_number":136,"context_line":"        # Creates a member"},{"line_number":137,"context_line":"        resp, body \u003d self.client.create_member(\"10.0.9.47\", 80,"},{"line_number":138,"context_line":"                                               self.pool[\u0027id\u0027])"},{"line_number":139,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"},{"line_number":140,"context_line":"        member \u003d body[\u0027member\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BsYM%3D","line":137,"in_reply_to":"AAAATn%2F%2BseQ%3D","updated":"2013-11-21 13:16:35.000000000","message":"I agree with enikanorov on this one.  It\u0027s both overkill, and probably wrong for what we\u0027re trying to test.\n\nIf you agree, can you remove the -1?\n\nThanks,","commit_id":"abbe563c1a7e76c84ac23f97c08e688aba90bb1d"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"01fa323f498fc46b65d4883907f73170bd87c219","unresolved":false,"context_lines":[{"line_number":134,"context_line":"    @attr(type\u003d\u0027smoke\u0027)"},{"line_number":135,"context_line":"    def test_create_update_delete_member(self):"},{"line_number":136,"context_line":"        # Creates a member"},{"line_number":137,"context_line":"        resp, body \u003d self.client.create_member(\"10.0.9.47\", 80,"},{"line_number":138,"context_line":"                                               self.pool[\u0027id\u0027])"},{"line_number":139,"context_line":"        self.assertEqual(\u0027201\u0027, resp[\u0027status\u0027])"},{"line_number":140,"context_line":"        member \u003d body[\u0027member\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BseQ%3D","line":137,"in_reply_to":"AAAATn%2F%2Bsks%3D","updated":"2013-11-21 12:53:05.000000000","message":"No, random values in tests should be generally avoided.\nIt may make sense to get IP from some predefined pool, but for this particular task it is an overkill IMO.","commit_id":"abbe563c1a7e76c84ac23f97c08e688aba90bb1d"}]}