)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":490,"name":"Sumit Naiksatam","email":"sumitnaiksatam@gmail.com","username":"snaiksat"},"unresolved":false,"context_lines":[{"line_number":14,"context_line":"Fixing pep8 and dependency issues"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Implements: blueprint cisco-plugin-enhancements"},{"line_number":17,"context_line":"            (https://blueprints.launchpad.net/quantum/+spec/cisco-plugin-enhancements)"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I6275eb1815310d0d5a8123ca2edbc0a0937718e9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAM3%2F%2F5Hw%3D","line":17,"updated":"2012-12-24 16:13:59.000000000","message":"Not sure if you added the URL indicating the blueprint. If so, it\u0027s not required since the system adds a hyperlink automatically (unless something has changed in this behavior).","commit_id":"b8e4fe0c7be466fb7b1fc0469aab43dc44ed8623"},{"author":{"_account_id":490,"name":"Sumit Naiksatam","email":"sumitnaiksatam@gmail.com","username":"snaiksat"},"unresolved":false,"context_lines":[{"line_number":12,"context_line":"switches when no longer required."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Implements: blueprint cisco-plugin-enhancements"},{"line_number":15,"context_line":"            (https://blueprints.launchpad.net/quantum/+spec/cisco-plugin-enhancements)"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I6275eb1815310d0d5a8123ca2edbc0a0937718e9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"AAAAM3%2F%2Foco%3D","line":15,"updated":"2013-01-11 21:28:29.000000000","message":"The URL to blueprint is not required here.","commit_id":"b2034beb04176c70bbf33f8048eeec4aff77d8e2"},{"author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Implements: blueprint cisco-plugin-enhancements"},{"line_number":15,"context_line":"            (https://blueprints.launchpad.net/quantum/+spec/cisco-plugin-enhancements)"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I6275eb1815310d0d5a8123ca2edbc0a0937718e9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"AAAAM3%2F%2FoZg%3D","line":16,"updated":"2013-01-11 21:57:00.000000000","message":"The commit message should follow Gerrit workflow:\nhttp://wiki.openstack.org/GerritWorkflow","commit_id":"b2034beb04176c70bbf33f8048eeec4aff77d8e2"}],"etc/quantum/plugins/cisco/cisco_plugins.ini":[{"author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"unresolved":false,"context_lines":[{"line_number":2,"context_line":"#ucs_plugin\u003dquantum.plugins.cisco.ucs.cisco_ucs_plugin_v2.UCSVICPlugin"},{"line_number":3,"context_line":"nexus_plugin\u003dquantum.plugins.cisco.nexus.cisco_nexus_plugin_v2.NexusPlugin"},{"line_number":4,"context_line":"vswitch_plugin\u003dquantum.plugins.openvswitch.ovs_quantum_plugin.OVSQuantumPluginV2"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"[INVENTORY]"},{"line_number":7,"context_line":"#ucs_plugin\u003dquantum.plugins.cisco.ucs.cisco_ucs_inventory_v2.UCSInventory"},{"line_number":8,"context_line":"#ucs_plugin\u003dquantum.plugins.cisco.tests.unit.v2.ucs.cisco_ucs_inventory_fake.UCSInventory"}],"source_content_type":"text/x-properties","patch_set":5,"id":"AAAAM3%2F%2FoZE%3D","line":5,"updated":"2013-01-11 21:57:00.000000000","message":"Uncommenting this line will assume that Nexus-based switches will be always available, is this what you want?","commit_id":"b2034beb04176c70bbf33f8048eeec4aff77d8e2"}],"etc/quantum/plugins/cisco/credentials.ini":[{"author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"unresolved":false,"context_lines":[{"line_number":7,"context_line":"[1.1.1.1]"},{"line_number":8,"context_line":"username\u003duser"},{"line_number":9,"context_line":"password\u003dpass"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"# Provide credentials and endpoint"},{"line_number":12,"context_line":"# for keystone here"},{"line_number":13,"context_line":"[keystone]"}],"source_content_type":"text/x-properties","patch_set":5,"id":"AAAAM3%2F%2FoY4%3D","line":10,"updated":"2013-01-11 21:57:00.000000000","message":"Using default values is not recommended, take a look above","commit_id":"b2034beb04176c70bbf33f8048eeec4aff77d8e2"}],"etc/quantum/plugins/cisco/db_conn.ini":[{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"2d8b77235576b1de68ae8a362eba25ee2ca1ebee","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[DATABASE]"},{"line_number":2,"context_line":"name \u003d ovs_quantum"},{"line_number":3,"context_line":"user \u003d root"},{"line_number":4,"context_line":"pass \u003d nova"},{"line_number":5,"context_line":"host \u003d 127.0.0.1"}],"source_content_type":"text/x-properties","patch_set":3,"id":"AAAAM3%2F%2F5FY%3D","line":3,"updated":"2012-12-24 17:28:58.000000000","message":"Why the change here?  Seems like you would want either the obvious placeholders from the prior revision or comments indicating to change these values.","commit_id":"b8e4fe0c7be466fb7b1fc0469aab43dc44ed8623"}],"etc/quantum/plugins/cisco/l2network_plugin.ini":[{"author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"# IMPORTANT: Comment the following lines for production deployments"},{"line_number":23,"context_line":"[TEST]"},{"line_number":24,"context_line":"host\u003dtesthost"}],"source_content_type":"text/x-properties","patch_set":5,"id":"AAAAM3%2F%2FoYw%3D","line":24,"updated":"2013-01-11 21:57:00.000000000","message":"Should be in the other way around, this line could be uncommented for testing","commit_id":"b2034beb04176c70bbf33f8048eeec4aff77d8e2"}],"etc/quantum/plugins/cisco/nexus.ini":[{"author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"unresolved":false,"context_lines":[{"line_number":1,"context_line":"[SWITCH]"},{"line_number":2,"context_line":"# Ip address of the switch"},{"line_number":3,"context_line":"[[1.1.1.1]]"},{"line_number":4,"context_line":"# Hostname of the node"},{"line_number":5,"context_line":"[[[testhost]]]"},{"line_number":6,"context_line":"# Port this node is connected to on the nexus switch"},{"line_number":7,"context_line":"ports\u003d1/1"}],"source_content_type":"text/x-properties","patch_set":5,"id":"AAAAM3%2F%2FoYg%3D","line":4,"updated":"2013-01-11 21:57:00.000000000","message":"same comment that before, it will recommend to go back to \n\u003cput_nexus_switch_ip_address_here\u003e","commit_id":"b2034beb04176c70bbf33f8048eeec4aff77d8e2"},{"author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"unresolved":false,"context_lines":[{"line_number":5,"context_line":"[[[testhost]]]"},{"line_number":6,"context_line":"# Port this node is connected to on the nexus switch"},{"line_number":7,"context_line":"ports\u003d1/1"},{"line_number":8,"context_line":"# Port number where the SSH will be running at the Nexus Switch, e.g.: 22 (Default)"},{"line_number":9,"context_line":"[[[ssh_port]]]"},{"line_number":10,"context_line":"ssh_port\u003d22"},{"line_number":11,"context_line":""}],"source_content_type":"text/x-properties","patch_set":5,"id":"AAAAM3%2F%2FoYo%3D","line":8,"updated":"2013-01-11 21:57:00.000000000","message":"same comment that before, it will recommend to go back to \n\u003cput_interfaces_names_here_separated_by_commas\u003e","commit_id":"b2034beb04176c70bbf33f8048eeec4aff77d8e2"}],"quantum/plugins/cisco/db/nexus_db_v2.py":[{"author":{"_account_id":1994,"name":"Zhongyue Luo","email":"zhongyue.luo@gmail.com","username":"zyluo"},"change_message_id":"3a9718536fd6d58fe7564141445a6b97e6ea9c76","unresolved":false,"context_lines":[{"line_number":16,"context_line":"#"},{"line_number":17,"context_line":"# @author: Rohit Agarwalla, Cisco Systems, Inc."},{"line_number":18,"context_line":"# @author: Arvind Somya, Cisco Systems, Inc. (asomya@cisco.com)"},{"line_number":19,"context_line":"#"},{"line_number":20,"context_line":"import logging as LOG"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from sqlalchemy.orm import exc"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2F5mM%3D","line":19,"updated":"2012-12-22 15:36:15.000000000","message":"A blank line below here would be nice","commit_id":"b8e4fe0c7be466fb7b1fc0469aab43dc44ed8623"},{"author":{"_account_id":1994,"name":"Zhongyue Luo","email":"zhongyue.luo@gmail.com","username":"zyluo"},"change_message_id":"3a9718536fd6d58fe7564141445a6b97e6ea9c76","unresolved":false,"context_lines":[{"line_number":17,"context_line":"# @author: Rohit Agarwalla, Cisco Systems, Inc."},{"line_number":18,"context_line":"# @author: Arvind Somya, Cisco Systems, Inc. (asomya@cisco.com)"},{"line_number":19,"context_line":"#"},{"line_number":20,"context_line":"import logging as LOG"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from sqlalchemy.orm import exc"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2F5mI%3D","line":20,"updated":"2012-12-22 15:36:15.000000000","message":"Should use quantum.openstack.common.log","commit_id":"b8e4fe0c7be466fb7b1fc0469aab43dc44ed8623"},{"author":{"_account_id":1994,"name":"Zhongyue Luo","email":"zhongyue.luo@gmail.com","username":"zyluo"},"change_message_id":"3a9718536fd6d58fe7564141445a6b97e6ea9c76","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from sqlalchemy.orm import exc"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import quantum.db.api as db"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"from quantum.plugins.cisco.common import cisco_exceptions as c_exc"},{"line_number":27,"context_line":"from quantum.plugins.cisco.db import nexus_models_v2"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2F5mQ%3D","line":25,"updated":"2012-12-22 15:36:15.000000000","message":"This line should be deleted","commit_id":"b8e4fe0c7be466fb7b1fc0469aab43dc44ed8623"},{"author":{"_account_id":490,"name":"Sumit Naiksatam","email":"sumitnaiksatam@gmail.com","username":"snaiksat"},"unresolved":false,"context_lines":[{"line_number":18,"context_line":"# @author: Arvind Somya, Cisco Systems, Inc. (asomya@cisco.com)"},{"line_number":19,"context_line":"#"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"from quantum.openstack.common import log as logging"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"from sqlalchemy.orm import exc"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FodE%3D","line":21,"updated":"2013-01-11 21:28:29.000000000","message":"This should go in the imports block related to quantum","commit_id":"b2034beb04176c70bbf33f8048eeec4aff77d8e2"},{"author":{"_account_id":490,"name":"Sumit Naiksatam","email":"sumitnaiksatam@gmail.com","username":"snaiksat"},"unresolved":false,"context_lines":[{"line_number":25,"context_line":"import quantum.db.api as db"},{"line_number":26,"context_line":"from quantum.plugins.cisco.common import cisco_exceptions as c_exc"},{"line_number":27,"context_line":"from quantum.plugins.cisco.db import nexus_models_v2"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"LOG \u003d logging.getLogger()"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2Foc8%3D","line":28,"updated":"2013-01-11 21:28:29.000000000","message":"There should be an extra blank line here.","commit_id":"b2034beb04176c70bbf33f8048eeec4aff77d8e2"},{"author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"unresolved":false,"context_lines":[{"line_number":27,"context_line":"from quantum.plugins.cisco.db import nexus_models_v2"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"LOG \u003d logging.getLogger()"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"def get_all_nexusport_bindings():"},{"line_number":33,"context_line":"    \"\"\"Lists all the nexusport bindings\"\"\""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FoYA%3D","line":30,"updated":"2013-01-11 21:57:00.000000000","message":"The right logging should be:\nLOG \u003d logging.getLogger(__name__)","commit_id":"b2034beb04176c70bbf33f8048eeec4aff77d8e2"}],"quantum/plugins/cisco/models/virt_phy_sw_v2.py":[{"author":{"_account_id":1994,"name":"Zhongyue Luo","email":"zhongyue.luo@gmail.com","username":"zyluo"},"change_message_id":"3a9718536fd6d58fe7564141445a6b97e6ea9c76","unresolved":false,"context_lines":[{"line_number":28,"context_line":"from quantum.plugins.cisco.common import cisco_credentials_v2 as cred"},{"line_number":29,"context_line":"from quantum.plugins.cisco.db import network_db_v2 as cdb"},{"line_number":30,"context_line":"from quantum.plugins.cisco import l2network_plugin_configuration as conf"},{"line_number":31,"context_line":"from quantum.plugins.cisco.common import cisco_credentials_v2 as cred"},{"line_number":32,"context_line":"from quantum.plugins.openvswitch import ovs_db_v2 as odb"},{"line_number":33,"context_line":"from quantum import quantum_plugin_base_v2"},{"line_number":34,"context_line":"from novaclient.v1_1 import client as nova_client"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2F5mE%3D","line":31,"updated":"2012-12-22 15:36:15.000000000","message":"This module is already imported in line 28","commit_id":"b8e4fe0c7be466fb7b1fc0469aab43dc44ed8623"},{"author":{"_account_id":1994,"name":"Zhongyue Luo","email":"zhongyue.luo@gmail.com","username":"zyluo"},"change_message_id":"3a9718536fd6d58fe7564141445a6b97e6ea9c76","unresolved":false,"context_lines":[{"line_number":32,"context_line":"from quantum.plugins.openvswitch import ovs_db_v2 as odb"},{"line_number":33,"context_line":"from quantum import quantum_plugin_base_v2"},{"line_number":34,"context_line":"from novaclient.v1_1 import client as nova_client"},{"line_number":35,"context_line":"from keystoneclient.v2_0 import client as keystone_client"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":38,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2F5mA%3D","line":35,"updated":"2012-12-22 15:36:15.000000000","message":"import order error in lines 34 and 35","commit_id":"b8e4fe0c7be466fb7b1fc0469aab43dc44ed8623"},{"author":{"_account_id":490,"name":"Sumit Naiksatam","email":"sumitnaiksatam@gmail.com","username":"snaiksat"},"unresolved":false,"context_lines":[{"line_number":317,"context_line":"            net_dict \u003d self.get_network(context, net_id)"},{"line_number":318,"context_line":"            net_name \u003d net_dict[\u0027name\u0027]"},{"line_number":319,"context_line":""},{"line_number":320,"context_line":"            base_plugin_ref \u003d QuantumManager.get_plugin()"},{"line_number":321,"context_line":"            vlan_id \u003d self._get_segmentation_id(net_id)"},{"line_number":322,"context_line":"            host \u003d \u0027\u0027"},{"line_number":323,"context_line":"            if hasattr(conf, \u0027TEST\u0027):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2F5K0%3D","line":320,"updated":"2012-12-24 13:29:32.000000000","message":"Is base_plugin_ref being used anywhere? If not, it can be removed.","commit_id":"b8e4fe0c7be466fb7b1fc0469aab43dc44ed8623"},{"author":{"_account_id":490,"name":"Sumit Naiksatam","email":"sumitnaiksatam@gmail.com","username":"snaiksat"},"unresolved":false,"context_lines":[{"line_number":321,"context_line":"            vlan_id \u003d self._get_segmentation_id(net_id)"},{"line_number":322,"context_line":"            host \u003d \u0027\u0027"},{"line_number":323,"context_line":"            if hasattr(conf, \u0027TEST\u0027):"},{"line_number":324,"context_line":"                host \u003d conf.TEST[\u0027host\u0027]"},{"line_number":325,"context_line":"            else:"},{"line_number":326,"context_line":"                host \u003d self._get_instance_host(tenant_id, instance_id)"},{"line_number":327,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2F5Ks%3D","line":324,"updated":"2012-12-24 13:29:32.000000000","message":"Suggestion: There is probably a better way to pick up test configuration. One option might be to mock the _get_instance_host() method.","commit_id":"b8e4fe0c7be466fb7b1fc0469aab43dc44ed8623"},{"author":{"_account_id":490,"name":"Sumit Naiksatam","email":"sumitnaiksatam@gmail.com","username":"snaiksat"},"unresolved":false,"context_lines":[{"line_number":31,"context_line":"from quantum.plugins.openvswitch import ovs_db_v2 as odb"},{"line_number":32,"context_line":"from quantum import quantum_plugin_base_v2"},{"line_number":33,"context_line":"from keystoneclient.v2_0 import client as keystone_client"},{"line_number":34,"context_line":"from novaclient.v1_1 import client as nova_client"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAM3%2F%2FocY%3D","line":34,"updated":"2013-01-11 21:28:29.000000000","message":"ordering of imports is incorrect","commit_id":"b2034beb04176c70bbf33f8048eeec4aff77d8e2"}],"quantum/plugins/cisco/nexus/cisco_nexus_network_driver_v2.py":[{"author":{"_account_id":1994,"name":"Zhongyue Luo","email":"zhongyue.luo@gmail.com","username":"zyluo"},"change_message_id":"3a9718536fd6d58fe7564141445a6b97e6ea9c76","unresolved":false,"context_lines":[{"line_number":22,"context_line":"\"\"\""},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import logging"},{"line_number":25,"context_line":"import eventlet"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"from ncclient import manager"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2F5l4%3D","line":25,"updated":"2012-12-22 15:36:15.000000000","message":"lines 24 and 25 should switch places","commit_id":"b8e4fe0c7be466fb7b1fc0469aab43dc44ed8623"}],"quantum/plugins/cisco/nexus/cisco_nexus_plugin_v2.py":[{"author":{"_account_id":1994,"name":"Zhongyue Luo","email":"zhongyue.luo@gmail.com","username":"zyluo"},"change_message_id":"3a9718536fd6d58fe7564141445a6b97e6ea9c76","unresolved":false,"context_lines":[{"line_number":91,"context_line":"        # Check if this network is already in the DB"},{"line_number":92,"context_line":"        binding \u003d nxos_db.get_port_vlan_switch_binding("},{"line_number":93,"context_line":"            port_id, vlan_id, switch_ip)"},{"line_number":94,"context_line":"        if len(binding) \u003d\u003d 0:"},{"line_number":95,"context_line":"            _nexus_ip \u003d switch_ip"},{"line_number":96,"context_line":"            _nexus_ports \u003d (port_id,)"},{"line_number":97,"context_line":"            _nexus_ssh_port \u003d \\"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2F5l0%3D","line":94,"updated":"2012-12-22 15:36:15.000000000","message":"A list should not use len() to determine if empty according to pep8\n\nuse \"if not binding:\"","commit_id":"b8e4fe0c7be466fb7b1fc0469aab43dc44ed8623"},{"author":{"_account_id":1994,"name":"Zhongyue Luo","email":"zhongyue.luo@gmail.com","username":"zyluo"},"change_message_id":"3a9718536fd6d58fe7564141445a6b97e6ea9c76","unresolved":false,"context_lines":[{"line_number":101,"context_line":"            _nexus_password \u003d _nexus_creds[\u0027password\u0027]"},{"line_number":102,"context_line":"            # Check for vlan/switch binding"},{"line_number":103,"context_line":"            vbinding \u003d nxos_db.get_nexusvlan_binding(vlan_id, switch_ip)"},{"line_number":104,"context_line":"            if len(vbinding) \u003d\u003d 0:"},{"line_number":105,"context_line":"                # Create vlan and trunk vlan on the port"},{"line_number":106,"context_line":"                self._client.create_vlan("},{"line_number":107,"context_line":"                    vlan_name, str(vlan_id), _nexus_ip,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2F5lw%3D","line":104,"updated":"2012-12-22 15:36:15.000000000","message":"same here","commit_id":"b8e4fe0c7be466fb7b1fc0469aab43dc44ed8623"},{"author":{"_account_id":1994,"name":"Zhongyue Luo","email":"zhongyue.luo@gmail.com","username":"zyluo"},"change_message_id":"3a9718536fd6d58fe7564141445a6b97e6ea9c76","unresolved":false,"context_lines":[{"line_number":180,"context_line":"            bindings \u003d nxos_db.get_nexusvlan_binding("},{"line_number":181,"context_line":"                row[\u0027vlan_id\u0027], row[\u0027switch_ip\u0027])"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"            if len(bindings) \u003d\u003d 0:"},{"line_number":184,"context_line":"                # Delete this vlan from this switch"},{"line_number":185,"context_line":"                _nexus_ip \u003d row[\u0027switch_ip\u0027]"},{"line_number":186,"context_line":"                _nexus_ports \u003d (row[\u0027port_id\u0027],)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2F5ls%3D","line":183,"updated":"2012-12-22 15:36:15.000000000","message":"same here","commit_id":"b8e4fe0c7be466fb7b1fc0469aab43dc44ed8623"}],"tools/pip-requires":[{"author":{"_account_id":490,"name":"Sumit Naiksatam","email":"sumitnaiksatam@gmail.com","username":"snaiksat"},"unresolved":false,"context_lines":[{"line_number":14,"context_line":"sqlalchemy\u003d\u003d0.7.9"},{"line_number":15,"context_line":"webob\u003d\u003d1.0.8"},{"line_number":16,"context_line":"python-keystoneclient\u003e\u003d0.2.0"},{"line_number":17,"context_line":"python-novaclient"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAM3%2F%2F5Ko%3D","line":17,"updated":"2012-12-24 13:29:32.000000000","message":"Suggestion: Is this being only used by the Cisco plugin? If so, it might be good to qualify it as such. I was told earlier that it\u0027s easier to manage (i.e. delete) when it is known what the dependencies are.","commit_id":"b8e4fe0c7be466fb7b1fc0469aab43dc44ed8623"},{"author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"unresolved":false,"context_lines":[{"line_number":12,"context_line":"python-quantumclient\u003e\u003d2.0"},{"line_number":13,"context_line":"pyudev"},{"line_number":14,"context_line":"sqlalchemy\u003d\u003d0.7.9"},{"line_number":15,"context_line":"webob\u003e\u003d1.0.8"},{"line_number":16,"context_line":"alembic\u003e\u003d0.4.1"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"# Cisco plugin dependencies"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAM3%2F%2FoXU%3D","line":15,"updated":"2013-01-11 21:57:00.000000000","message":"python-keystoneclient\u003e\u003d0.2.0 should not be in Cisco\u0027s section","commit_id":"b2034beb04176c70bbf33f8048eeec4aff77d8e2"},{"author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"# Cisco plugin dependencies"},{"line_number":19,"context_line":"python-keystoneclient\u003e\u003d0.2.0"},{"line_number":20,"context_line":"python-novaclient"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAAM3%2F%2FoXM%3D","line":20,"updated":"2013-01-11 21:57:00.000000000","message":"Why is python-novaclient needed in this code?","commit_id":"b2034beb04176c70bbf33f8048eeec4aff77d8e2"}]}