)]}'
{".zuul.yaml":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c9069d627cc853a82d01b12957ca72673b25b822","unresolved":true,"context_lines":[{"line_number":250,"context_line":"        br-int-flows: false"},{"line_number":251,"context_line":"        ovn-controller: false"},{"line_number":252,"context_line":"        ovn-northd: false"},{"line_number":253,"context_line":"        ovs-vswitchd: false"},{"line_number":254,"context_line":"        ovsdb-server: false"},{"line_number":255,"context_line":"        q-ovn-metadata-agent: false"},{"line_number":256,"context_line":"        # Neutron services"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"c205aff8_92db9482","line":253,"updated":"2021-03-03 04:00:18.000000000","message":"ovs-vswitchd and  ovsdb-server proably should still be enable no?\ni really hope this is not contoling compileing ovs from source.","commit_id":"036cc637d911103da3edec419d6d4585cd2da8e4"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"7c209fdb73849612294e2b0b8c6e2e6e994cbba8","unresolved":true,"context_lines":[{"line_number":250,"context_line":"        br-int-flows: false"},{"line_number":251,"context_line":"        ovn-controller: false"},{"line_number":252,"context_line":"        ovn-northd: false"},{"line_number":253,"context_line":"        ovs-vswitchd: false"},{"line_number":254,"context_line":"        ovsdb-server: false"},{"line_number":255,"context_line":"        q-ovn-metadata-agent: false"},{"line_number":256,"context_line":"        # Neutron services"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"6cae8a70_ffe95b18","line":253,"in_reply_to":"c205aff8_92db9482","updated":"2021-03-03 09:42:12.000000000","message":"I think only the OVN module check those those services being enabled/disabled. But looks odd here indeed. I will remove them.\n\nRegarding compiling from source, we no longer do that since https://review.opendev.org/c/openstack/devstack/+/763402/. Now we only will compile it if the OVN_BUILD_FROM_SOURCE is set to True (it defaults to False), but it\u0027s not set anywhere other than a few jobs testing OVN from master branch in the Neutron experimental pipeline.","commit_id":"036cc637d911103da3edec419d6d4585cd2da8e4"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"91a37ec3c4c8b2c6bb10931b5c74f638e2b96837","unresolved":true,"context_lines":[{"line_number":261,"context_line":"        ovn-northd: false"},{"line_number":262,"context_line":"        ovs-vswitchd: false"},{"line_number":263,"context_line":"        ovsdb-server: false"},{"line_number":264,"context_line":"        q-ovn-metadata-agent: false"},{"line_number":265,"context_line":"        # Neutron services"},{"line_number":266,"context_line":"        q-agt: true"},{"line_number":267,"context_line":"        q-dhcp: true"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"b24a1796_c3c6cfaf","line":264,"updated":"2021-04-08 10:32:23.000000000","message":"are we sure we shoudl be setting\novs-vswitchd: false\novsdb-server: false\n\ngiven we are still using ovs?","commit_id":"a0aa25538cae814c2678549853b2c76e58937871"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"0d9a8289f8df402a6974271df69e18538fecb2c7","unresolved":true,"context_lines":[{"line_number":261,"context_line":"        ovn-northd: false"},{"line_number":262,"context_line":"        ovs-vswitchd: false"},{"line_number":263,"context_line":"        ovsdb-server: false"},{"line_number":264,"context_line":"        q-ovn-metadata-agent: false"},{"line_number":265,"context_line":"        # Neutron services"},{"line_number":266,"context_line":"        q-agt: true"},{"line_number":267,"context_line":"        q-dhcp: true"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"a0fc6d3c_b61b6487","line":264,"in_reply_to":"b24a1796_c3c6cfaf","updated":"2021-04-08 12:42:23.000000000","message":"It should be fine as-is given https://review.opendev.org/c/openstack/nova/+/776945/ being happy with this change but, I also think that there\u0027s no harm in removing those from there. I believe it\u0027s only the OVN module for DevStack that checks for those services being enabled/disabled.\n\nI will remove it and re-test.","commit_id":"a0aa25538cae814c2678549853b2c76e58937871"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6cf6832a3db2c435b4d25379fe1cd5c37299b4e9","unresolved":true,"context_lines":[{"line_number":171,"context_line":"        # Disable non-essential services that we don\u0027t need for this job."},{"line_number":172,"context_line":"        c-bak: false"},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"#TODO(lucasagomes): Move this job to ML2/OVN when QoS Minimum Bandwidth"},{"line_number":175,"context_line":"# support is implemented."},{"line_number":176,"context_line":"# See: https://docs.openstack.org/neutron/latest/ovn/gaps.html"},{"line_number":177,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"093781ec_71143ccf","line":174,"range":{"start_line":174,"start_character":1,"end_line":174,"end_character":2},"updated":"2021-04-16 11:21:07.000000000","message":"nit: missing space","commit_id":"a3c44c62437c630003bf88de45ce16aa4115d622"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"73ac7694f91473345ed814408675b23deb76c8df","unresolved":true,"context_lines":[{"line_number":171,"context_line":"        # Disable non-essential services that we don\u0027t need for this job."},{"line_number":172,"context_line":"        c-bak: false"},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"#TODO(lucasagomes): Move this job to ML2/OVN when QoS Minimum Bandwidth"},{"line_number":175,"context_line":"# support is implemented."},{"line_number":176,"context_line":"# See: https://docs.openstack.org/neutron/latest/ovn/gaps.html"},{"line_number":177,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"7c7f84fa_ea7187a3","line":174,"range":{"start_line":174,"start_character":1,"end_line":174,"end_character":2},"in_reply_to":"093781ec_71143ccf","updated":"2021-04-19 08:39:55.000000000","message":"Thanks, if I need to re-spin a new patch-set I will fix it","commit_id":"a3c44c62437c630003bf88de45ce16aa4115d622"}],"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"b6e125b627f4cff6e8e226a2fb9f3bafbcf8cdea","unresolved":true,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Devstack is changing the Neutron default to OVN backend. Currently, the"},{"line_number":12,"context_line":"nova-next job requires QoS minimum bandwidth support in its"},{"line_number":13,"context_line":"post_test_hook.sh script [0] which is currently not supported in OVN [1]"},{"line_number":14,"context_line":"(feature needs to come to core OVN first)."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"[0]"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"4ac0311a_9bcb12c2","line":13,"range":{"start_line":13,"start_character":47,"end_line":13,"end_character":60},"updated":"2021-04-19 08:44:07.000000000","message":"Currently working on this. We\u0027ll support minBW in OVN to support placement scheduling based on those rules [1], not driver enforcement. Just to inform.\n\n\n[1]https://review.opendev.org/c/openstack/neutron/+/786441","commit_id":"a3c44c62437c630003bf88de45ce16aa4115d622"}]}