)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"4e33c0c0e243463bce967147cbcf77e0c1580e84","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ec315219_526e4bc1","updated":"2021-11-26 07:43:31.000000000","message":"LGTM","commit_id":"eeff797ffe7c7ce1864d1fbd0c0412bbf9844564"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"85d7f4ecc83f897129de1e3d033532adc2a55488","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9a7541e5_35e4eb37","updated":"2021-11-26 08:00:24.000000000","message":"Lets unblock Neutron\u0027s gate with this 😊","commit_id":"eeff797ffe7c7ce1864d1fbd0c0412bbf9844564"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"688e298230e11707ed7e0d3495eadf6dc588f781","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"39bebe5f_a3393562","updated":"2021-11-26 08:52:07.000000000","message":"Being tested in https://review.opendev.org/819264","commit_id":"b575af0cfe8aac71825b406394e9927a33ca51bd"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"a66f514ff224e0d8b625e89c8d283f9ca2a8c8b4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"965426f9_9ce9e644","updated":"2021-11-26 11:53:02.000000000","message":"Need to abandon/restore as a test failed in nova-ceph-multistore job https://zuul.opendev.org/t/openstack/buildset/8a0a07f7051b49629bc18c8bd522a968, also a nv fedora job timedout and is in retry mode.","commit_id":"b575af0cfe8aac71825b406394e9927a33ca51bd"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"543c64c16d01a94d0f55377acb62c264d8872245","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"64f00537_88eb39fe","updated":"2021-11-27 10:09:39.000000000","message":"recheck","commit_id":"b575af0cfe8aac71825b406394e9927a33ca51bd"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"79a43bf454dab4512144dd2535cec8e5c5bd9baf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"cef604fb_e3083a4e","updated":"2021-11-26 15:53:44.000000000","message":"recheck","commit_id":"b575af0cfe8aac71825b406394e9927a33ca51bd"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"0f3b6ce1fd92eaad9c48f1dfa0880652bafa8fda","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c51ce722_274fb068","updated":"2021-11-26 15:53:56.000000000","message":"recheck\n\nnova-ceph-multistore job failed due to OOM exceptions: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_d62/819402/2/gate/nova-ceph-multistore/d620d21/controller/logs/syslog.txt","commit_id":"b575af0cfe8aac71825b406394e9927a33ca51bd"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"d1cde7441fec8d6a9b0f992d3458cab44a76decd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2c488078_43bd595b","updated":"2021-11-26 12:02:56.000000000","message":"thanks! .. jobs in https://review.opendev.org/c/openstack/neutron/+/819264/ passed, +2","commit_id":"b575af0cfe8aac71825b406394e9927a33ca51bd"}],"lib/neutron_plugins/ovn_agent":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"4e33c0c0e243463bce967147cbcf77e0c1580e84","unresolved":true,"context_lines":[{"line_number":126,"context_line":"    # When using OVN from packages, the data dir for OVN DBs is"},{"line_number":127,"context_line":"    # /var/lib/ovn"},{"line_number":128,"context_line":"    OVN_DATADIR\u003d/var/lib/ovn"},{"line_number":129,"context_line":"    SUDO\u003d\"sudo\""},{"line_number":130,"context_line":"fi"},{"line_number":131,"context_line":"OVN_SHAREDIR\u003d$OVS_PREFIX/share/ovn"},{"line_number":132,"context_line":"OVN_SCRIPTDIR\u003d$OVN_SHAREDIR/scripts"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"44c11ca4_d819e424","line":129,"range":{"start_line":129,"start_character":4,"end_line":129,"end_character":8},"updated":"2021-11-26 07:43:31.000000000","message":"nit: we should probably use a more descriptive variable name, people may reuse $SUDO out of the OVN_BUILD_FROM_SOURCE True/False context","commit_id":"eeff797ffe7c7ce1864d1fbd0c0412bbf9844564"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"a9be1f18604e2905b399a4dcc8ff90985dab46f0","unresolved":false,"context_lines":[{"line_number":126,"context_line":"    # When using OVN from packages, the data dir for OVN DBs is"},{"line_number":127,"context_line":"    # /var/lib/ovn"},{"line_number":128,"context_line":"    OVN_DATADIR\u003d/var/lib/ovn"},{"line_number":129,"context_line":"    SUDO\u003d\"sudo\""},{"line_number":130,"context_line":"fi"},{"line_number":131,"context_line":"OVN_SHAREDIR\u003d$OVS_PREFIX/share/ovn"},{"line_number":132,"context_line":"OVN_SCRIPTDIR\u003d$OVN_SHAREDIR/scripts"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"0e724e35_c1899900","line":129,"range":{"start_line":129,"start_character":4,"end_line":129,"end_character":8},"in_reply_to":"44c11ca4_d819e424","updated":"2021-11-26 08:15:48.000000000","message":"Done avoided using SUDO var altogether.","commit_id":"eeff797ffe7c7ce1864d1fbd0c0412bbf9844564"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"f4c3b0c1f9465b043e846a9b9a443ba549e76200","unresolved":true,"context_lines":[{"line_number":570,"context_line":""},{"line_number":571,"context_line":"    _disable_libvirt_apparmor"},{"line_number":572,"context_line":""},{"line_number":573,"context_line":"    $SUDO mkdir -p $OVN_DATADIR"},{"line_number":574,"context_line":"    mkdir -p $OVS_DATADIR"},{"line_number":575,"context_line":""},{"line_number":576,"context_line":"    rm -f $OVS_DATADIR/*.db"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"84e99f6c_205e640c","line":573,"updated":"2021-11-26 07:52:00.000000000","message":"I\u0027m sorry to see that my original suggestion to make things easier in https://review.opendev.org/c/openstack/devstack/+/806858/comment/0b3cc9e2_38500e6d/ has led to so much trouble.\n\nSince actually only the mkdir seems to be causing issues, maybe just wrap this statement in another\n\nif [[ \"$OVN_BUILD_FROM_SOURCE\" \u003d\u003d \"True\" ]]; then\n\nand only have sudo in the \"else\" part?","commit_id":"eeff797ffe7c7ce1864d1fbd0c0412bbf9844564"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"a9be1f18604e2905b399a4dcc8ff90985dab46f0","unresolved":false,"context_lines":[{"line_number":570,"context_line":""},{"line_number":571,"context_line":"    _disable_libvirt_apparmor"},{"line_number":572,"context_line":""},{"line_number":573,"context_line":"    $SUDO mkdir -p $OVN_DATADIR"},{"line_number":574,"context_line":"    mkdir -p $OVS_DATADIR"},{"line_number":575,"context_line":""},{"line_number":576,"context_line":"    rm -f $OVS_DATADIR/*.db"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"dbee0a57_43ad6a5c","line":573,"in_reply_to":"758b54e4_cb96ce87","updated":"2021-11-26 08:15:48.000000000","message":"Done","commit_id":"eeff797ffe7c7ce1864d1fbd0c0412bbf9844564"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"0042e3dd7aa49c750e49d590b643c847e325c127","unresolved":true,"context_lines":[{"line_number":570,"context_line":""},{"line_number":571,"context_line":"    _disable_libvirt_apparmor"},{"line_number":572,"context_line":""},{"line_number":573,"context_line":"    $SUDO mkdir -p $OVN_DATADIR"},{"line_number":574,"context_line":"    mkdir -p $OVS_DATADIR"},{"line_number":575,"context_line":""},{"line_number":576,"context_line":"    rm -f $OVS_DATADIR/*.db"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"758b54e4_cb96ce87","line":573,"in_reply_to":"84e99f6c_205e640c","updated":"2021-11-26 08:11:17.000000000","message":"Ok will update like this and that should also address Gregory Thiemon suggestion to not use SUDO as generic name.","commit_id":"eeff797ffe7c7ce1864d1fbd0c0412bbf9844564"}]}