)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"794e4b9f6cc17b7320f055faf431ad21dea688fc","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"MAC addresses in the iptables rules are always added by iptables_manager"},{"line_number":10,"context_line":"with uppercase. It was done like that in [1]."},{"line_number":11,"context_line":"For some reason, iptables on Centos 9 Stream (1.8.7) returns MAC"},{"line_number":12,"context_line":"addresses all in lowercase and difflib library treats such new and old"},{"line_number":13,"context_line":"rules as different ones so iptables manager always tries to remove and"},{"line_number":14,"context_line":"add antispoofing rules which have MAC addresses set."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"1bf6845a_c09dabf7","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":44},"updated":"2022-07-29 07:47:37.000000000","message":"nice","commit_id":"6c2d2744d5e841f431ae637624dd5954429f9607"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5da082a398de6b66ba39ddd11db392aa766fe285","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b69482dd_1c431aea","updated":"2022-07-28 15:18:32.000000000","message":"In another patch https://db185d1cb4ca2bd86029-362332867e0b7ca258ba1c1b46a070a7.ssl.cf1.rackcdn.com/843245/10/check/neutron-functional-with-uwsgi-fips/5956538/testr_results.html You can see that those tests which were impacted by that issue are green now.","commit_id":"6c2d2744d5e841f431ae637624dd5954429f9607"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"209b4aab4fc00f60a0cc534908b6347e303c1146","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"52d2624e_be309df7","updated":"2022-07-29 13:07:02.000000000","message":"recheck - all tests in functional job passed, POST_FAILURE wasn\u0027t related to that patch","commit_id":"6c2d2744d5e841f431ae637624dd5954429f9607"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"9a6c9423e44ccd6ba99b38822e6cc73b6b2fcb2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"670a2834_dad86d12","updated":"2022-07-28 15:17:28.000000000","message":"recheck - post failures not related to that patch","commit_id":"6c2d2744d5e841f431ae637624dd5954429f9607"}],"neutron/agent/linux/iptables_manager.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"534f0240928fed54eef418f258253f53d688ee68","unresolved":true,"context_lines":[{"line_number":863,"context_line":"            # skip ? because that\u0027s a guide string for intraline differences"},{"line_number":864,"context_line":"            continue"},{"line_number":865,"context_line":"        line_uppercase \u003d line.upper()[2:]"},{"line_number":866,"context_line":"        if (line_uppercase in old_rules_uppercase and"},{"line_number":867,"context_line":"                line_uppercase in new_rules_uppercase):"},{"line_number":868,"context_line":"            continue"},{"line_number":869,"context_line":""},{"line_number":870,"context_line":"        if line.startswith(\u0027-\u0027):  # line deleted"},{"line_number":871,"context_line":"            statements.append(\u0027-D %s %d\u0027 % (chain, old_index))"}],"source_content_type":"text/x-python","patch_set":1,"id":"c8d8544f_4123d8e6","line":868,"range":{"start_line":866,"start_character":8,"end_line":868,"end_character":20},"updated":"2022-06-03 13:38:05.000000000","message":"How is that possible, if that comes from the diff? Why do we need a second check?","commit_id":"f162e34e6de3c450c0125e93a30adc813c16e4df"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"98a75b356ba5154a2f545982c8a517545c478dd8","unresolved":true,"context_lines":[{"line_number":863,"context_line":"            # skip ? because that\u0027s a guide string for intraline differences"},{"line_number":864,"context_line":"            continue"},{"line_number":865,"context_line":"        line_uppercase \u003d line.upper()[2:]"},{"line_number":866,"context_line":"        if (line_uppercase in old_rules_uppercase and"},{"line_number":867,"context_line":"                line_uppercase in new_rules_uppercase):"},{"line_number":868,"context_line":"            continue"},{"line_number":869,"context_line":""},{"line_number":870,"context_line":"        if line.startswith(\u0027-\u0027):  # line deleted"},{"line_number":871,"context_line":"            statements.append(\u0027-D %s %d\u0027 % (chain, old_index))"}],"source_content_type":"text/x-python","patch_set":1,"id":"b480746e_f4cda4aa","line":868,"range":{"start_line":866,"start_character":8,"end_line":868,"end_character":20},"in_reply_to":"c8d8544f_4123d8e6","updated":"2022-06-03 13:51:30.000000000","message":"Understood (explained in a meeting). Slawek will push a comment to document it.","commit_id":"f162e34e6de3c450c0125e93a30adc813c16e4df"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"3d762cb222778eaec231ccb3cbd99a81fd97685a","unresolved":true,"context_lines":[{"line_number":856,"context_line":"    # in the right position"},{"line_number":857,"context_line":"    old_index \u003d 1"},{"line_number":858,"context_line":"    statements \u003d []"},{"line_number":859,"context_line":"    old_rules_uppercase \u003d [r.upper() for r in old_chain_rules]"},{"line_number":860,"context_line":"    new_rules_uppercase \u003d [r.upper() for r in new_chain_rules]"},{"line_number":861,"context_line":"    for line in difflib.ndiff(old_chain_rules, new_chain_rules):"},{"line_number":862,"context_line":"        if line.startswith(\u0027?\u0027):"}],"source_content_type":"text/x-python","patch_set":2,"id":"a6ea3796_89d2c006","line":859,"range":{"start_line":859,"start_character":26,"end_line":859,"end_character":27},"updated":"2022-07-08 14:59:51.000000000","message":"You can speed L866 up a lot by using sets here.\n\n  {r.upper() for r in old_chain_rules}","commit_id":"1216c8379cea4dc12f4099da860265e7fc571daf"}]}