)]}'
{"neutron/tests/contrib/gate_hook.sh":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"14fc6cf94f7f6e4d3865bcfe463272fb8ebdb0e2","unresolved":false,"context_lines":[{"line_number":91,"context_line":"    if [[ \"$VENV\" \u003d~ \"dsvm-scenario\" ]]; then"},{"line_number":92,"context_line":"        load_conf_hook iptables_verify"},{"line_number":93,"context_line":"        load_rc_hook ubuntu_image"},{"line_number":94,"context_line":"        if [[ ! \"$VENV\" \u003d~ \"linuxbridge\" ]]; then"},{"line_number":95,"context_line":"            # OVS scenario needs ovsfw for trunk filtering"},{"line_number":96,"context_line":"            load_conf_hook ovsfw"},{"line_number":97,"context_line":"        fi"}],"source_content_type":"text/x-sh","patch_set":2,"id":"ba2be162_d9fd863b","line":94,"updated":"2017-03-02 15:36:41.000000000","message":"suggest to instead match against ovs here, more explicit","commit_id":"67916d0e05b52ae7a8e2ffd11fef82f07802ad5f"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"5c35a0bb246ace6d8335150dd3ce362176a7bf79","unresolved":false,"context_lines":[{"line_number":91,"context_line":"    if [[ \"$VENV\" \u003d~ \"dsvm-scenario\" ]]; then"},{"line_number":92,"context_line":"        load_conf_hook iptables_verify"},{"line_number":93,"context_line":"        load_rc_hook ubuntu_image"},{"line_number":94,"context_line":"        if [[ ! \"$VENV\" \u003d~ \"linuxbridge\" ]]; then"},{"line_number":95,"context_line":"            # OVS scenario needs ovsfw for trunk filtering"},{"line_number":96,"context_line":"            load_conf_hook ovsfw"},{"line_number":97,"context_line":"        fi"}],"source_content_type":"text/x-sh","patch_set":2,"id":"ba2be162_3ea201fe","line":94,"in_reply_to":"ba2be162_94cbd352","updated":"2017-03-02 16:50:32.000000000","message":"well i tried this because ovs isn\u0027t mentioned in the job name anywhere.","commit_id":"67916d0e05b52ae7a8e2ffd11fef82f07802ad5f"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"86ccf6f20c130420ed88937bcde9e57963a18bfa","unresolved":false,"context_lines":[{"line_number":91,"context_line":"    if [[ \"$VENV\" \u003d~ \"dsvm-scenario\" ]]; then"},{"line_number":92,"context_line":"        load_conf_hook iptables_verify"},{"line_number":93,"context_line":"        load_rc_hook ubuntu_image"},{"line_number":94,"context_line":"        if [[ ! \"$VENV\" \u003d~ \"linuxbridge\" ]]; then"},{"line_number":95,"context_line":"            # OVS scenario needs ovsfw for trunk filtering"},{"line_number":96,"context_line":"            load_conf_hook ovsfw"},{"line_number":97,"context_line":"        fi"}],"source_content_type":"text/x-sh","patch_set":2,"id":"ba2be162_94cbd352","line":94,"in_reply_to":"ba2be162_d9fd863b","updated":"2017-03-02 15:45:28.000000000","message":"+1, say we add myfancybackend to the gate, this will end up breaking. Positive logic wins with me in this case.","commit_id":"67916d0e05b52ae7a8e2ffd11fef82f07802ad5f"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"92000a9acdbe1fea4eedb0f2e2e44751b243ad59","unresolved":false,"context_lines":[{"line_number":91,"context_line":"    if [[ \"$VENV\" \u003d~ \"dsvm-scenario\" ]]; then"},{"line_number":92,"context_line":"        load_conf_hook iptables_verify"},{"line_number":93,"context_line":"        load_rc_hook ubuntu_image"},{"line_number":94,"context_line":"        if [[ \"$VENV\" \u003d~ \"ovs\" ]]; then"},{"line_number":95,"context_line":"            # OVS scenario needs ovsfw for trunk filtering"},{"line_number":96,"context_line":"            load_conf_hook ovsfw"},{"line_number":97,"context_line":"        fi"},{"line_number":98,"context_line":"    fi"},{"line_number":99,"context_line":"    if [[ \"$VENV\" \u003d~ \"pecan\" ]]; then"},{"line_number":100,"context_line":"        load_conf_hook pecan"}],"source_content_type":"text/x-sh","patch_set":3,"id":"ba2be162_5b7f9165","line":97,"range":{"start_line":94,"start_character":8,"end_line":97,"end_character":10},"updated":"2017-03-03 16:29:16.000000000","message":"sec, won\u0027t line 102 suffice if only we change it to matching with \u0027ovs\u0027? also, line 83 will not allow dsvm-scenario-[ovs|linuxbridge] in...","commit_id":"3c577b41ee7e9236a0e7801d32d77a74316f050b"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"e06d36207be065ab804a91c989f226f95800cf24","unresolved":false,"context_lines":[{"line_number":91,"context_line":"    if [[ \"$VENV\" \u003d~ \"dsvm-scenario\" ]]; then"},{"line_number":92,"context_line":"        load_conf_hook iptables_verify"},{"line_number":93,"context_line":"        load_rc_hook ubuntu_image"},{"line_number":94,"context_line":"        if [[ \"$VENV\" \u003d~ \"ovs\" ]]; then"},{"line_number":95,"context_line":"            # OVS scenario needs ovsfw for trunk filtering"},{"line_number":96,"context_line":"            load_conf_hook ovsfw"},{"line_number":97,"context_line":"        fi"},{"line_number":98,"context_line":"    fi"},{"line_number":99,"context_line":"    if [[ \"$VENV\" \u003d~ \"pecan\" ]]; then"},{"line_number":100,"context_line":"        load_conf_hook pecan"}],"source_content_type":"text/x-sh","patch_set":3,"id":"ba2be162_ad6054d0","line":97,"range":{"start_line":94,"start_character":8,"end_line":97,"end_character":10},"in_reply_to":"ba2be162_5b7f9165","updated":"2017-03-03 16:46:08.000000000","message":"Sure","commit_id":"3c577b41ee7e9236a0e7801d32d77a74316f050b"}]}