)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"f153b0442f021719c0f5671a09a52b7d64c9ca49","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2134f8ed_ab45e973","updated":"2023-07-05 11:02:42.000000000","message":"Thanks Lucas, can you also include the fix for victoria ovn job too along","commit_id":"09e94103286c76576980f4041a1a748aa46dc887"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"4fade914b6ff292daad24428e432d6a743e33d6b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"6090944c_d7c598db","updated":"2023-07-05 14:31:05.000000000","message":"thanks, I just run into this","commit_id":"24b5edd88c8eb889e05dfccd72ccba136661dcb9"}],"zuul.d/wallaby_jobs.yaml":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"5f5980d74d2ce71f76f1672fa74e217216fab074","unresolved":true,"context_lines":[{"line_number":178,"context_line":"        # TODO(lucasagomes): Remove the OVN_BRANCH and OVS_BRANCH config"},{"line_number":179,"context_line":"        # once https://review.opendev.org/c/openstack/devstack/+/887184"},{"line_number":180,"context_line":"        # is merged"},{"line_number":181,"context_line":"        OVN_BRANCH: \"branch-22.03\""},{"line_number":182,"context_line":"        OVS_BRANCH: \"branch-3.0\""},{"line_number":183,"context_line":"      devstack_local_conf:"},{"line_number":184,"context_line":"        test-config:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"4458a842_c50be162","line":181,"updated":"2023-07-05 11:09:57.000000000","message":"also i think would be enough just to set OVS_BRANCH here, to avoid following all changes in ovn 22.03 branch.","commit_id":"09e94103286c76576980f4041a1a748aa46dc887"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"75449d6f92e480f8be14b6caf854bd9fb3e4934f","unresolved":true,"context_lines":[{"line_number":178,"context_line":"        # TODO(lucasagomes): Remove the OVN_BRANCH and OVS_BRANCH config"},{"line_number":179,"context_line":"        # once https://review.opendev.org/c/openstack/devstack/+/887184"},{"line_number":180,"context_line":"        # is merged"},{"line_number":181,"context_line":"        OVN_BRANCH: \"branch-22.03\""},{"line_number":182,"context_line":"        OVS_BRANCH: \"branch-3.0\""},{"line_number":183,"context_line":"      devstack_local_conf:"},{"line_number":184,"context_line":"        test-config:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"80dc3381_769129a8","line":181,"in_reply_to":"20018435_72f7c33a","updated":"2023-07-05 11:20:20.000000000","message":"can ignore considering this temporary.\n\nwhat i meant was we used to keep OVN/OVS_BRANCH to be fixed by pinning them to tag or commits so the jobs keep testing same versions. Using branches here makes the job not testing fixed version but also any new changes in the branches which we should avoid.\n\nOVS_BRANCH already support pinning to commits, devstack patch was to make it support for OVN_BRANCH. So here we could have only set OVN_BRANCH And keep using OVS_BRANCH from master jobs so the jobs not get impacted with any new changes to branch-3.0.","commit_id":"09e94103286c76576980f4041a1a748aa46dc887"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"eb558dea62cadb395b02bddf7adab04a53d6165e","unresolved":true,"context_lines":[{"line_number":178,"context_line":"        # TODO(lucasagomes): Remove the OVN_BRANCH and OVS_BRANCH config"},{"line_number":179,"context_line":"        # once https://review.opendev.org/c/openstack/devstack/+/887184"},{"line_number":180,"context_line":"        # is merged"},{"line_number":181,"context_line":"        OVN_BRANCH: \"branch-22.03\""},{"line_number":182,"context_line":"        OVS_BRANCH: \"branch-3.0\""},{"line_number":183,"context_line":"      devstack_local_conf:"},{"line_number":184,"context_line":"        test-config:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"20018435_72f7c33a","line":181,"in_reply_to":"4458a842_c50be162","updated":"2023-07-05 11:12:17.000000000","message":"sorry it should be OVS_BRANCH as that works with commits not OVN_BRANCH.","commit_id":"09e94103286c76576980f4041a1a748aa46dc887"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"c59e9a174042511a786d7a7b9d7d21e35c7f88e1","unresolved":true,"context_lines":[{"line_number":178,"context_line":"        # TODO(lucasagomes): Remove the OVN_BRANCH and OVS_BRANCH config"},{"line_number":179,"context_line":"        # once https://review.opendev.org/c/openstack/devstack/+/887184"},{"line_number":180,"context_line":"        # is merged"},{"line_number":181,"context_line":"        OVN_BRANCH: \"branch-22.03\""},{"line_number":182,"context_line":"        OVS_BRANCH: \"branch-3.0\""},{"line_number":183,"context_line":"      devstack_local_conf:"},{"line_number":184,"context_line":"        test-config:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"67d6fcbc_4baac66b","line":181,"in_reply_to":"4458a842_c50be162","updated":"2023-07-05 11:12:10.000000000","message":"what you mean ?","commit_id":"09e94103286c76576980f4041a1a748aa46dc887"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"fc950b2ad2f2a70711208ea6058d71ddcee01bf7","unresolved":true,"context_lines":[{"line_number":178,"context_line":"        # TODO(lucasagomes): Remove the OVN_BRANCH and OVS_BRANCH config"},{"line_number":179,"context_line":"        # once https://review.opendev.org/c/openstack/devstack/+/887184"},{"line_number":180,"context_line":"        # is merged"},{"line_number":181,"context_line":"        OVN_BRANCH: \"branch-22.03\""},{"line_number":182,"context_line":"        OVS_BRANCH: \"branch-3.0\""},{"line_number":183,"context_line":"      devstack_local_conf:"},{"line_number":184,"context_line":"        test-config:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"92ee76f8_005673aa","line":181,"in_reply_to":"67d6fcbc_4baac66b","updated":"2023-07-05 11:13:13.000000000","message":"The failure is because OVN_BRANCH and OVS_BRANCH options in the master template is pointing to a commit hash right ? So, we need to set both.","commit_id":"09e94103286c76576980f4041a1a748aa46dc887"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"ca9252d45653f44980a9a85918b71ed8519e866c","unresolved":true,"context_lines":[{"line_number":178,"context_line":"        # TODO(lucasagomes): Remove the OVN_BRANCH and OVS_BRANCH config"},{"line_number":179,"context_line":"        # once https://review.opendev.org/c/openstack/devstack/+/887184"},{"line_number":180,"context_line":"        # is merged"},{"line_number":181,"context_line":"        OVN_BRANCH: \"branch-22.03\""},{"line_number":182,"context_line":"        OVS_BRANCH: \"branch-3.0\""},{"line_number":183,"context_line":"      devstack_local_conf:"},{"line_number":184,"context_line":"        test-config:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"a3e8bd76_fc8cbfb9","line":181,"in_reply_to":"80dc3381_769129a8","updated":"2023-07-05 12:38:42.000000000","message":"I see, I thought they both used the same method to clone the repository","commit_id":"09e94103286c76576980f4041a1a748aa46dc887"}]}