)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"45591e94d958b2a9ada6d3aace463b8e6df80986","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Bandit: Remove bandit B604 from skip list"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Remove B604 (any_other_function_with_shell_equals_true),"},{"line_number":10,"context_line":"so use subprocess.Popen with sehll\u003dFalse, thus removing"},{"line_number":11,"context_line":"the shell parameter of common.utils.subprocess_popen method."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Depends-On: https://review.opendev.org/c/915991"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"d4aee375_2296c787","line":10,"range":{"start_line":10,"start_character":29,"end_line":10,"end_character":34},"updated":"2024-04-16 20:39:42.000000000","message":"nit: shell","commit_id":"2bbeac6fe578e15558bcb52032ca2ecaf086bc83"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"4ccdfce872c97757f2abe93dd56dbcf5e01cc8df","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Bandit: Remove bandit B604 from skip list"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Remove B604 (any_other_function_with_shell_equals_true),"},{"line_number":10,"context_line":"so use subprocess.Popen with the default shelll\u003dFalse,"},{"line_number":11,"context_line":"thus removing the shell parameter of common.utils.subprocess"},{"line_number":12,"context_line":"popen method."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9329e076_769c385d","line":10,"range":{"start_line":10,"start_character":41,"end_line":10,"end_character":47},"updated":"2024-04-22 13:29:42.000000000","message":"shelll","commit_id":"bacff278afbd779e799a94557d7d3d99ebad2b6b"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"4ccdfce872c97757f2abe93dd56dbcf5e01cc8df","unresolved":true,"context_lines":[{"line_number":9,"context_line":"Remove B604 (any_other_function_with_shell_equals_true),"},{"line_number":10,"context_line":"so use subprocess.Popen with the default shelll\u003dFalse,"},{"line_number":11,"context_line":"thus removing the shell parameter of common.utils.subprocess"},{"line_number":12,"context_line":"popen method."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Depends-On: https://review.opendev.org/c/915991"},{"line_number":15,"context_line":"Change-Id: I1857e982211c4de29b3e24ec2485ea9c17c98b10"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"c7233819_65bda400","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":5},"updated":"2024-04-22 13:29:42.000000000","message":"Popen (upper case)","commit_id":"bacff278afbd779e799a94557d7d3d99ebad2b6b"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"d2b496a481654cd56ade1cf14076d5270bcd584c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"64720081_6d53b176","updated":"2024-04-12 11:49:59.000000000","message":"recheck\nneutron-tempest-plugin-openvswitch-enforce-scope-old-defaults failed with ssh error","commit_id":"63468c79d1f106061407d2b4f3bbd64fb08dc740"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"a523c3b77e41a79523db73b4eff60f242f9976c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"accde883_b7b21b4b","updated":"2024-04-11 16:53:26.000000000","message":"recheck\nseems like a hit of https://bugs.launchpad.net/neutron/+bug/1961740","commit_id":"63468c79d1f106061407d2b4f3bbd64fb08dc740"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"45591e94d958b2a9ada6d3aace463b8e6df80986","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5f079a96_9af823cd","updated":"2024-04-16 20:39:42.000000000","message":"recheck unrelated test failure","commit_id":"2bbeac6fe578e15558bcb52032ca2ecaf086bc83"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"b4e5fbb33749319f57ca7601259014e0ddb68b7e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"58e1c267_a803c9c0","updated":"2024-04-18 17:16:56.000000000","message":"remove passing the argument, and fix commit message.","commit_id":"2bbeac6fe578e15558bcb52032ca2ecaf086bc83"}],"neutron/common/utils.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"3ab0b74988ee3af29c280ab27508b203925ee696","unresolved":true,"context_lines":[{"line_number":129,"context_line":"    signal.signal(signal.SIGPIPE, signal.SIG_DFL)"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"def subprocess_popen(args, stdin\u003dNone, stdout\u003dNone, stderr\u003dNone, env\u003dNone,"},{"line_number":133,"context_line":"                     preexec_fn\u003d_subprocess_setup, close_fds\u003dTrue):"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"    return subprocess.Popen(args, shell\u003dFalse, stdin\u003dstdin, stdout\u003dstdout,"},{"line_number":136,"context_line":"                            stderr\u003dstderr, preexec_fn\u003dpreexec_fn,"},{"line_number":137,"context_line":"                            close_fds\u003dclose_fds, env\u003denv)"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"def get_first_host_ip(net, ip_version):"}],"source_content_type":"text/x-python","patch_set":1,"id":"fbfa55c0_fc8173b0","line":137,"range":{"start_line":132,"start_character":0,"end_line":137,"end_character":57},"updated":"2024-04-11 08:40:25.000000000","message":"I can\u0027t find any usage of this method, so perhaps it an be dropped totally.\nLet\u0027s see if zuul can find any issue with this change, locally it was fine","commit_id":"63468c79d1f106061407d2b4f3bbd64fb08dc740"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"fe0175940e6a856fb0a5b33e7061ead7a56769a4","unresolved":true,"context_lines":[{"line_number":129,"context_line":"    signal.signal(signal.SIGPIPE, signal.SIG_DFL)"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"def subprocess_popen(args, stdin\u003dNone, stdout\u003dNone, stderr\u003dNone, env\u003dNone,"},{"line_number":133,"context_line":"                     preexec_fn\u003d_subprocess_setup, close_fds\u003dTrue):"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"    return subprocess.Popen(args, shell\u003dFalse, stdin\u003dstdin, stdout\u003dstdout,"},{"line_number":136,"context_line":"                            stderr\u003dstderr, preexec_fn\u003dpreexec_fn,"},{"line_number":137,"context_line":"                            close_fds\u003dclose_fds, env\u003denv)"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"def get_first_host_ip(net, ip_version):"}],"source_content_type":"text/x-python","patch_set":1,"id":"6169c4ef_4c6131d0","line":137,"range":{"start_line":132,"start_character":0,"end_line":137,"end_character":57},"in_reply_to":"31b94e54_41a6fcc8","updated":"2024-04-16 13:41:31.000000000","message":"thanks,  I added that as depends-on","commit_id":"63468c79d1f106061407d2b4f3bbd64fb08dc740"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"5f1ee4716ae973f0eb7e594c59f9a3716a82c940","unresolved":true,"context_lines":[{"line_number":129,"context_line":"    signal.signal(signal.SIGPIPE, signal.SIG_DFL)"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"def subprocess_popen(args, stdin\u003dNone, stdout\u003dNone, stderr\u003dNone, env\u003dNone,"},{"line_number":133,"context_line":"                     preexec_fn\u003d_subprocess_setup, close_fds\u003dTrue):"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"    return subprocess.Popen(args, shell\u003dFalse, stdin\u003dstdin, stdout\u003dstdout,"},{"line_number":136,"context_line":"                            stderr\u003dstderr, preexec_fn\u003dpreexec_fn,"},{"line_number":137,"context_line":"                            close_fds\u003dclose_fds, env\u003denv)"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"def get_first_host_ip(net, ip_version):"}],"source_content_type":"text/x-python","patch_set":1,"id":"31b94e54_41a6fcc8","line":137,"range":{"start_line":132,"start_character":0,"end_line":137,"end_character":57},"in_reply_to":"fbfa55c0_fc8173b0","updated":"2024-04-16 12:11:54.000000000","message":"used https://opendev.org/openstack/neutron-vpnaas/src/branch/master/neutron_vpnaas/services/vpn/common/netns_wrapper.py#L59 atleast  so need to fix other places where it\u0027s used.","commit_id":"63468c79d1f106061407d2b4f3bbd64fb08dc740"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"b4e5fbb33749319f57ca7601259014e0ddb68b7e","unresolved":true,"context_lines":[{"line_number":132,"context_line":"def subprocess_popen(args, stdin\u003dNone, stdout\u003dNone, stderr\u003dNone, env\u003dNone,"},{"line_number":133,"context_line":"                     preexec_fn\u003d_subprocess_setup, close_fds\u003dTrue):"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"    return subprocess.Popen(args, shell\u003dFalse, stdin\u003dstdin, stdout\u003dstdout,"},{"line_number":136,"context_line":"                            stderr\u003dstderr, preexec_fn\u003dpreexec_fn,"},{"line_number":137,"context_line":"                            close_fds\u003dclose_fds, env\u003denv)"},{"line_number":138,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"2eecbcce_b4224216","line":135,"updated":"2024-04-18 17:16:56.000000000","message":"shell\u003dFalse is default, you can remove it.","commit_id":"2bbeac6fe578e15558bcb52032ca2ecaf086bc83"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"e07e20af89af55eb6dcbd6ddecf136db0af0c9fb","unresolved":false,"context_lines":[{"line_number":132,"context_line":"def subprocess_popen(args, stdin\u003dNone, stdout\u003dNone, stderr\u003dNone, env\u003dNone,"},{"line_number":133,"context_line":"                     preexec_fn\u003d_subprocess_setup, close_fds\u003dTrue):"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"    return subprocess.Popen(args, shell\u003dFalse, stdin\u003dstdin, stdout\u003dstdout,"},{"line_number":136,"context_line":"                            stderr\u003dstderr, preexec_fn\u003dpreexec_fn,"},{"line_number":137,"context_line":"                            close_fds\u003dclose_fds, env\u003denv)"},{"line_number":138,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"49d8ffc2_362c4efd","line":135,"in_reply_to":"2eecbcce_b4224216","updated":"2024-04-22 12:41:00.000000000","message":"Done","commit_id":"2bbeac6fe578e15558bcb52032ca2ecaf086bc83"}]}
