)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"558b9e8feb2a26cea33192d816aa15ea456920b9","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Cleanup of SecurityGroup classes"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Commit Ifb70a118cef48c3c4cd313e22e907aa47bc51ad0 intended to remove some"},{"line_number":10,"context_line":"classes, but without DeprecationWarnings. To prevent from possible"},{"line_number":11,"context_line":"problems, if it would be now removed, added Warnings and included info,"},{"line_number":12,"context_line":"about future removal."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa7ab95a_c28b621d","line":9,"updated":"2016-08-30 22:56:30.000000000","message":"have we checked if we have out of tree users of these classes? Just wondering...","commit_id":"bb0de98ea5189eddbf6403a69663c2d846e91f0f"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"842da250f87af54d4ae174418f00eea33967ba88","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Cleanup of SecurityGroup classes"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Commit Ifb70a118cef48c3c4cd313e22e907aa47bc51ad0 intended to remove some"},{"line_number":10,"context_line":"classes, but without DeprecationWarnings. To prevent from possible"},{"line_number":11,"context_line":"problems, if it would be now removed, added Warnings and included info,"},{"line_number":12,"context_line":"about future removal."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa7ab95a_1d25a45b","line":9,"in_reply_to":"fa7ab95a_c28b621d","updated":"2016-08-31 05:48:36.000000000","message":"I\u0027ve found couple repos with class `SecurityGroupServerRpcApi` [1]:\nnetworking-vsphere, networking-hyperv, networking-bigswitch, networking-mlnx and networking-ofagent\n\n[1] http://codesearch.openstack.org/?q\u003dSecurityGroupServerRpcApi\u0026i\u003dnope\u0026files\u003d\u0026repos\u003d","commit_id":"bb0de98ea5189eddbf6403a69663c2d846e91f0f"}],"neutron/agent/securitygroups_rpc.py":[{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"e2e432045da3ca0dedc207439edd6ff19e760c76","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":"#"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from debtcollector import moves"},{"line_number":18,"context_line":"import functools"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from oslo_config import cfg"}],"source_content_type":"text/x-python","patch_set":6,"id":"3a98d1f4_415642d3","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":31},"updated":"2016-09-29 08:34:23.000000000","message":"The debtcollector is third-party lib so please move below[1].\n\n[1]: http://docs.openstack.org/developer/hacking/#import-order-template","commit_id":"42951a44f6103554cb0437565f04299f9763ba62"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"4ea585bcb8c0c98922c78d6bb1a16dce55f09cc3","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":"#"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from debtcollector import moves"},{"line_number":18,"context_line":"import functools"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from oslo_config import cfg"}],"source_content_type":"text/x-python","patch_set":6,"id":"3a98d1f4_c35eb3e8","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":31},"in_reply_to":"3a98d1f4_415642d3","updated":"2016-09-29 18:09:33.000000000","message":"Done","commit_id":"42951a44f6103554cb0437565f04299f9763ba62"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"e2e432045da3ca0dedc207439edd6ff19e760c76","unresolved":false,"context_lines":[{"line_number":288,"context_line":"# NOTE(dasm): Should be already removed, but didn\u0027t have  DeprecationWarning."},{"line_number":289,"context_line":"SG_RPC_VERSION \u003d moves.moved_function("},{"line_number":290,"context_line":"    securitygroups_rpc.SecurityGroupAgentRpcApiMixin.SG_RPC_VERSION,"},{"line_number":291,"context_line":"    \u0027SG_RPC_VERSION\u0027, __name__, version\u003d\u0027Liberty\u0027, removal_version\u003d\u0027Ocata\u0027"},{"line_number":292,"context_line":")"},{"line_number":293,"context_line":"SecurityGroupServerRpcApi \u003d moves.moved_class("},{"line_number":294,"context_line":"    securitygroups_rpc.SecurityGroupServerRpcApi,"}],"source_content_type":"text/x-python","patch_set":6,"id":"3a98d1f4_61609ecf","line":291,"range":{"start_line":291,"start_character":67,"end_line":291,"end_character":74},"updated":"2016-09-29 08:34:23.000000000","message":"Ocata already has opened so we should set \u0027Pike\u0027 to removal_version.","commit_id":"42951a44f6103554cb0437565f04299f9763ba62"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"4ea585bcb8c0c98922c78d6bb1a16dce55f09cc3","unresolved":false,"context_lines":[{"line_number":288,"context_line":"# NOTE(dasm): Should be already removed, but didn\u0027t have  DeprecationWarning."},{"line_number":289,"context_line":"SG_RPC_VERSION \u003d moves.moved_function("},{"line_number":290,"context_line":"    securitygroups_rpc.SecurityGroupAgentRpcApiMixin.SG_RPC_VERSION,"},{"line_number":291,"context_line":"    \u0027SG_RPC_VERSION\u0027, __name__, version\u003d\u0027Liberty\u0027, removal_version\u003d\u0027Ocata\u0027"},{"line_number":292,"context_line":")"},{"line_number":293,"context_line":"SecurityGroupServerRpcApi \u003d moves.moved_class("},{"line_number":294,"context_line":"    securitygroups_rpc.SecurityGroupServerRpcApi,"}],"source_content_type":"text/x-python","patch_set":6,"id":"3a98d1f4_235cb7ec","line":291,"range":{"start_line":291,"start_character":67,"end_line":291,"end_character":74},"in_reply_to":"3a98d1f4_61609ecf","updated":"2016-09-29 18:09:33.000000000","message":"Yep, you\u0027re right. It was waiting in queue for some time.\nThanks","commit_id":"42951a44f6103554cb0437565f04299f9763ba62"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"c98f1e5b46402c1f6809ca4cf1f44f57dde10e9e","unresolved":false,"context_lines":[{"line_number":284,"context_line":"                self.refresh_firewall(updated_devices)"},{"line_number":285,"context_line":""},{"line_number":286,"context_line":""},{"line_number":287,"context_line":"# TODO(armax): For bw compat with external dependencies; to be dropped in M."},{"line_number":288,"context_line":"# NOTE(dasm): Should be already removed, but didn\u0027t have  DeprecationWarning."},{"line_number":289,"context_line":"SG_RPC_VERSION \u003d moves.moved_function("},{"line_number":290,"context_line":"    securitygroups_rpc.SecurityGroupAgentRpcApiMixin.SG_RPC_VERSION,"}],"source_content_type":"text/x-python","patch_set":8,"id":"fa6399be_a6160453","line":287,"range":{"start_line":287,"start_character":57,"end_line":287,"end_character":75},"updated":"2016-10-13 04:20:06.000000000","message":"needs an update?","commit_id":"f1ad0795f4bdb47da459acea0ae88bce2159a577"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"18a40f7ed78967440e6248b42fb5b4ffa0e2db0f","unresolved":false,"context_lines":[{"line_number":284,"context_line":"                self.refresh_firewall(updated_devices)"},{"line_number":285,"context_line":""},{"line_number":286,"context_line":""},{"line_number":287,"context_line":"# TODO(armax): For bw compat with external dependencies; to be dropped in M."},{"line_number":288,"context_line":"# NOTE(dasm): Should be already removed, but didn\u0027t have  DeprecationWarning."},{"line_number":289,"context_line":"SG_RPC_VERSION \u003d moves.moved_function("},{"line_number":290,"context_line":"    securitygroups_rpc.SecurityGroupAgentRpcApiMixin.SG_RPC_VERSION,"}],"source_content_type":"text/x-python","patch_set":8,"id":"fa6399be_f24a5a25","line":287,"range":{"start_line":287,"start_character":57,"end_line":287,"end_character":75},"in_reply_to":"fa6399be_a6160453","updated":"2016-10-13 15:37:49.000000000","message":"Done","commit_id":"f1ad0795f4bdb47da459acea0ae88bce2159a577"},{"author":{"_account_id":8726,"name":"Victor Morales","email":"chipahuac@hotmail.com","username":"electrocucaracha"},"change_message_id":"7abfd2a23f5caa9e7b22c4c61b6943126bcfe415","unresolved":false,"context_lines":[{"line_number":285,"context_line":""},{"line_number":286,"context_line":""},{"line_number":287,"context_line":"# TODO(armax): For bw compat with external dependencies; to be dropped in P."},{"line_number":288,"context_line":"# NOTE(dasm): Should be already removed, but didn\u0027t have  DeprecationWarning."},{"line_number":289,"context_line":"SG_RPC_VERSION \u003d moves.moved_function("},{"line_number":290,"context_line":"    securitygroups_rpc.SecurityGroupAgentRpcApiMixin.SG_RPC_VERSION,"},{"line_number":291,"context_line":"    \u0027SG_RPC_VERSION\u0027, __name__, version\u003d\u0027Liberty\u0027, removal_version\u003d\u0027Pike\u0027"}],"source_content_type":"text/x-python","patch_set":10,"id":"fa6399be_d08847c9","line":288,"range":{"start_line":288,"start_character":57,"end_line":288,"end_character":58},"updated":"2016-10-18 19:28:48.000000000","message":"meh","commit_id":"97338258967d3b95f382f188ab2ab573a3432c17"}],"neutron/plugins/ml2/drivers/agent/_common_agent.py":[{"author":{"_account_id":22220,"name":"Anindita Das","email":"anindita.das@utexas.edu","username":"anudas"},"change_message_id":"23d64fe408f3cdca410ed0b558882c4d0d479cb6","unresolved":false,"context_lines":[{"line_number":149,"context_line":"    def setup_rpc(self):"},{"line_number":150,"context_line":"        self.plugin_rpc \u003d agent_rpc.PluginApi(topics.PLUGIN)"},{"line_number":151,"context_line":"        self.sg_plugin_rpc \u003d sg_rpc.SecurityGroupServerRpcApi(topics.PLUGIN)"},{"line_number":152,"context_line":"        self.sg_agent \u003d securitygroups_rpc.SecurityGroupAgentRpc("},{"line_number":153,"context_line":"            self.context, self.sg_plugin_rpc, defer_refresh_firewall\u003dTrue)"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"        self.agent_id \u003d self.mgr.get_agent_id()"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_b169988e","line":152,"range":{"start_line":152,"start_character":24,"end_line":152,"end_character":42},"updated":"2016-09-02 14:52:11.000000000","message":"The import section has \"from neutron.api.rpc.handlers import securitygroups_rpc as sg_rpc\"\n\nsecuritygroups_rpc needs to be replaced by sg_rpc or \"as sg_rpc\"  to be removed from the import","commit_id":"07962b23077fbbec41b9c420dd82563fc99c3186"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"edc934c88dfcf612054c355c0779e7fa73a23eca","unresolved":false,"context_lines":[{"line_number":149,"context_line":"    def setup_rpc(self):"},{"line_number":150,"context_line":"        self.plugin_rpc \u003d agent_rpc.PluginApi(topics.PLUGIN)"},{"line_number":151,"context_line":"        self.sg_plugin_rpc \u003d sg_rpc.SecurityGroupServerRpcApi(topics.PLUGIN)"},{"line_number":152,"context_line":"        self.sg_agent \u003d securitygroups_rpc.SecurityGroupAgentRpc("},{"line_number":153,"context_line":"            self.context, self.sg_plugin_rpc, defer_refresh_firewall\u003dTrue)"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"        self.agent_id \u003d self.mgr.get_agent_id()"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a89bdaa_9b71a7e3","line":152,"range":{"start_line":152,"start_character":24,"end_line":152,"end_character":42},"in_reply_to":"da7fb54a_615536c5","updated":"2016-09-09 10:36:23.000000000","message":"Renamed as `agent_sg_rpc` because `agent_rpc` is already taken.","commit_id":"07962b23077fbbec41b9c420dd82563fc99c3186"},{"author":{"_account_id":22220,"name":"Anindita Das","email":"anindita.das@utexas.edu","username":"anudas"},"change_message_id":"e62012a2e5a92cc8e954393c3f0d646fea9982e6","unresolved":false,"context_lines":[{"line_number":149,"context_line":"    def setup_rpc(self):"},{"line_number":150,"context_line":"        self.plugin_rpc \u003d agent_rpc.PluginApi(topics.PLUGIN)"},{"line_number":151,"context_line":"        self.sg_plugin_rpc \u003d sg_rpc.SecurityGroupServerRpcApi(topics.PLUGIN)"},{"line_number":152,"context_line":"        self.sg_agent \u003d securitygroups_rpc.SecurityGroupAgentRpc("},{"line_number":153,"context_line":"            self.context, self.sg_plugin_rpc, defer_refresh_firewall\u003dTrue)"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"        self.agent_id \u003d self.mgr.get_agent_id()"}],"source_content_type":"text/x-python","patch_set":2,"id":"da7fb54a_615536c5","line":152,"range":{"start_line":152,"start_character":24,"end_line":152,"end_character":42},"in_reply_to":"fa7ab95a_a355cb1e","updated":"2016-09-02 15:24:28.000000000","message":"Got it. To improve readability can we add \"as agent_rpc\" in this import from neutron.agent import securitygroups_rpc and use it.","commit_id":"07962b23077fbbec41b9c420dd82563fc99c3186"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"3fe7ad2109920714091a22748f40066e4b35cfb4","unresolved":false,"context_lines":[{"line_number":149,"context_line":"    def setup_rpc(self):"},{"line_number":150,"context_line":"        self.plugin_rpc \u003d agent_rpc.PluginApi(topics.PLUGIN)"},{"line_number":151,"context_line":"        self.sg_plugin_rpc \u003d sg_rpc.SecurityGroupServerRpcApi(topics.PLUGIN)"},{"line_number":152,"context_line":"        self.sg_agent \u003d securitygroups_rpc.SecurityGroupAgentRpc("},{"line_number":153,"context_line":"            self.context, self.sg_plugin_rpc, defer_refresh_firewall\u003dTrue)"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"        self.agent_id \u003d self.mgr.get_agent_id()"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_a355cb1e","line":152,"range":{"start_line":152,"start_character":24,"end_line":152,"end_character":42},"in_reply_to":"fa7ab95a_b169988e","updated":"2016-09-02 15:18:16.000000000","message":"It cannot be done because ServerRpcApi is located in api.rpc.handlers, and AgentRpc is in agent.securitygroups_rpc","commit_id":"07962b23077fbbec41b9c420dd82563fc99c3186"}],"neutron/plugins/ml2/drivers/mech_sriov/agent/sriov_nic_agent.py":[{"author":{"_account_id":22220,"name":"Anindita Das","email":"anindita.das@utexas.edu","username":"anudas"},"change_message_id":"23d64fe408f3cdca410ed0b558882c4d0d479cb6","unresolved":false,"context_lines":[{"line_number":123,"context_line":"        self.context \u003d context.get_admin_context_without_session()"},{"line_number":124,"context_line":"        self.plugin_rpc \u003d agent_rpc.PluginApi(topics.PLUGIN)"},{"line_number":125,"context_line":"        self.sg_plugin_rpc \u003d sg_rpc.SecurityGroupServerRpcApi(topics.PLUGIN)"},{"line_number":126,"context_line":"        self.sg_agent \u003d securitygroups_rpc.SecurityGroupAgentRpc(self.context,"},{"line_number":127,"context_line":"                self.sg_plugin_rpc)"},{"line_number":128,"context_line":"        self._setup_rpc()"},{"line_number":129,"context_line":"        self.ext_manager \u003d self._create_agent_extension_manager("}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_f1a160aa","line":126,"updated":"2016-09-02 14:52:11.000000000","message":"ditto","commit_id":"07962b23077fbbec41b9c420dd82563fc99c3186"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"3fe7ad2109920714091a22748f40066e4b35cfb4","unresolved":false,"context_lines":[{"line_number":123,"context_line":"        self.context \u003d context.get_admin_context_without_session()"},{"line_number":124,"context_line":"        self.plugin_rpc \u003d agent_rpc.PluginApi(topics.PLUGIN)"},{"line_number":125,"context_line":"        self.sg_plugin_rpc \u003d sg_rpc.SecurityGroupServerRpcApi(topics.PLUGIN)"},{"line_number":126,"context_line":"        self.sg_agent \u003d securitygroups_rpc.SecurityGroupAgentRpc(self.context,"},{"line_number":127,"context_line":"                self.sg_plugin_rpc)"},{"line_number":128,"context_line":"        self._setup_rpc()"},{"line_number":129,"context_line":"        self.ext_manager \u003d self._create_agent_extension_manager("}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_632663cf","line":126,"in_reply_to":"fa7ab95a_f1a160aa","updated":"2016-09-02 15:18:16.000000000","message":"Please look at other response.","commit_id":"07962b23077fbbec41b9c420dd82563fc99c3186"},{"author":{"_account_id":22220,"name":"Anindita Das","email":"anindita.das@utexas.edu","username":"anudas"},"change_message_id":"6e98481440c71a4bc011727e85c48ec644ab9e5e","unresolved":false,"context_lines":[{"line_number":123,"context_line":"        self.context \u003d context.get_admin_context_without_session()"},{"line_number":124,"context_line":"        self.plugin_rpc \u003d agent_rpc.PluginApi(topics.PLUGIN)"},{"line_number":125,"context_line":"        self.sg_plugin_rpc \u003d sg_rpc.SecurityGroupServerRpcApi(topics.PLUGIN)"},{"line_number":126,"context_line":"        self.sg_agent \u003d securitygroups_rpc.SecurityGroupAgentRpc(self.context,"},{"line_number":127,"context_line":"                self.sg_plugin_rpc)"},{"line_number":128,"context_line":"        self._setup_rpc()"},{"line_number":129,"context_line":"        self.ext_manager \u003d self._create_agent_extension_manager("}],"source_content_type":"text/x-python","patch_set":4,"id":"9a89bdaa_57689521","line":126,"range":{"start_line":126,"start_character":24,"end_line":126,"end_character":42},"updated":"2016-09-12 03:02:56.000000000","message":"nit: can we do the same here import securitygroups_rpc as agent_sg_rpc","commit_id":"c8272e8b342da9efb3b349c5481e1692629e0189"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"37c4001c0f432e4879a37ad34c87bbe810ebb4d8","unresolved":false,"context_lines":[{"line_number":123,"context_line":"        self.context \u003d context.get_admin_context_without_session()"},{"line_number":124,"context_line":"        self.plugin_rpc \u003d agent_rpc.PluginApi(topics.PLUGIN)"},{"line_number":125,"context_line":"        self.sg_plugin_rpc \u003d sg_rpc.SecurityGroupServerRpcApi(topics.PLUGIN)"},{"line_number":126,"context_line":"        self.sg_agent \u003d securitygroups_rpc.SecurityGroupAgentRpc(self.context,"},{"line_number":127,"context_line":"                self.sg_plugin_rpc)"},{"line_number":128,"context_line":"        self._setup_rpc()"},{"line_number":129,"context_line":"        self.ext_manager \u003d self._create_agent_extension_manager("}],"source_content_type":"text/x-python","patch_set":4,"id":"9a89bdaa_a75ef742","line":126,"range":{"start_line":126,"start_character":24,"end_line":126,"end_character":42},"in_reply_to":"9a89bdaa_57689521","updated":"2016-09-12 13:28:36.000000000","message":"Done","commit_id":"c8272e8b342da9efb3b349c5481e1692629e0189"}],"neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py":[{"author":{"_account_id":22220,"name":"Anindita Das","email":"anindita.das@utexas.edu","username":"anudas"},"change_message_id":"23d64fe408f3cdca410ed0b558882c4d0d479cb6","unresolved":false,"context_lines":[{"line_number":236,"context_line":"        self._restore_local_vlan_map()"},{"line_number":237,"context_line":""},{"line_number":238,"context_line":"        # Security group agent support"},{"line_number":239,"context_line":"        self.sg_agent \u003d securitygroups_rpc.SecurityGroupAgentRpc("},{"line_number":240,"context_line":"            self.context, self.sg_plugin_rpc, defer_refresh_firewall\u003dTrue,"},{"line_number":241,"context_line":"            integration_bridge\u003dself.int_br)"},{"line_number":242,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_3196683e","line":239,"updated":"2016-09-02 14:52:11.000000000","message":"ditto","commit_id":"07962b23077fbbec41b9c420dd82563fc99c3186"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"3fe7ad2109920714091a22748f40066e4b35cfb4","unresolved":false,"context_lines":[{"line_number":236,"context_line":"        self._restore_local_vlan_map()"},{"line_number":237,"context_line":""},{"line_number":238,"context_line":"        # Security group agent support"},{"line_number":239,"context_line":"        self.sg_agent \u003d securitygroups_rpc.SecurityGroupAgentRpc("},{"line_number":240,"context_line":"            self.context, self.sg_plugin_rpc, defer_refresh_firewall\u003dTrue,"},{"line_number":241,"context_line":"            integration_bridge\u003dself.int_br)"},{"line_number":242,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_230ffb39","line":239,"in_reply_to":"fa7ab95a_3196683e","updated":"2016-09-02 15:18:16.000000000","message":"Please look at other response.","commit_id":"07962b23077fbbec41b9c420dd82563fc99c3186"},{"author":{"_account_id":22220,"name":"Anindita Das","email":"anindita.das@utexas.edu","username":"anudas"},"change_message_id":"6e98481440c71a4bc011727e85c48ec644ab9e5e","unresolved":false,"context_lines":[{"line_number":238,"context_line":"        self._restore_local_vlan_map()"},{"line_number":239,"context_line":""},{"line_number":240,"context_line":"        # Security group agent support"},{"line_number":241,"context_line":"        self.sg_agent \u003d securitygroups_rpc.SecurityGroupAgentRpc("},{"line_number":242,"context_line":"            self.context, self.sg_plugin_rpc, defer_refresh_firewall\u003dTrue,"},{"line_number":243,"context_line":"            integration_bridge\u003dself.int_br)"},{"line_number":244,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9a89bdaa_b75f2174","line":241,"range":{"start_line":241,"start_character":24,"end_line":241,"end_character":42},"updated":"2016-09-12 03:02:56.000000000","message":"ditto","commit_id":"c8272e8b342da9efb3b349c5481e1692629e0189"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"37c4001c0f432e4879a37ad34c87bbe810ebb4d8","unresolved":false,"context_lines":[{"line_number":238,"context_line":"        self._restore_local_vlan_map()"},{"line_number":239,"context_line":""},{"line_number":240,"context_line":"        # Security group agent support"},{"line_number":241,"context_line":"        self.sg_agent \u003d securitygroups_rpc.SecurityGroupAgentRpc("},{"line_number":242,"context_line":"            self.context, self.sg_plugin_rpc, defer_refresh_firewall\u003dTrue,"},{"line_number":243,"context_line":"            integration_bridge\u003dself.int_br)"},{"line_number":244,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9a89bdaa_67122f64","line":241,"range":{"start_line":241,"start_character":24,"end_line":241,"end_character":42},"in_reply_to":"9a89bdaa_b75f2174","updated":"2016-09-12 13:28:36.000000000","message":"Done","commit_id":"c8272e8b342da9efb3b349c5481e1692629e0189"}]}