)]}'
{"lib/neutron_plugins/cisco":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"442218ea12f46797da93d4f07b1fb5cd43e9cbb2","unresolved":false,"context_lines":[{"line_number":146,"context_line":""},{"line_number":147,"context_line":"function has_neutron_plugin_security_group {"},{"line_number":148,"context_line":"    if _has_ovs_subplugin; then"},{"line_number":149,"context_line":"        ovs_has_neutron_plugin_security_group"},{"line_number":150,"context_line":"    else"},{"line_number":151,"context_line":"        return 1"},{"line_number":152,"context_line":"    fi"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"fa98f980_5831a3a0","side":"PARENT","line":149,"updated":"2014-09-15 22:32:43.000000000","message":"this is the only call to has_neutron_plugin_security_group, so we can remove the fn all together?","commit_id":"887fb54013feff6e063001fb4d879666b38fdbbd"},{"author":{"_account_id":5892,"name":"pritesh","email":"pritesh@romana.io","username":"pritkoth"},"change_message_id":"204136de85a60d417414a992527bd366e74cc541","unresolved":false,"context_lines":[{"line_number":146,"context_line":""},{"line_number":147,"context_line":"function has_neutron_plugin_security_group {"},{"line_number":148,"context_line":"    if _has_ovs_subplugin; then"},{"line_number":149,"context_line":"        ovs_has_neutron_plugin_security_group"},{"line_number":150,"context_line":"    else"},{"line_number":151,"context_line":"        return 1"},{"line_number":152,"context_line":"    fi"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"fa98f980_6d2e8efc","side":"PARENT","line":149,"in_reply_to":"fa98f980_5831a3a0","updated":"2014-09-16 16:17:41.000000000","message":"Actually this is ovs_has_neutron_plugin_security_group function not has_neutron_plugin_security_group and secondly the has_neutron_plugin_security_group is called from lib/neutron:289 so it can\u0027t be removed need to be there else devstack fails.","commit_id":"887fb54013feff6e063001fb4d879666b38fdbbd"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"442218ea12f46797da93d4f07b1fb5cd43e9cbb2","unresolved":false,"context_lines":[{"line_number":52,"context_line":"_prefix_function has_neutron_plugin_security_group ovs"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"function has_neutron_plugin_security_group {"},{"line_number":55,"context_line":"    return 1"},{"line_number":56,"context_line":"}"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"function is_neutron_ovs_base_plugin {"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"fa98f980_d8f593cb","line":55,"updated":"2014-09-15 22:32:43.000000000","message":"this is not required?","commit_id":"3ad716cd9a5abb1286c979497bc4e26d691dabc7"},{"author":{"_account_id":5892,"name":"pritesh","email":"pritesh@romana.io","username":"pritkoth"},"change_message_id":"204136de85a60d417414a992527bd366e74cc541","unresolved":false,"context_lines":[{"line_number":52,"context_line":"_prefix_function has_neutron_plugin_security_group ovs"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"function has_neutron_plugin_security_group {"},{"line_number":55,"context_line":"    return 1"},{"line_number":56,"context_line":"}"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"function is_neutron_ovs_base_plugin {"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"fa98f980_8d0ad277","line":55,"in_reply_to":"fa98f980_d8f593cb","updated":"2014-09-16 16:17:41.000000000","message":"this is required as explained earlier becuase it is called from lib/neutron:289 so it can\u0027t be removed, needs to be there else devstack fails.","commit_id":"3ad716cd9a5abb1286c979497bc4e26d691dabc7"}]}