)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f445a224831f15bef60d9e057503020936d346ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"341c8f07_c1427778","updated":"2022-08-11 17:40:36.000000000","message":"I am ok with the change but one comment to fix it.","commit_id":"fc4bd1f6f01736252c3799d9b489fe9ea94a5794"},{"author":{"_account_id":8104,"name":"June Yi","display_name":"gochist","email":"june.yi@samsung.com","username":"juneyi"},"change_message_id":"5de648305da7f346540fa3894d6f997e24dd3ca1","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ab7daf95_17ce38d1","updated":"2022-08-12 00:13:33.000000000","message":"Thanks for your review! I have a question.","commit_id":"fc4bd1f6f01736252c3799d9b489fe9ea94a5794"},{"author":{"_account_id":8104,"name":"June Yi","display_name":"gochist","email":"june.yi@samsung.com","username":"juneyi"},"change_message_id":"e823185b7d3ac391ca6c7843c9a22f1a1feb24d5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2126b0b3_7d516820","in_reply_to":"ab7daf95_17ce38d1","updated":"2022-08-25 11:04:31.000000000","message":"Done","commit_id":"fc4bd1f6f01736252c3799d9b489fe9ea94a5794"},{"author":{"_account_id":8104,"name":"June Yi","display_name":"gochist","email":"june.yi@samsung.com","username":"juneyi"},"change_message_id":"e823185b7d3ac391ca6c7843c9a22f1a1feb24d5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"38be8c9c_49dc6ee9","updated":"2022-08-25 11:04:31.000000000","message":"I addressed all the review comments.","commit_id":"b9b6d6b862ce69a875c152ad30da8f4717c75272"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"ce3ffebc4aff47583ac2bc32a017e93d5f35df92","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ae45110c_87617006","updated":"2022-08-26 19:53:39.000000000","message":"looks good to me. I can see the failure which you are trying to describe/fix here in https://review.opendev.org/c/openstack/ceilometer/+/853668 too.\n\nlet\u0027s backport this to stable branches also.","commit_id":"b9b6d6b862ce69a875c152ad30da8f4717c75272"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"faa1b2e8c7e86bab0489670319e3f24640d2eba2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bc54913b_e0d0d97c","updated":"2022-08-30 17:47:01.000000000","message":"recheck\n","commit_id":"b9b6d6b862ce69a875c152ad30da8f4717c75272"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"44a92f380f520be6249851f14b1b671d25b1c4d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a31db819_8a8a4ab8","updated":"2022-08-30 14:51:42.000000000","message":"this is needed by ceilometer gate to unblock https://review.opendev.org/c/openstack/ceilometer/+/853668","commit_id":"b9b6d6b862ce69a875c152ad30da8f4717c75272"}],"lib/tempest":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f445a224831f15bef60d9e057503020936d346ef","unresolved":true,"context_lines":[{"line_number":695,"context_line":"    local tmp_cfg_file"},{"line_number":696,"context_line":"    tmp_cfg_file\u003d$(mktemp)"},{"line_number":697,"context_line":"    cd $TEMPEST_DIR"},{"line_number":698,"context_line":"    if [[ \"$OFFLINE\" !\u003d \"True\" ]]; then"},{"line_number":699,"context_line":"        tox -revenv-tempest --notest"},{"line_number":700,"context_line":"    fi"},{"line_number":701,"context_line":""},{"line_number":702,"context_line":"    local tmp_u_c_m"},{"line_number":703,"context_line":"    tmp_u_c_m\u003d$(mktemp -t tempest_u_c_m.XXXXXXXXXX)"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"21878fa0_f9ca5957","side":"PARENT","line":700,"range":{"start_line":698,"start_character":0,"end_line":700,"end_character":6},"updated":"2022-08-11 17:40:36.000000000","message":"here- let\u0027s move this part after the L704","commit_id":"02f8c16c9b43814d2958bad9a37b1949658081f1"},{"author":{"_account_id":8104,"name":"June Yi","display_name":"gochist","email":"june.yi@samsung.com","username":"juneyi"},"change_message_id":"e823185b7d3ac391ca6c7843c9a22f1a1feb24d5","unresolved":false,"context_lines":[{"line_number":695,"context_line":"    local tmp_cfg_file"},{"line_number":696,"context_line":"    tmp_cfg_file\u003d$(mktemp)"},{"line_number":697,"context_line":"    cd $TEMPEST_DIR"},{"line_number":698,"context_line":"    if [[ \"$OFFLINE\" !\u003d \"True\" ]]; then"},{"line_number":699,"context_line":"        tox -revenv-tempest --notest"},{"line_number":700,"context_line":"    fi"},{"line_number":701,"context_line":""},{"line_number":702,"context_line":"    local tmp_u_c_m"},{"line_number":703,"context_line":"    tmp_u_c_m\u003d$(mktemp -t tempest_u_c_m.XXXXXXXXXX)"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"ed87c574_3ad2beb7","side":"PARENT","line":700,"range":{"start_line":698,"start_character":0,"end_line":700,"end_character":6},"in_reply_to":"21878fa0_f9ca5957","updated":"2022-08-25 11:04:31.000000000","message":"Done","commit_id":"02f8c16c9b43814d2958bad9a37b1949658081f1"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f445a224831f15bef60d9e057503020936d346ef","unresolved":true,"context_lines":[{"line_number":700,"context_line":"    tmp_u_c_m\u003d$(mktemp -t tempest_u_c_m.XXXXXXXXXX)"},{"line_number":701,"context_line":"    set_tempest_venv_constraints $tmp_u_c_m"},{"line_number":702,"context_line":"    if [[ \"$OFFLINE\" \u003d\u003d \"True\" ]]; then"},{"line_number":703,"context_line":"        tox -evenv-tempest -- pip install -c $tmp_u_c_m -r requirements.txt"},{"line_number":704,"context_line":"    else"},{"line_number":705,"context_line":"        tox -revenv-tempest --notest"},{"line_number":706,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"4ba976c7_58aac23c","line":703,"range":{"start_line":703,"start_character":0,"end_line":703,"end_character":75},"updated":"2022-08-11 17:40:36.000000000","message":"we need to do this online mode too. What we can do is Move the -revenv things after the L701 line.","commit_id":"fc4bd1f6f01736252c3799d9b489fe9ea94a5794"},{"author":{"_account_id":8104,"name":"June Yi","display_name":"gochist","email":"june.yi@samsung.com","username":"juneyi"},"change_message_id":"5de648305da7f346540fa3894d6f997e24dd3ca1","unresolved":true,"context_lines":[{"line_number":700,"context_line":"    tmp_u_c_m\u003d$(mktemp -t tempest_u_c_m.XXXXXXXXXX)"},{"line_number":701,"context_line":"    set_tempest_venv_constraints $tmp_u_c_m"},{"line_number":702,"context_line":"    if [[ \"$OFFLINE\" \u003d\u003d \"True\" ]]; then"},{"line_number":703,"context_line":"        tox -evenv-tempest -- pip install -c $tmp_u_c_m -r requirements.txt"},{"line_number":704,"context_line":"    else"},{"line_number":705,"context_line":"        tox -revenv-tempest --notest"},{"line_number":706,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"6279d9b2_9079da61","line":703,"range":{"start_line":703,"start_character":0,"end_line":703,"end_character":75},"in_reply_to":"4ba976c7_58aac23c","updated":"2022-08-12 00:13:33.000000000","message":"It is understood as saying that both L705 and L703 must be executed in online mode.\n\nBut in my humble opinion, the L705 performed after the L701 includes the same behavior that the L703 does. If this is correct, wouldn\u0027t it be ok to do L705 without L703 in online mode?","commit_id":"fc4bd1f6f01736252c3799d9b489fe9ea94a5794"},{"author":{"_account_id":8104,"name":"June Yi","display_name":"gochist","email":"june.yi@samsung.com","username":"juneyi"},"change_message_id":"e823185b7d3ac391ca6c7843c9a22f1a1feb24d5","unresolved":false,"context_lines":[{"line_number":700,"context_line":"    tmp_u_c_m\u003d$(mktemp -t tempest_u_c_m.XXXXXXXXXX)"},{"line_number":701,"context_line":"    set_tempest_venv_constraints $tmp_u_c_m"},{"line_number":702,"context_line":"    if [[ \"$OFFLINE\" \u003d\u003d \"True\" ]]; then"},{"line_number":703,"context_line":"        tox -evenv-tempest -- pip install -c $tmp_u_c_m -r requirements.txt"},{"line_number":704,"context_line":"    else"},{"line_number":705,"context_line":"        tox -revenv-tempest --notest"},{"line_number":706,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"80413dd0_b1e5aafd","line":703,"range":{"start_line":703,"start_character":0,"end_line":703,"end_character":75},"in_reply_to":"6279d9b2_9079da61","updated":"2022-08-25 11:04:31.000000000","message":"Done","commit_id":"fc4bd1f6f01736252c3799d9b489fe9ea94a5794"}]}