)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"3aaad58e4cf040aca04effd642afc5a63b174b74","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Micheal Thompson \u003cMThompson@a10networks.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-12-16 20:38:11 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"\tAdded A10 LBaaS Driver to branch."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Implements: blueprint a10networks-lbaas-driver"},{"line_number":10,"context_line":"new file:   a10networks/__init__.py"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAATn%2F9D1U%3D","line":7,"updated":"2013-12-17 07:40:16.000000000","message":"Commit message should not end with a period.","commit_id":"61c790fa5bfad7cfbc1ce7bcb2b44dd01b81875e"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"3aaad58e4cf040aca04effd642afc5a63b174b74","unresolved":false,"context_lines":[{"line_number":7,"context_line":"\tAdded A10 LBaaS Driver to branch."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Implements: blueprint a10networks-lbaas-driver"},{"line_number":10,"context_line":"new file:   a10networks/__init__.py"},{"line_number":11,"context_line":"new file:   a10networks/a10_api_2_1/__init__.py"},{"line_number":12,"context_line":"new file:   a10networks/a10_api_2_1/core/__init__.py"},{"line_number":13,"context_line":"new file:   a10networks/a10_api_2_1/core/a10logger.py"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAATn%2F9Dzo%3D","line":10,"updated":"2013-12-17 07:40:16.000000000","message":"Could you please remove the list of files and briefly describe what/how is implemented in the patch:\nplugin driver, agent(?), how do they interact, etc.\nFor 3k lines of code I\u0027d expect 20-30 lines of description in the commit message.\nAlso specify what parts remain (I saw your follow up commit).\n\nIs this a consistent commit? I mean, will it work withould follow up?","commit_id":"61c790fa5bfad7cfbc1ce7bcb2b44dd01b81875e"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"b2d4089ab0fe3c8b6347be6c23381b66ac109ece","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     mthompson \u003cmthompson@a10networks.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-12-17 21:49:43 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixed PEP8 issues hopefully."},{"line_number":8,"context_line":"Corrected Cookie issue"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Implements: blueprint a10networks-lbaas-driver"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"AAAATn%2F8%2Bog%3D","line":7,"updated":"2013-12-18 06:23:12.000000000","message":"Please follow these guidelines for commit messages: https://wiki.openstack.org/wiki/GitCommitMessages\n\nYou don\u0027t need to specify in the commit message what you\u0027ve done in particular change. You can leave it as a general comment for a patchset. So I\u0027d still expect to see a brief description of how the driver is implemented and what are major parts of it.\nAlso, you can verify pep8 check locally by running:\n tox -epep8","commit_id":"59cdf94b09a2bb3253e92e2ad6d6f54932728a44"}],"neutron/services/loadbalancer/drivers/a10networks/a10_api_2_1/core/a10logger.py":[{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"3aaad58e4cf040aca04effd642afc5a63b174b74","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# vim: tabstop\u003d4 shiftwidth\u003d4 softtabstop\u003d4"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Copyright 2013,  Mike Thompson,  A10 Networks."},{"line_number":4,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F9Dxo%3D","line":1,"updated":"2013-12-17 07:40:16.000000000","message":"Isn\u0027t standard logger sufficient?","commit_id":"61c790fa5bfad7cfbc1ce7bcb2b44dd01b81875e"}],"neutron/services/loadbalancer/drivers/a10networks/device_conf.ini":[{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"3aaad58e4cf040aca04effd642afc5a63b174b74","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[devices]"},{"line_number":2,"context_line":"#format is device FQDN then host object"},{"line_number":3,"context_line":"#ax1.foo.com \u003d {\"host\":\"192.168.212.120\","},{"line_number":4,"context_line":"#                \"username\" : \"admin\","}],"source_content_type":"text/x-properties","patch_set":2,"id":"AAAATn%2F9ECs%3D","line":1,"updated":"2013-12-17 07:40:16.000000000","message":"i think this should go to etc/services.conf.\n Also, it make sense to name sections with more specific names like [a10_devices] etc, since they are placed in a common file.","commit_id":"61c790fa5bfad7cfbc1ce7bcb2b44dd01b81875e"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"3aaad58e4cf040aca04effd642afc5a63b174b74","unresolved":false,"context_lines":[{"line_number":6,"context_line":"#                 \"status\":\"1\","},{"line_number":7,"context_line":"#                 \"autosnat\":True}"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"ax1.foo.com \u003d {\"host\":\"192.168.212.120\","},{"line_number":10,"context_line":"              \"username\":\"admin\","},{"line_number":11,"context_line":"              \"password\":\"a10\", \"status\":\"1\", \"autosnat\":\"True\"}"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-properties","patch_set":2,"id":"AAAATn%2F9ECk%3D","line":9,"updated":"2013-12-17 07:40:16.000000000","message":"it\u0027s better to set defaults in configuration definition in the code.\nHowever this particular defaults doesn\u0027t seem like it should to work everywhere, or does it?","commit_id":"61c790fa5bfad7cfbc1ce7bcb2b44dd01b81875e"}],"neutron/services/loadbalancer/drivers/a10networks/thunder.py":[{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"3aaad58e4cf040aca04effd642afc5a63b174b74","unresolved":false,"context_lines":[{"line_number":34,"context_line":""},{"line_number":35,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"class thunder_driver(plugin.LoadBalancerPlugin):"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def __init__(self, plugin):"},{"line_number":40,"context_line":"        self.plugin \u003d plugin"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F9DxA%3D","line":37,"updated":"2013-12-17 07:40:16.000000000","message":"camel case for class names","commit_id":"61c790fa5bfad7cfbc1ce7bcb2b44dd01b81875e"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"3aaad58e4cf040aca04effd642afc5a63b174b74","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        self.plugin \u003d plugin"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def set_device_context(self,tenant_id):"},{"line_number":44,"context_line":"        cfg_file \u003d os.path.dirname(__file__) + \u0027/device_conf.ini\u0027"},{"line_number":45,"context_line":"        try:"},{"line_number":46,"context_line":"            self.config \u003d ConfigParser.ConfigParser()"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F9Dw4%3D","line":43,"updated":"2013-12-17 07:40:16.000000000","message":"isn\u0027t it better to utilize oslo.cfg in a usual way?","commit_id":"61c790fa5bfad7cfbc1ce7bcb2b44dd01b81875e"}]}