)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"85434b2232404bb5320f7439180058ebddfb6304","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Noam Angel \u003cnoama@mellanox.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-01-22 06:39:00 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"cleanup inspector interface if not equal to OVS_PHYSICAL_BRIDGE"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In a scenario we want to deploy DevStack with ironic inspector for real"},{"line_number":10,"context_line":"hardware and use OVS_PHYSICAL_BRIDGE for IRONIC_INSPECTOR_INTERFACE as"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5a3905b3_92dd70e2","line":7,"updated":"2017-01-23 12:41:14.000000000","message":"nit: \"[devstack] Remove bridge only if it\u0027s not OVB_PHYSICAL_BRIDGE\"","commit_id":"add1856d7306a6f684be2d9d17542520fb968ecf"}],"devstack/plugin.sh":[{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"1257b8c77d556051c0d2cd2499748464ea00f3c3","unresolved":false,"context_lines":[{"line_number":32,"context_line":"IRONIC_INSPECTOR_INTERNAL_IP_WITH_NET\u003d\"$IRONIC_INSPECTOR_INTERNAL_IP/$IRONIC_INSPECTOR_INTERNAL_SUBNET_SIZE\""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"IRONIC_INSPECTOR_NODE_NOT_FOUND_HOOK\u003d${IRONIC_INSPECTOR_NODE_NOT_FOUND_HOOK:-\"\"}"},{"line_number":35,"context_line":"OVS_PHYSICAL_BRIDGE\u003d${OVS_PHYSICAL_BRIDGE:-\"\"}"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"GITDIR[\"python-ironic-inspector-client\"]\u003d$DEST/python-ironic-inspector-client"},{"line_number":38,"context_line":"GITREPO[\"python-ironic-inspector-client\"]\u003d${IRONIC_INSPECTOR_CLIENT_REPO:-${GIT_BASE}/openstack/python-ironic-inspector-client.git}"}],"source_content_type":"text/x-sh","patch_set":3,"id":"5a3905b3_3303670f","line":35,"updated":"2017-01-25 12:09:44.000000000","message":"nit: no need to specify \"\", using ${OVS_PHYSICAL_BRIDGE:-} is enough.\nAlso this should come from Neutron, It makes no sense to set OVS_PHYSYCAL_BRIDGE for standalone inspector as no code using this variable.\nThe default is set on neutron devstack plugin side, no need to redefine it here.\nJust drop L35.","commit_id":"78beebf88f5d10d3a70a28ffc9516932350d4e80"}]}