)]}'
{"/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":"e953fd78c4cff278153f8c019c6d712474a1e89d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e3ed6b2e_1ea7640d","updated":"2022-09-16 09:25:29.000000000","message":"thanks","commit_id":"639257c77b381b831da99bdb7bc89aba16ce0412"}],"devstack/plugin.sh":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"e953fd78c4cff278153f8c019c6d712474a1e89d","unresolved":true,"context_lines":[{"line_number":118,"context_line":"                # tcpdump monitor on br-ex for ARP, reverse ARP and ICMP v4 / v6 packets"},{"line_number":119,"context_line":"                sudo ip link set dev $PUBLIC_BRIDGE up"},{"line_number":120,"context_line":"                if [[ \"$os_CODENAME\" \u003d\u003d \"jammy\" ]]; then"},{"line_number":121,"context_line":"                    TCPDUMP\u003d/usr/bin/tcpdump"},{"line_number":122,"context_line":"                else"},{"line_number":123,"context_line":"                    TCPDUMP\u003d/usr/sbin/tcpdump"},{"line_number":124,"context_line":"                fi"},{"line_number":125,"context_line":"                run_process br-ex-tcpdump \"$TCPDUMP -i $PUBLIC_BRIDGE arp or rarp or icmp or icmp6 -enlX\" \"$STACK_GROUP\" root"},{"line_number":126,"context_line":"            fi"}],"source_content_type":"text/x-sh","patch_set":1,"id":"5839959f_f0ee5245","line":123,"range":{"start_line":121,"start_character":0,"end_line":123,"end_character":45},"updated":"2022-09-16 09:25:29.000000000","message":"If I ever understand why these things must be changed....","commit_id":"639257c77b381b831da99bdb7bc89aba16ce0412"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"ac808f75803db3177cb3da626e8f0b0f588b7683","unresolved":false,"context_lines":[{"line_number":118,"context_line":"                # tcpdump monitor on br-ex for ARP, reverse ARP and ICMP v4 / v6 packets"},{"line_number":119,"context_line":"                sudo ip link set dev $PUBLIC_BRIDGE up"},{"line_number":120,"context_line":"                if [[ \"$os_CODENAME\" \u003d\u003d \"jammy\" ]]; then"},{"line_number":121,"context_line":"                    TCPDUMP\u003d/usr/bin/tcpdump"},{"line_number":122,"context_line":"                else"},{"line_number":123,"context_line":"                    TCPDUMP\u003d/usr/sbin/tcpdump"},{"line_number":124,"context_line":"                fi"},{"line_number":125,"context_line":"                run_process br-ex-tcpdump \"$TCPDUMP -i $PUBLIC_BRIDGE arp or rarp or icmp or icmp6 -enlX\" \"$STACK_GROUP\" root"},{"line_number":126,"context_line":"            fi"}],"source_content_type":"text/x-sh","patch_set":1,"id":"3455e2d4_5de29d55","line":123,"range":{"start_line":121,"start_character":0,"end_line":123,"end_character":45},"in_reply_to":"5839959f_f0ee5245","updated":"2022-09-16 09:30:34.000000000","message":"Change is the essence of life, hardcoding paths is what is unnatural. I\u0027ve been wondering whether it would be better to determine the path dynamically, but decided to stick to this minor change for now.\n\nSide note: First I thought that tcpdump might not be installed at all, maybe it should also be added to bindep?","commit_id":"639257c77b381b831da99bdb7bc89aba16ce0412"}]}