)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"cc956b3cbce66454553a51cefd73cad3eb34a765","unresolved":false,"context_lines":[{"line_number":15,"context_line":"test cases because of the foreign key enforcement."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I89f0cbbd75bb685b50dfe6628116fa971c5e78cb"},{"line_number":18,"context_line":"Fixes: bug #1021023"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAMX%2F%2F4EQ%3D","line":18,"updated":"2012-12-11 13:00:50.000000000","message":"Would it be possible to move this above the Change-id?","commit_id":"d1c652552213f00a3e4e4596bbf802335160a864"},{"author":{"_account_id":4184,"name":"Jason Zhang","email":"zhesen@nttmcl.com","username":"zzs"},"change_message_id":"307a239b492dd15017f9155b9dc9f6b5780e6f3c","unresolved":false,"context_lines":[{"line_number":15,"context_line":"test cases because of the foreign key enforcement."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I89f0cbbd75bb685b50dfe6628116fa971c5e78cb"},{"line_number":18,"context_line":"Fixes: bug #1021023"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAMX%2F%2F3jw%3D","line":18,"in_reply_to":"AAAAMX%2F%2F4EQ%3D","updated":"2012-12-11 19:25:40.000000000","message":"It looks like the gerrit automatically put the Change-Id above it.\nAnyway I will move it above Change-Id.","commit_id":"d1c652552213f00a3e4e4596bbf802335160a864"}],"quantum/db/api.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"cc956b3cbce66454553a51cefd73cad3eb34a765","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"import sqlalchemy as sql"},{"line_number":23,"context_line":"from sqlalchemy import create_engine"},{"line_number":24,"context_line":"from sqlalchemy.interfaces import PoolListener"},{"line_number":25,"context_line":"from sqlalchemy.exc import DisconnectionError"},{"line_number":26,"context_line":"from sqlalchemy.orm import sessionmaker, exc"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F4EI%3D","line":24,"updated":"2012-12-11 13:00:50.000000000","message":"I think that this should be below sqlalchemy.exc (please see HACKING rules)","commit_id":"d1c652552213f00a3e4e4596bbf802335160a864"},{"author":{"_account_id":4184,"name":"Jason Zhang","email":"zhesen@nttmcl.com","username":"zzs"},"change_message_id":"307a239b492dd15017f9155b9dc9f6b5780e6f3c","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"import sqlalchemy as sql"},{"line_number":23,"context_line":"from sqlalchemy import create_engine"},{"line_number":24,"context_line":"from sqlalchemy.interfaces import PoolListener"},{"line_number":25,"context_line":"from sqlalchemy.exc import DisconnectionError"},{"line_number":26,"context_line":"from sqlalchemy.orm import sessionmaker, exc"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F3hg%3D","line":24,"in_reply_to":"AAAAMX%2F%2F4EI%3D","updated":"2012-12-11 19:30:36.000000000","message":"Hi Gary, Thanks for pointing out it, I will fix them. Thanks again!","commit_id":"d1c652552213f00a3e4e4596bbf802335160a864"}],"quantum/tests/unit/linuxbridge/test_lb_db.py":[{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"unresolved":false,"context_lines":[{"line_number":150,"context_line":"        lb_db.initialize()"},{"line_number":151,"context_line":"        self.session \u003d db.get_session()"},{"line_number":152,"context_line":""},{"line_number":153,"context_line":"    def tearDown(self):"},{"line_number":154,"context_line":"        super(NetworkBindingsTest, self).tearDown()"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    def test_add_network_binding(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAMX%2F%2F07U%3D","line":153,"updated":"2012-12-13 19:14:01.000000000","message":"There\u0027s no reason to keep this method since it only calls the super version.","commit_id":"f5a5e4b1b118042a6083deb111b0a57accc529e9"},{"author":{"_account_id":4184,"name":"Jason Zhang","email":"zhesen@nttmcl.com","username":"zzs"},"change_message_id":"307a239b492dd15017f9155b9dc9f6b5780e6f3c","unresolved":false,"context_lines":[{"line_number":150,"context_line":"        lb_db.initialize()"},{"line_number":151,"context_line":"        self.session \u003d db.get_session()"},{"line_number":152,"context_line":""},{"line_number":153,"context_line":"    def tearDown(self):"},{"line_number":154,"context_line":"        super(NetworkBindingsTest, self).tearDown()"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    def test_add_network_binding(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAMX%2F%2F03Y%3D","line":153,"in_reply_to":"AAAAMX%2F%2F07U%3D","updated":"2012-12-13 20:23:05.000000000","message":"Hi Mark, thank you very much for pointing out, I will fix them.","commit_id":"f5a5e4b1b118042a6083deb111b0a57accc529e9"}],"quantum/tests/unit/metaplugin/test_metaplugin.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"cc956b3cbce66454553a51cefd73cad3eb34a765","unresolved":false,"context_lines":[{"line_number":34,"context_line":"from quantum.plugins.metaplugin.meta_quantum_plugin import MetaPluginV2"},{"line_number":35,"context_line":"from quantum.plugins.metaplugin.proxy_quantum_plugin import ProxyPluginV2"},{"line_number":36,"context_line":"from quantum.tests.unit.metaplugin import fake_plugin"},{"line_number":37,"context_line":"from quantum.plugins.metaplugin.meta_quantum_plugin import FlavorNotFound"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"CONF_FILE \u003d \"\""},{"line_number":40,"context_line":"ROOTDIR \u003d os.path.dirname(os.path.dirname(__file__))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F4EA%3D","line":37,"updated":"2012-12-11 13:00:50.000000000","message":"This should be above  line 34","commit_id":"d1c652552213f00a3e4e4596bbf802335160a864"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"unresolved":false,"context_lines":[{"line_number":295,"context_line":""},{"line_number":296,"context_line":"        self.plugin.delete_router(self.context, router_ret1[\u0027id\u0027])"},{"line_number":297,"context_line":"        self.plugin.delete_router(self.context, router_ret2[\u0027id\u0027])"},{"line_number":298,"context_line":"        with self.assertRaises(FlavorNotFound):"},{"line_number":299,"context_line":"            self.plugin.get_router(self.context, router_ret1[\u0027id\u0027])"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":"    def test_extension_method(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAMX%2F%2F07Q%3D","line":298,"updated":"2012-12-13 19:14:01.000000000","message":"Not sure why this is being changed here.  This is outside the scope of the bug that is being fixed.","commit_id":"f5a5e4b1b118042a6083deb111b0a57accc529e9"},{"author":{"_account_id":4184,"name":"Jason Zhang","email":"zhesen@nttmcl.com","username":"zzs"},"change_message_id":"307a239b492dd15017f9155b9dc9f6b5780e6f3c","unresolved":false,"context_lines":[{"line_number":295,"context_line":""},{"line_number":296,"context_line":"        self.plugin.delete_router(self.context, router_ret1[\u0027id\u0027])"},{"line_number":297,"context_line":"        self.plugin.delete_router(self.context, router_ret2[\u0027id\u0027])"},{"line_number":298,"context_line":"        with self.assertRaises(FlavorNotFound):"},{"line_number":299,"context_line":"            self.plugin.get_router(self.context, router_ret1[\u0027id\u0027])"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":"    def test_extension_method(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAMX%2F%2F03E%3D","line":298,"in_reply_to":"AAAAMX%2F%2F07Q%3D","updated":"2012-12-13 20:23:05.000000000","message":"As we enabled the foreign key constraint, the get_router method will get exception of FlavorNotFound during the run-time since the flavor had been deleted by the execution of delete_router method previously.","commit_id":"f5a5e4b1b118042a6083deb111b0a57accc529e9"}],"quantum/tests/unit/openvswitch/test_ovs_db.py":[{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"unresolved":false,"context_lines":[{"line_number":248,"context_line":"        ovs_db_v2.initialize()"},{"line_number":249,"context_line":"        self.session \u003d db.get_session()"},{"line_number":250,"context_line":""},{"line_number":251,"context_line":"    def tearDown(self):"},{"line_number":252,"context_line":"        super(NetworkBindingsTest, self).tearDown()"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"    def test_add_network_binding(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAMX%2F%2F07M%3D","line":251,"updated":"2012-12-13 19:14:01.000000000","message":"There\u0027s no reason to keep this method since it only calls the super version.","commit_id":"f5a5e4b1b118042a6083deb111b0a57accc529e9"}],"quantum/tests/unit/ryu/test_ryu_db.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"cc956b3cbce66454553a51cefd73cad3eb34a765","unresolved":false,"context_lines":[{"line_number":16,"context_line":"#    under the License."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import operator"},{"line_number":19,"context_line":"from contextlib import nested"},{"line_number":20,"context_line":"import unittest2"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from quantum.db import api as db"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F4D4%3D","line":19,"updated":"2012-12-11 13:00:50.000000000","message":"should be above line 18","commit_id":"d1c652552213f00a3e4e4596bbf802335160a864"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"unresolved":false,"context_lines":[{"line_number":38,"context_line":"                       ofp_service_type.REST_API)]"},{"line_number":39,"context_line":"        db_api_v2.set_ofp_servers(self.hosts)"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    def tearDown(self):"},{"line_number":42,"context_line":"        super(RyuDBTest, self).tearDown()"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    def test_ofp_server(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAMX%2F%2F07I%3D","line":41,"updated":"2012-12-13 19:14:01.000000000","message":"There\u0027s no reason to keep this method since it only calls the super version.","commit_id":"f5a5e4b1b118042a6083deb111b0a57accc529e9"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"                tunnel_key.delete(session, network_id1)"},{"line_number":81,"context_line":"                key_list \u003d tunnel_key.all_list()"},{"line_number":82,"context_line":"                self.assertEqual(self._tunnel_key_sort(key_list), [])"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAMX%2F%2F068%3D","line":82,"updated":"2012-12-13 19:14:01.000000000","message":"Why are we sorting a list that we expect to be empty?","commit_id":"f5a5e4b1b118042a6083deb111b0a57accc529e9"}]}