)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4d8d6c69f84e2f0139e9778a20c7ce269557cfac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b4a4a24b_73508793","updated":"2022-03-15 15:22:37.000000000","message":"Something is not working... investigating now","commit_id":"7a1b1d13fb9566bdf6d861c90875b1524f48da8e"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"aaeff8f1f537a8b280cac5031700eca7e0bc137f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b75763d2_f2c1d576","updated":"2022-03-16 01:20:42.000000000","message":"Looking at local.conf of the ovn-octavia-provider-functional-release job, it still does not include OVN/OVS related variables..... I believe these configurations need to be under vars:devstack_localrc.\n\n[1] https://zuul.opendev.org/t/openstack/build/db7127cc90c94f88a7c42a4cf6cc5583/log/controller/logs/local_conf.txt","commit_id":"a4f69935d107a40f55dbcc980f0daa6cad951300"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"eb58feff93ac66ec14530214461d0099e05a7287","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"fabc4ad0_08f1591c","updated":"2022-03-18 10:20:36.000000000","message":"strange, for some reason it ","commit_id":"4aa0326feabb67ad94677f0846d72120b65ff52e"}],"zuul.d/base.yaml":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4ba34d48250926b563550665b45e9156d5e8ebd3","unresolved":true,"context_lines":[{"line_number":40,"context_line":"        OVN_BUILD_FROM_SOURCE: True"},{"line_number":41,"context_line":"        Q_BUILD_OVS_FROM_GIT: True"},{"line_number":42,"context_line":"        INSTALL_OVN: True"},{"line_number":43,"context_line":"        OVN_BRANCH: v20.06.0"},{"line_number":44,"context_line":"        OVS_BRANCH: v2.13.0"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"- job:"},{"line_number":47,"context_line":"    name: ovn-octavia-provider-functional-master"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"053529f1_ac39bc84","line":44,"range":{"start_line":43,"start_character":19,"end_line":44,"end_character":27},"updated":"2022-03-15 14:35:22.000000000","message":"Shouldn\u0027t we use newer versions?\n      OVN_BRANCH: v21.06.0\n      OVS_BRANCH: v2.16.0","commit_id":"c7e0231127677f8cab9f93893b062a12a90d1887"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"66b11188ffe7983a23a131892cfe2006e8743c97","unresolved":true,"context_lines":[{"line_number":40,"context_line":"        OVN_BUILD_FROM_SOURCE: True"},{"line_number":41,"context_line":"        Q_BUILD_OVS_FROM_GIT: True"},{"line_number":42,"context_line":"        INSTALL_OVN: True"},{"line_number":43,"context_line":"        OVN_BRANCH: v20.06.0"},{"line_number":44,"context_line":"        OVS_BRANCH: v2.13.0"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"- job:"},{"line_number":47,"context_line":"    name: ovn-octavia-provider-functional-master"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"e305ab2a_b83932f4","line":44,"range":{"start_line":43,"start_character":19,"end_line":44,"end_character":27},"in_reply_to":"053529f1_ac39bc84","updated":"2022-03-15 14:38:50.000000000","message":"++\n\nThanks for the pointer!","commit_id":"c7e0231127677f8cab9f93893b062a12a90d1887"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"f60dafa3b25bc830ccf37ad563aab57e2a4c5d74","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        OVN_BUILD_FROM_SOURCE: True"},{"line_number":41,"context_line":"        Q_BUILD_OVS_FROM_GIT: True"},{"line_number":42,"context_line":"        INSTALL_OVN: True"},{"line_number":43,"context_line":"        OVN_BRANCH: v20.06.0"},{"line_number":44,"context_line":"        OVS_BRANCH: v2.13.0"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"- job:"},{"line_number":47,"context_line":"    name: ovn-octavia-provider-functional-master"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"625bf807_7ea87863","line":44,"range":{"start_line":43,"start_character":19,"end_line":44,"end_character":27},"in_reply_to":"e305ab2a_b83932f4","updated":"2022-03-15 14:39:01.000000000","message":"Done","commit_id":"c7e0231127677f8cab9f93893b062a12a90d1887"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"0cf95fc9d76046ea8e1029ad99c73aeef8e1400e","unresolved":true,"context_lines":[{"line_number":36,"context_line":"    parent: ovn-octavia-provider-functional-base"},{"line_number":37,"context_line":"    description: Run OVN Octavia provider functional tests"},{"line_number":38,"context_line":"    vars:"},{"line_number":39,"context_line":"      devstack_localrc:"},{"line_number":40,"context_line":"        OVN_BUILD_FROM_SOURCE: True"},{"line_number":41,"context_line":"        Q_BUILD_OVS_FROM_GIT: True"},{"line_number":42,"context_line":"        INSTALL_OVN: True"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"e062072a_9695fc67","line":39,"range":{"start_line":39,"start_character":6,"end_line":39,"end_character":22},"updated":"2022-03-15 15:39:18.000000000","message":"Those variables should be under \"vars\", not \"vars:devstack_localrc\"","commit_id":"7a1b1d13fb9566bdf6d861c90875b1524f48da8e"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"eb58feff93ac66ec14530214461d0099e05a7287","unresolved":true,"context_lines":[{"line_number":36,"context_line":"    parent: ovn-octavia-provider-functional-base"},{"line_number":37,"context_line":"    description: Run OVN Octavia provider functional tests"},{"line_number":38,"context_line":"    vars:"},{"line_number":39,"context_line":"      devstack_localrc:"},{"line_number":40,"context_line":"        OVN_BUILD_FROM_SOURCE: True"},{"line_number":41,"context_line":"        Q_BUILD_OVS_FROM_GIT: True"},{"line_number":42,"context_line":"        INSTALL_OVN: True"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"0501863c_1caaf186","line":39,"range":{"start_line":39,"start_character":6,"end_line":39,"end_character":22},"in_reply_to":"4aa877f4_eb86d9ef","updated":"2022-03-18 10:20:36.000000000","message":"interesting... it seems the local.conf does not have that info, yet ovs is configure with the right one, so, yep, vars is right here, and devstack_localrc is right on the tempest side","commit_id":"7a1b1d13fb9566bdf6d861c90875b1524f48da8e"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"2daac4a403e1f3660a41a21ce88394867e3130a7","unresolved":true,"context_lines":[{"line_number":36,"context_line":"    parent: ovn-octavia-provider-functional-base"},{"line_number":37,"context_line":"    description: Run OVN Octavia provider functional tests"},{"line_number":38,"context_line":"    vars:"},{"line_number":39,"context_line":"      devstack_localrc:"},{"line_number":40,"context_line":"        OVN_BUILD_FROM_SOURCE: True"},{"line_number":41,"context_line":"        Q_BUILD_OVS_FROM_GIT: True"},{"line_number":42,"context_line":"        INSTALL_OVN: True"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"4aa877f4_eb86d9ef","line":39,"range":{"start_line":39,"start_character":6,"end_line":39,"end_character":22},"in_reply_to":"e062072a_9695fc67","updated":"2022-03-15 16:18:50.000000000","message":"Ah thanks for fixing it, I thought \"devstack_localrc\" was needed because see L170","commit_id":"7a1b1d13fb9566bdf6d861c90875b1524f48da8e"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"824c3fa356f283b4ca47713b298082b813abbbc8","unresolved":true,"context_lines":[{"line_number":35,"context_line":"    name: ovn-octavia-provider-functional-release"},{"line_number":36,"context_line":"    parent: ovn-octavia-provider-functional-base"},{"line_number":37,"context_line":"    description: Run OVN Octavia provider functional tests"},{"line_number":38,"context_line":"    vars:"},{"line_number":39,"context_line":"      OVN_BUILD_FROM_SOURCE: True"},{"line_number":40,"context_line":"      Q_BUILD_OVS_FROM_GIT: True"},{"line_number":41,"context_line":"      INSTALL_OVN: True"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"0cb0783f_a1f8d3fe","line":38,"range":{"start_line":38,"start_character":1,"end_line":38,"end_character":9},"updated":"2022-03-16 08:00:39.000000000","message":"either this is with vars/devstack_locals, or the ones further down should drop it, right? (line 75 for instance), perhaps, if it should not be there ,that is why the local.conf is not properly formatted and devstack_localrc should be removed from the other points","commit_id":"a4f69935d107a40f55dbcc980f0daa6cad951300"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"f71402a54557f91dd65e858971f7ee750fc1d437","unresolved":true,"context_lines":[{"line_number":35,"context_line":"    name: ovn-octavia-provider-functional-release"},{"line_number":36,"context_line":"    parent: ovn-octavia-provider-functional-base"},{"line_number":37,"context_line":"    description: Run OVN Octavia provider functional tests"},{"line_number":38,"context_line":"    vars:"},{"line_number":39,"context_line":"      OVN_BUILD_FROM_SOURCE: True"},{"line_number":40,"context_line":"      Q_BUILD_OVS_FROM_GIT: True"},{"line_number":41,"context_line":"      INSTALL_OVN: True"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"04485f4c_4a3e1780","line":38,"range":{"start_line":38,"start_character":1,"end_line":38,"end_character":9},"in_reply_to":"0cb0783f_a1f8d3fe","updated":"2022-03-18 08:50:05.000000000","message":"checkint the local.conf generated, I think it is wrong to not use devstack_localrc. The generated local.conf in functional-release is wrong (e.g: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_db7/833798/3/check/ovn-octavia-provider-functional-release/db7127c/controller/logs/local_conf.txt), while the one on tempest-release looks good (e.g.: https://71d6e15cf166f312dff0-9dc687b1ca3b864db38ca15564d88e40.ssl.cf2.rackcdn.com/833798/3/check/ovn-octavia-provider-tempest-release/c178734/controller/logs/local_conf.txt)","commit_id":"a4f69935d107a40f55dbcc980f0daa6cad951300"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"2daac4a403e1f3660a41a21ce88394867e3130a7","unresolved":true,"context_lines":[{"line_number":167,"context_line":"    name: ovn-octavia-provider-tempest-release"},{"line_number":168,"context_line":"    parent: ovn-octavia-provider-tempest-base"},{"line_number":169,"context_line":"    vars:"},{"line_number":170,"context_line":"      devstack_localrc:"},{"line_number":171,"context_line":"        OVN_BUILD_FROM_SOURCE: True"},{"line_number":172,"context_line":"        OVN_BRANCH: v20.06.0"},{"line_number":173,"context_line":"        OVS_BRANCH: v2.13.0"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"5faff212_91e35ad4","line":170,"updated":"2022-03-15 16:18:50.000000000","message":"here ^","commit_id":"a4f69935d107a40f55dbcc980f0daa6cad951300"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"824c3fa356f283b4ca47713b298082b813abbbc8","unresolved":true,"context_lines":[{"line_number":169,"context_line":"    vars:"},{"line_number":170,"context_line":"      devstack_localrc:"},{"line_number":171,"context_line":"        OVN_BUILD_FROM_SOURCE: True"},{"line_number":172,"context_line":"        OVN_BRANCH: v20.06.0"},{"line_number":173,"context_line":"        OVS_BRANCH: v2.13.0"},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"- job:"},{"line_number":176,"context_line":"    name: ovn-octavia-provider-tempest-master"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"59bd739d_984f56a7","line":173,"range":{"start_line":172,"start_character":1,"end_line":173,"end_character":27},"updated":"2022-03-16 08:00:39.000000000","message":"we should bump this too, right?","commit_id":"a4f69935d107a40f55dbcc980f0daa6cad951300"}]}
