)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"change_message_id":"6c8bcc548540bd19dfa724126a257bd0110f2581","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Functional tests: Use the latest version of OVN"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Compile the newer OVN version (from ovn-org) and use it in the"},{"line_number":10,"context_line":"functional tests."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I89d08dcb37d1fe6dba2b4db16303f0660b2b034a"},{"line_number":13,"context_line":"Signed-off-by: Lucas Alvares Gomes \u003clucasagomes@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"3fa7e38b_94d87452","line":10,"range":{"start_line":10,"start_character":16,"end_line":10,"end_character":17},"updated":"2019-10-25 13:58:01.000000000","message":"Maybe add:\n\nThis partially reverts commit 6d540e49e6c483990d3ae87d35992a9d67e670be.\n\nor\n\n    Related changes:\n    * https://review.opendev.org/#/c/680995/\n\nFor easy reference?","commit_id":"01e49f48ca33411441a2f0b41448f34166105d42"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"6b68a3175e10bd1cf9fdcc68663d0901c9d0eccc","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Functional tests: Use the latest version of OVN"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Compile the newer OVN version (from ovn-org) and use it in the"},{"line_number":10,"context_line":"functional tests."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I89d08dcb37d1fe6dba2b4db16303f0660b2b034a"},{"line_number":13,"context_line":"Signed-off-by: Lucas Alvares Gomes \u003clucasagomes@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"3fa7e38b_f4c208e9","line":10,"range":{"start_line":10,"start_character":16,"end_line":10,"end_character":17},"in_reply_to":"3fa7e38b_94d87452","updated":"2019-10-25 14:10:41.000000000","message":"Thanks for the suggestion! It would be a good addition but, since the gate is broken and this patch is in merging status (2x+2\u0027s and +A) I think it\u0027s better to pass on this one ?","commit_id":"01e49f48ca33411441a2f0b41448f34166105d42"},{"author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"change_message_id":"5815648dcf0ff4f3f20cb5d206f4542e2d9a4341","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Functional tests: Use the latest version of OVN"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Compile the newer OVN version (from ovn-org) and use it in the"},{"line_number":10,"context_line":"functional tests."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I89d08dcb37d1fe6dba2b4db16303f0660b2b034a"},{"line_number":13,"context_line":"Signed-off-by: Lucas Alvares Gomes \u003clucasagomes@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"3fa7e38b_f72c4a9a","line":10,"range":{"start_line":10,"start_character":16,"end_line":10,"end_character":17},"in_reply_to":"3fa7e38b_f4c208e9","updated":"2019-10-25 14:18:24.000000000","message":"Absolutely okay!","commit_id":"01e49f48ca33411441a2f0b41448f34166105d42"}],"devstack/lib/ovn":[{"author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"change_message_id":"fcf5191d4452ea67a7656de77b5ffc383b029264","unresolved":false,"context_lines":[{"line_number":147,"context_line":"    local localstatedir\u003d$3"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"    # Install the dependencies"},{"line_number":150,"context_line":"    install_package autoconf automake libtool gcc patch make bridge-utils python3-six"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"    # First, compile OVS"},{"line_number":153,"context_line":"    _compile_ovs $build_modules"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"3fa7e38b_2b784f8d","line":150,"range":{"start_line":150,"start_character":61,"end_line":150,"end_character":73},"updated":"2019-10-24 19:48:52.000000000","message":"bridge-utils needed? I\u0027m a bit surprised...","commit_id":"9875af0a99193a9287e603b640d89cb41c411541"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"83fd7283bb7592f8fd5eb30070cacfddac32cea0","unresolved":false,"context_lines":[{"line_number":147,"context_line":"    local localstatedir\u003d$3"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"    # Install the dependencies"},{"line_number":150,"context_line":"    install_package autoconf automake libtool gcc patch make bridge-utils python3-six"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"    # First, compile OVS"},{"line_number":153,"context_line":"    _compile_ovs $build_modules"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"3fa7e38b_a2445975","line":150,"range":{"start_line":150,"start_character":61,"end_line":150,"end_character":73},"in_reply_to":"3fa7e38b_2b784f8d","updated":"2019-10-25 08:54:12.000000000","message":"Yeah perhaps it\u0027s not needed, I saw some error in the logs related to br-ctl not being found so I\u0027ve added it here.\n\nBut yeah, lemme take it out since I think it\u0027s not related to the OVN/OVS compilation","commit_id":"9875af0a99193a9287e603b640d89cb41c411541"},{"author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"change_message_id":"08af49ef7782614b6048fdd754126ae24ba07aa9","unresolved":false,"context_lines":[{"line_number":147,"context_line":"    local localstatedir\u003d$3"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"    # Install the dependencies"},{"line_number":150,"context_line":"    install_package autoconf automake libtool gcc patch make python3-six"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"    # First, compile OVS"},{"line_number":153,"context_line":"    _compile_ovs $build_modules"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"3fa7e38b_70cd925b","line":150,"range":{"start_line":150,"start_character":61,"end_line":150,"end_character":72},"updated":"2019-10-29 21:46:26.000000000","message":"heh, when using devstack on centos7 box, I am getting in trouble due to this now.\n\n[vagrant@devstack logs]$ sudo yum install python3-six\nNo package python3-six available.","commit_id":"01e49f48ca33411441a2f0b41448f34166105d42"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"830274c54933265386279e602702911e74855033","unresolved":false,"context_lines":[{"line_number":147,"context_line":"    local localstatedir\u003d$3"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"    # Install the dependencies"},{"line_number":150,"context_line":"    install_package autoconf automake libtool gcc patch make python3-six"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"    # First, compile OVS"},{"line_number":153,"context_line":"    _compile_ovs $build_modules"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"3fa7e38b_71791180","line":150,"range":{"start_line":150,"start_character":61,"end_line":150,"end_character":72},"in_reply_to":"3fa7e38b_70cd925b","updated":"2019-10-30 08:54:50.000000000","message":"Ops, yeah there\u0027s no python3 on centos7. Perhaps it would be better to install six from pip instead of packages ?","commit_id":"01e49f48ca33411441a2f0b41448f34166105d42"}],"networking_ovn/tests/contrib/gate_hook.sh":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"05369513072dd9fd3c123d4a998a84431c4e305b","unresolved":false,"context_lines":[{"line_number":34,"context_line":"    # In order to run functional tests, we want to compile OVS"},{"line_number":35,"context_line":"    # from sources and installed. We don\u0027t need to start ovs services."},{"line_number":36,"context_line":"    remove_ovs_packages"},{"line_number":37,"context_line":"    # compile_ovs expects \"DEST\" to be defined"},{"line_number":38,"context_line":"    DEST\u003d$GATE_DEST"},{"line_number":39,"context_line":"    compile_ovn False /usr/local /var"},{"line_number":40,"context_line":""}],"source_content_type":"text/x-sh","patch_set":6,"id":"3fa7e38b_9da97e91","line":37,"range":{"start_line":37,"start_character":16,"end_line":37,"end_character":17},"updated":"2019-10-25 09:31:40.000000000","message":"n","commit_id":"01e49f48ca33411441a2f0b41448f34166105d42"}]}