)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"2170c0d33474ee4deafde8f11f8d50bb7ae7fce3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"6753721a_1aa0f802","updated":"2022-03-23 12:52:07.000000000","message":"recheck","commit_id":"ab6b050af6840e7f0329c16a1d5ec780b8239cdd"}],"logscraper/logsender.py":[{"author":{"_account_id":9311,"name":"Tristan Cacqueray","email":"tdecacqu@redhat.com","username":"tristanC"},"change_message_id":"15c06e4ed44a1a26d0f6358939d1adfc22f2b19a","unresolved":true,"context_lines":[{"line_number":124,"context_line":"    try:"},{"line_number":125,"context_line":"        build_inventory \u003d read_yaml_file(\"%s/inventory.yaml\" % directory)"},{"line_number":126,"context_line":"        build_inventory[\u0027all\u0027][\u0027vars\u0027][\u0027zuul\u0027][\u0027hosts_id\u0027] \u003d get_hosts_id("},{"line_number":127,"context_line":"            build_inventory)"},{"line_number":128,"context_line":"        return build_inventory[\u0027all\u0027][\u0027vars\u0027][\u0027zuul\u0027]"},{"line_number":129,"context_line":"    except FileNotFoundError:"},{"line_number":130,"context_line":"        logging.warning(\"Can not find inventory.yaml in build \""}],"source_content_type":"text/x-python","patch_set":2,"id":"67e16d11_89ba1e7e","line":127,"updated":"2022-03-14 13:45:38.000000000","message":"That seems un-necessary. Perhaps `get_inventory_info` should return the full inventory, and then the `makeFields` function should takes care of calling `get_hosts_id()`, so that we avoid mutating the zuul vars object.","commit_id":"9339c8ad7e35703fff9146fe70bfe23cc88503ef"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"48598f1f63a4b516c4593023688ee8181d59a70b","unresolved":true,"context_lines":[{"line_number":124,"context_line":"    try:"},{"line_number":125,"context_line":"        build_inventory \u003d read_yaml_file(\"%s/inventory.yaml\" % directory)"},{"line_number":126,"context_line":"        build_inventory[\u0027all\u0027][\u0027vars\u0027][\u0027zuul\u0027][\u0027hosts_id\u0027] \u003d get_hosts_id("},{"line_number":127,"context_line":"            build_inventory)"},{"line_number":128,"context_line":"        return build_inventory[\u0027all\u0027][\u0027vars\u0027][\u0027zuul\u0027]"},{"line_number":129,"context_line":"    except FileNotFoundError:"},{"line_number":130,"context_line":"        logging.warning(\"Can not find inventory.yaml in build \""}],"source_content_type":"text/x-python","patch_set":2,"id":"6c6c5251_3e103672","line":127,"in_reply_to":"67e16d11_89ba1e7e","updated":"2022-03-15 07:54:04.000000000","message":"ok","commit_id":"9339c8ad7e35703fff9146fe70bfe23cc88503ef"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"308e2fd15e8fa8ac549879490d226880c6e86ed7","unresolved":false,"context_lines":[{"line_number":124,"context_line":"    try:"},{"line_number":125,"context_line":"        build_inventory \u003d read_yaml_file(\"%s/inventory.yaml\" % directory)"},{"line_number":126,"context_line":"        build_inventory[\u0027all\u0027][\u0027vars\u0027][\u0027zuul\u0027][\u0027hosts_id\u0027] \u003d get_hosts_id("},{"line_number":127,"context_line":"            build_inventory)"},{"line_number":128,"context_line":"        return build_inventory[\u0027all\u0027][\u0027vars\u0027][\u0027zuul\u0027]"},{"line_number":129,"context_line":"    except FileNotFoundError:"},{"line_number":130,"context_line":"        logging.warning(\"Can not find inventory.yaml in build \""}],"source_content_type":"text/x-python","patch_set":2,"id":"1ca4e813_9f85e60b","line":127,"in_reply_to":"6c6c5251_3e103672","updated":"2022-03-16 09:10:41.000000000","message":"Done","commit_id":"9339c8ad7e35703fff9146fe70bfe23cc88503ef"}]}