)]}'
{"tempest/api/network/test_load_balancer.py":[{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"501dceb67b5e1bc113db583a47cb7b1ccaffbc1c","unresolved":false,"context_lines":[{"line_number":211,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":212,"context_line":"    def test_get_lb_pool_stats(self):"},{"line_number":213,"context_line":"        # Verify the details of pool stats"},{"line_number":214,"context_line":"        resp, body \u003d self.client.list_lb_pool_stats(self.pool[\u0027id\u0027])"},{"line_number":215,"context_line":"        stats \u003d body[\u0027stats\u0027]"},{"line_number":216,"context_line":"        self.assertIsNotNone(stats)"},{"line_number":217,"context_line":"        self.assertIn(\"bytes_in\", stats)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUX%2F%2FuY8%3D","line":214,"updated":"2014-02-11 08:07:03.000000000","message":"suggest adding a status code checking like other api tests.","commit_id":"fbd5c5148f736fce8288a60615c5049fe2191dc1"},{"author":{"_account_id":2238,"name":"rajalakshmi-ganesan","email":"rajalakshmi.girish@hpe.com","username":"rajalakshmi-ganesan"},"change_message_id":"db441536323edabea26375e15bd40b614540c075","unresolved":false,"context_lines":[{"line_number":211,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":212,"context_line":"    def test_get_lb_pool_stats(self):"},{"line_number":213,"context_line":"        # Verify the details of pool stats"},{"line_number":214,"context_line":"        resp, body \u003d self.client.list_lb_pool_stats(self.pool[\u0027id\u0027])"},{"line_number":215,"context_line":"        stats \u003d body[\u0027stats\u0027]"},{"line_number":216,"context_line":"        self.assertIsNotNone(stats)"},{"line_number":217,"context_line":"        self.assertIn(\"bytes_in\", stats)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUX%2F%2FsvE%3D","line":214,"in_reply_to":"AAAAUX%2F%2FuY8%3D","updated":"2014-02-11 11:54:08.000000000","message":"Done","commit_id":"fbd5c5148f736fce8288a60615c5049fe2191dc1"},{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"54cf44f69052aa8e7127744b84ff945190f42d86","unresolved":false,"context_lines":[{"line_number":215,"context_line":"        resp, body \u003d self.client.list_lb_pool_stats(self.pool[\u0027id\u0027])"},{"line_number":216,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":217,"context_line":"        stats \u003d body[\u0027stats\u0027]"},{"line_number":218,"context_line":"        self.assertIsNotNone(stats)"},{"line_number":219,"context_line":"        self.assertIn(\"bytes_in\", stats)"},{"line_number":220,"context_line":"        self.assertIn(\"total_connections\", stats)"},{"line_number":221,"context_line":"        self.assertIn(\"active_connections\", stats)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUX%2F%2Fsyk%3D","line":218,"updated":"2014-02-11 11:48:34.000000000","message":"I dont think this check ^^^ is necessary, I mean, with the bellow checks you will do pretty much it, ie, assure that stats is not None by finding attributes like bytes_in on it","commit_id":"6a7754769346908f4d1a53d6f7321fd4b39b0b23"},{"author":{"_account_id":2238,"name":"rajalakshmi-ganesan","email":"rajalakshmi.girish@hpe.com","username":"rajalakshmi-ganesan"},"change_message_id":"dfcdf9e30d843fa50944f4e7e9fcdbbbbf5250f6","unresolved":false,"context_lines":[{"line_number":215,"context_line":"        resp, body \u003d self.client.list_lb_pool_stats(self.pool[\u0027id\u0027])"},{"line_number":216,"context_line":"        self.assertEqual(\u0027200\u0027, resp[\u0027status\u0027])"},{"line_number":217,"context_line":"        stats \u003d body[\u0027stats\u0027]"},{"line_number":218,"context_line":"        self.assertIsNotNone(stats)"},{"line_number":219,"context_line":"        self.assertIn(\"bytes_in\", stats)"},{"line_number":220,"context_line":"        self.assertIn(\"total_connections\", stats)"},{"line_number":221,"context_line":"        self.assertIn(\"active_connections\", stats)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUX%2F%2Fsvg%3D","line":218,"in_reply_to":"AAAAUX%2F%2Fsyk%3D","updated":"2014-02-11 11:53:15.000000000","message":"Done","commit_id":"6a7754769346908f4d1a53d6f7321fd4b39b0b23"}],"tempest/services/network/json/network_client.py":[{"author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"change_message_id":"13b583b64c9f5c73005d500747344b67b5ec2bae","unresolved":false,"context_lines":[{"line_number":418,"context_line":"        body \u003d json.loads(body)"},{"line_number":419,"context_line":"        return resp, body"},{"line_number":420,"context_line":""},{"line_number":421,"context_line":"    def list_lb_pool_stats(self, pool_id):"},{"line_number":422,"context_line":"        uri \u003d \u0027%s/lb/pools/%s/stats\u0027 % (self.uri_prefix, pool_id)"},{"line_number":423,"context_line":"        resp, body \u003d self.get(uri)"},{"line_number":424,"context_line":"        body \u003d json.loads(body)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F9Xq4%3D","line":421,"updated":"2014-02-07 11:49:40.000000000","message":"Is not it possible to use lister function from network_client_base.py to do list_lb_pool_stats?","commit_id":"fbd5c5148f736fce8288a60615c5049fe2191dc1"},{"author":{"_account_id":6455,"name":"Harika Vakadi","email":"harika_vakadi@persistent.co.in","username":"harika-vakadi"},"change_message_id":"01a9d297cae61737bd813368ef62165bd5509a2a","unresolved":false,"context_lines":[{"line_number":418,"context_line":"        body \u003d json.loads(body)"},{"line_number":419,"context_line":"        return resp, body"},{"line_number":420,"context_line":""},{"line_number":421,"context_line":"    def list_lb_pool_stats(self, pool_id):"},{"line_number":422,"context_line":"        uri \u003d \u0027%s/lb/pools/%s/stats\u0027 % (self.uri_prefix, pool_id)"},{"line_number":423,"context_line":"        resp, body \u003d self.get(uri)"},{"line_number":424,"context_line":"        body \u003d json.loads(body)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUX%2F%2F2eY%3D","line":421,"in_reply_to":"AAAAUH%2F9Xq4%3D","updated":"2014-02-10 13:33:24.000000000","message":"Hi Ann,\n\nIt is not possible to use lister function as lister is useful for filter url which will append (?) to the base url.\n\nThe other alternative is show function, but again this would accept resource name and resource id , where as in this case lb,pool,stat are to be appended in the URI.","commit_id":"fbd5c5148f736fce8288a60615c5049fe2191dc1"},{"author":{"_account_id":6455,"name":"Harika Vakadi","email":"harika_vakadi@persistent.co.in","username":"harika-vakadi"},"change_message_id":"5cd6d13e8b413d2dd6a58a1d849bae3ada6b6e9a","unresolved":false,"context_lines":[{"line_number":418,"context_line":"        body \u003d json.loads(body)"},{"line_number":419,"context_line":"        return resp, body"},{"line_number":420,"context_line":""},{"line_number":421,"context_line":"    def list_lb_pool_stats(self, pool_id):"},{"line_number":422,"context_line":"        uri \u003d \u0027%s/lb/pools/%s/stats\u0027 % (self.uri_prefix, pool_id)"},{"line_number":423,"context_line":"        resp, body \u003d self.get(uri)"},{"line_number":424,"context_line":"        body \u003d json.loads(body)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F9Vmc%3D","line":421,"in_reply_to":"AAAAUH%2F9Xq4%3D","updated":"2014-02-07 16:21:52.000000000","message":"okay if possible, will submit in next patch","commit_id":"fbd5c5148f736fce8288a60615c5049fe2191dc1"}]}