)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"27f0998dad64c70a90bee22a41909f89260f487b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6e486ab1_09fbd98d","updated":"2021-10-12 12:49:44.000000000","message":"Only +1 in ovsdbapp stable, but looks good I think (just need to confirm my comment below)","commit_id":"49ab1305e6b5aa9ddc30268f517bdb00a2c580f4"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4bfad2017f30f0096b1988f5f0e043af2cd33b76","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ec1f2437_4a381cd7","updated":"2021-10-13 07:45:04.000000000","message":"Right. Slawek did the same in Neutron in [1] and I for n-ovn in [2]\n\n[1]https://review.opendev.org/c/openstack/neutron/+/812674\n[2]https://review.opendev.org/c/openstack/networking-ovn/+/813145","commit_id":"49ab1305e6b5aa9ddc30268f517bdb00a2c580f4"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"e89452b1a971cf55658dd9c7ff601ca30dd0904a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fca3359d_be8946a2","updated":"2021-10-13 06:51:57.000000000","message":"Thanks Brian, I like to have these kind of discussions in the review for documentary purpose :-)","commit_id":"49ab1305e6b5aa9ddc30268f517bdb00a2c580f4"}],"tools/setup-ovs.sh":[{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"27f0998dad64c70a90bee22a41909f89260f487b","unresolved":true,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash -xe"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"OVS_BRANCH\u003d${OVS_BRANCH:-master}"},{"line_number":4,"context_line":"OVN_BRANCH\u003d${OVN_BRANCH:-$OVS_BRANCH}"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"function use_new_ovn_repository {"},{"line_number":7,"context_line":"    # If OVN_BRANCH \u003e branch-2.12 return 0"}],"source_content_type":"text/x-sh","patch_set":1,"id":"9364afe5_4001d9f4","side":"PARENT","line":4,"range":{"start_line":4,"start_character":25,"end_line":4,"end_character":36},"updated":"2021-10-12 12:49:44.000000000","message":"Just to be sure, now OVN and OVS have different versioning, so there is no sense in having a more complicated logic like \"OVS_BRANCH if it set to something else than master, else main\"?","commit_id":"35c8eae020ef5300e08b7869e5da11dd0e2c500e"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"e89452b1a971cf55658dd9c7ff601ca30dd0904a","unresolved":true,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash -xe"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"OVS_BRANCH\u003d${OVS_BRANCH:-master}"},{"line_number":4,"context_line":"OVN_BRANCH\u003d${OVN_BRANCH:-$OVS_BRANCH}"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"function use_new_ovn_repository {"},{"line_number":7,"context_line":"    # If OVN_BRANCH \u003e branch-2.12 return 0"}],"source_content_type":"text/x-sh","patch_set":1,"id":"a722a590_127fafd5","side":"PARENT","line":4,"range":{"start_line":4,"start_character":25,"end_line":4,"end_character":36},"in_reply_to":"9364afe5_4001d9f4","updated":"2021-10-13 06:51:57.000000000","message":"yes it seems that ovn more and more independent of ovs. I think it\u0027s more transparent to have 2 independent variable instead of some logic which tries to formalize what we have now but without the input from ovs and ovn community.\nAnd it is only a cherry-pick 😊","commit_id":"35c8eae020ef5300e08b7869e5da11dd0e2c500e"}]}