)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"dbc53f5c98a6b664ff68b1561ceb81941a3276cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d346fe09_d80f4184","updated":"2022-08-29 08:01:20.000000000","message":"I suppose the depends on now can be deleted as it was other patch that fixed that","commit_id":"204290d64717c1270f0c8fea761650adb444ab4f"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"1a45035022739e3815067effe4ec99d4c821f9ea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"af198494_4a97ba90","updated":"2022-08-29 08:01:27.000000000","message":"recheck","commit_id":"204290d64717c1270f0c8fea761650adb444ab4f"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"3ad5d3616fee968ec59d1ebd7751ee8cc750c1ea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c9e3b888_94a89995","updated":"2022-08-29 15:24:40.000000000","message":"Perhaps we should have this in sync as now it seems random:\nhttps://codesearch.opendev.org/?q\u003dOVS_BRANCH\u0026i\u003dnope\u0026literal\u003dnope\u0026files\u003d\u0026excludeFiles\u003d\u0026repos\u003dopenstack/neutron,openstack/neutron-tempest-plugin","commit_id":"2a7fbc1e979813ee61527081f148afdd62015a57"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"a2f676a01ae361acf6c40d722f49479f0b1ad8be","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b0dd3490_36417fe6","in_reply_to":"c9e3b888_94a89995","updated":"2022-08-30 07:06:13.000000000","message":"Unfortunately, when the BUILD_FROM_SOURCES flag is set, to compile ovn/ovs from the repo code, certain versions of OVN_BRANCH require a specific version of OVS_BRANCH for dependencies/fixes, hence the mix of versions of that search for jobs on -master, -release, -stables.","commit_id":"2a7fbc1e979813ee61527081f148afdd62015a57"}],"zuul.d/base.yaml":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"3e6d3ff035ffeed8100e9b458b52a062ecdedf10","unresolved":true,"context_lines":[{"line_number":51,"context_line":"      Q_BUILD_OVS_FROM_GIT: True"},{"line_number":52,"context_line":"      INSTALL_OVN: True"},{"line_number":53,"context_line":"      OVN_BRANCH: main"},{"line_number":54,"context_line":"      OVS_BRANCH: master"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"- job:"},{"line_number":57,"context_line":"    name: ovn-octavia-provider-tempest-base"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"fe316daf_3ae46a3c","line":54,"range":{"start_line":54,"start_character":18,"end_line":54,"end_character":24},"updated":"2022-08-30 07:14:47.000000000","message":"shouldn\u0027t it be set to 6f24c2bc769afde0a390ce344de1a7d9c592e5a6 instead of master(it keeps on changing and can break again) as pointed by ovs submodule in ovn repo: ovs @ 6f24c2b?\nAlso commit message says to set fe171e4f109f001f07b867756a261d898f0d2cfc but here it\u0027s set to master.\n\n[ovn]$ git submodule status\n-6f24c2bc769afde0a390ce344de1a7d9c592e5a6 ovs","commit_id":"2a7fbc1e979813ee61527081f148afdd62015a57"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"23c411d0c50fa59994f027ff484826c5276a97e7","unresolved":false,"context_lines":[{"line_number":51,"context_line":"      Q_BUILD_OVS_FROM_GIT: True"},{"line_number":52,"context_line":"      INSTALL_OVN: True"},{"line_number":53,"context_line":"      OVN_BRANCH: main"},{"line_number":54,"context_line":"      OVS_BRANCH: master"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"- job:"},{"line_number":57,"context_line":"    name: ovn-octavia-provider-tempest-base"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3eff4250_2d40eb0e","line":54,"range":{"start_line":54,"start_character":18,"end_line":54,"end_character":24},"in_reply_to":"bcf27572_7471fcab","updated":"2022-08-30 07:54:22.000000000","message":"LGTM, Thanks","commit_id":"2a7fbc1e979813ee61527081f148afdd62015a57"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"5ac4b3d4c9b8a2f79272ccb423da040aeaa07d90","unresolved":false,"context_lines":[{"line_number":51,"context_line":"      Q_BUILD_OVS_FROM_GIT: True"},{"line_number":52,"context_line":"      INSTALL_OVN: True"},{"line_number":53,"context_line":"      OVN_BRANCH: main"},{"line_number":54,"context_line":"      OVS_BRANCH: master"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"- job:"},{"line_number":57,"context_line":"    name: ovn-octavia-provider-tempest-base"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"bcf27572_7471fcab","line":54,"range":{"start_line":54,"start_character":18,"end_line":54,"end_character":24},"in_reply_to":"fe316daf_3ae46a3c","updated":"2022-08-30 07:24:10.000000000","message":"Right! I forgot to come back to the initial patchset according to commit msg. But  it is courious that commit [1] makes reference to [2], but [3] also reference the same tag, and indeed looks like references to [2], [3] or master now works fine, but lets keep the tradition and put the git submodule status output.\n\n[1] https://github.com/ovn-org/ovn/commit/9f25b2cedf2c9a9d6959b26911fc3adcc39a9a89\n[2]https://github.com/openvswitch/ovs/commit/fe171e4f109f001f07b867756a261d898f0d2cfc\n[3] https://github.com/openvswitch/ovs/commit/6f24c2bc769afde0a390ce344de1a7d9c592e5a6\n\nthx Yatin!","commit_id":"2a7fbc1e979813ee61527081f148afdd62015a57"}]}
