)]}'
{"tempest/services/network/json/network_client.py":[{"author":{"_account_id":12024,"name":"Tong Liu","email":"tongl@vmware.com","username":"tong101"},"change_message_id":"cf995fd6d23e6b6752d671c555ca61219ae0ac14","unresolved":false,"context_lines":[{"line_number":176,"context_line":"    def create_port(self, **kwargs):"},{"line_number":177,"context_line":"        uri \u003d \u0027/ports\u0027"},{"line_number":178,"context_line":"        post_data \u003d {\u0027port\u0027: kwargs}"},{"line_number":179,"context_line":"        return self._create_resource(uri, post_data)"},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"    def create_floatingip(self, **kwargs):"},{"line_number":182,"context_line":"        uri \u003d \u0027/floatingips\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa32b979_28acbd8a","line":179,"updated":"2015-06-18 22:42:04.000000000","message":"missing create_router?","commit_id":"68ee875328a5ec2ef64977fcab3152589242f13b"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"374cf3a81e57dddd44fcbd5597f431c2e2aaa716","unresolved":false,"context_lines":[{"line_number":176,"context_line":"    def create_port(self, **kwargs):"},{"line_number":177,"context_line":"        uri \u003d \u0027/ports\u0027"},{"line_number":178,"context_line":"        post_data \u003d {\u0027port\u0027: kwargs}"},{"line_number":179,"context_line":"        return self._create_resource(uri, post_data)"},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"    def create_floatingip(self, **kwargs):"},{"line_number":182,"context_line":"        uri \u003d \u0027/floatingips\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa32b979_c9d2f3d0","line":179,"in_reply_to":"fa32b979_28acbd8a","updated":"2015-06-19 00:49:22.000000000","message":"Thanks for review Liu. create_router is already present at L376. I can make its implementation same as others create but in another patch.","commit_id":"68ee875328a5ec2ef64977fcab3152589242f13b"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"79094b32b50b28ccfd8422c93fec39699ca92e13","unresolved":false,"context_lines":[{"line_number":183,"context_line":"        post_data \u003d {\u0027floatingip\u0027: kwargs}"},{"line_number":184,"context_line":"        return self._create_resource(uri, post_data)"},{"line_number":185,"context_line":""},{"line_number":186,"context_line":"    def create_pool(self, **kwargs):"},{"line_number":187,"context_line":"        uri \u003d \u0027/lb/pools\u0027"},{"line_number":188,"context_line":"        post_data \u003d {\u0027pool\u0027: kwargs}"},{"line_number":189,"context_line":"        return self._create_resource(uri, post_data)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa32b979_10eb54e0","line":186,"updated":"2015-06-18 02:03:24.000000000","message":"maybe here should be create_lb_pool ideally, but current patch is fine because of the scope.","commit_id":"68ee875328a5ec2ef64977fcab3152589242f13b"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"3052f37230d428b9727593951d9a27382b665ebb","unresolved":false,"context_lines":[{"line_number":183,"context_line":"        post_data \u003d {\u0027floatingip\u0027: kwargs}"},{"line_number":184,"context_line":"        return self._create_resource(uri, post_data)"},{"line_number":185,"context_line":""},{"line_number":186,"context_line":"    def create_pool(self, **kwargs):"},{"line_number":187,"context_line":"        uri \u003d \u0027/lb/pools\u0027"},{"line_number":188,"context_line":"        post_data \u003d {\u0027pool\u0027: kwargs}"},{"line_number":189,"context_line":"        return self._create_resource(uri, post_data)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa32b979_7000a066","line":186,"in_reply_to":"fa32b979_10eb54e0","updated":"2015-06-18 02:12:16.000000000","message":"Agree. We need to update those to appropriate naming as you mentioned.","commit_id":"68ee875328a5ec2ef64977fcab3152589242f13b"}]}