)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"9569b6ab3c4fd5c435b837b8d8f11590722850c9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b7283388_c7e18e00","updated":"2023-12-07 06:32:28.000000000","message":"Not helped https://1d532a9cf3978a3a6196-ad0bab2d2c85bd17ed58eecedfc78a9d.ssl.cf5.rackcdn.com/902774/2/check/neutron-tempest-plugin-designate-scenario/a01af31/job-output.txt","commit_id":"4d6702b5699a57c3d26ff72a84ec8bb259135d0a"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"cf2bd5080a6eb028cf6b144dfb7a5450202cf2e9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b2c3a9e0_d69f5d29","updated":"2023-12-06 16:22:18.000000000","message":"recheck test_rebuild_server_in_stop_state","commit_id":"4d6702b5699a57c3d26ff72a84ec8bb259135d0a"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"0aa86c81737a8e7bc9d443c912697dbdb42e456d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"08e939b2_1fe641c7","updated":"2023-12-07 15:36:43.000000000","message":"I don\u0027t have anything against that patch, but IIUC failures which we have, it is failing for us in the neutron-tempest-plugin devstack plugin, in https://github.com/openstack/neutron-tempest-plugin/blob/eb109725f08627c7749a97cec8d93a1893147cee/devstack/customize_image.sh#L33","commit_id":"8dbebcfad2f72a7d5cb47fb1baee716e48af2f07"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"9a6e49f91e6a871acd77ff78ac015716d05a63bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c0543ca8_0bf622e7","updated":"2023-12-07 16:55:54.000000000","message":"recheck test_rebuild_volume_backed_server","commit_id":"8dbebcfad2f72a7d5cb47fb1baee716e48af2f07"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"43cb0ba6e5dba578c193ed1289e61049b32756b3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"70247c22_dbe4b654","updated":"2023-12-08 08:01:44.000000000","message":"seems this is a duplicate of https://review.opendev.org/c/openstack/devstack/+/901699 , I also like doing the retry in a loop better","commit_id":"8dbebcfad2f72a7d5cb47fb1baee716e48af2f07"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"8436add8582f88b2665638d84be513232aede60f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"33186af6_20bad058","in_reply_to":"08e939b2_1fe641c7","updated":"2023-12-07 15:47:40.000000000","message":"yeap that is wrapper to the function here, tested in neutron job and it now handles those network issues https://zuul.opendev.org/t/openstack/build/7b640a62b8994a4bb5ebe1314f841501/log/job-output.txt#17528-17560","commit_id":"8dbebcfad2f72a7d5cb47fb1baee716e48af2f07"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"fc4d9d5b393c44f4589eb843bc462fc417bec91b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"111c42f2_b673763d","in_reply_to":"70247c22_dbe4b654","updated":"2023-12-08 10:26:34.000000000","message":"yes right retry loop would be better here, will review the other patch, but with quick look it will not be handling the current failures in neutron jobs. Will check/comment there, and will abandon this patch.","commit_id":"8dbebcfad2f72a7d5cb47fb1baee716e48af2f07"}],"functions":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"dba3138c2c58308a0e97d019e622ea565e4dc61c","unresolved":true,"context_lines":[{"line_number":139,"context_line":"    if [[ $image_url !\u003d file* ]]; then"},{"line_number":140,"context_line":"        # Downloads the image (uec ami+akistyle), then extracts it."},{"line_number":141,"context_line":"        if [[ ! -f $FILES/$image_fname || \"$(stat -c \"%s\" $FILES/$image_fname)\" \u003d \"0\" ]]; then"},{"line_number":142,"context_line":"            wget --progress\u003ddot:giga -w 3 -t 3 -c $image_url -O $FILES/$image_fname"},{"line_number":143,"context_line":"            if [[ $? -ne 0 ]]; then"},{"line_number":144,"context_line":"                echo \"Not found: $image_url\""},{"line_number":145,"context_line":"                return"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"912262c7_15585e4c","line":142,"range":{"start_line":142,"start_character":38,"end_line":142,"end_character":39},"updated":"2023-12-06 14:16:09.000000000","message":"nit: perhaps --waitretry is better to wait only for failed attempts, but here it is not interesting","commit_id":"4d6702b5699a57c3d26ff72a84ec8bb259135d0a"}]}