)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"7814b1b9596aeb3340ac7a2c2223913abe565c80","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Replace crudini with ConfigParser in set_service_setting"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Use configparser instead of crudini command to process INI"},{"line_number":10,"context_line":"configuration to avoid dependency on crudini binary in the pod."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Related-Bug: OSPRH-19617"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"6652fb7c_6359b737","line":10,"range":{"start_line":9,"start_character":0,"end_line":10,"end_character":63},"updated":"2025-12-23 14:02:49.000000000","message":"Hi @rxiao@redhat.com, thank you for the changes.\n\nThe TODO note handled might have been not elaborated enough, the purpose was to eliminate the dependency on crudini binary in pods by removing all occurrences in WNTP on crudini, otherwise such dependency on the binary would still exist.\n\nThe changes seem good to me, only need to be used as separate method(s) that could act on all remaining `crudini` occurrences used within python code (including the one that was already well handled).\n\nhttps://opendev.org/x/whitebox-neutron-tempest-plugin/search?q\u003dcrudini\u0026l\u003dPython","commit_id":"7e5b70422eb224c327aa9559eaa80f0fd9b389ff"},{"author":{"_account_id":37280,"name":"Renjing Xiao","display_name":"Renjing Xiao","email":"rxiao@redhat.com","username":"rxiao"},"change_message_id":"d77d03e3900658fd89294897a7dcf07d714c60f9","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Replace crudini with ConfigParser in set_service_setting"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Use configparser instead of crudini command to process INI"},{"line_number":10,"context_line":"configuration to avoid dependency on crudini binary in the pod."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Related-Bug: OSPRH-19617"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"0fa7fccd_5b0d5c88","line":10,"range":{"start_line":9,"start_character":0,"end_line":10,"end_character":63},"in_reply_to":"6652fb7c_6359b737","updated":"2026-01-06 14:01:54.000000000","message":"Thank you for pointing it out.\nUploaded https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/972281.","commit_id":"7e5b70422eb224c327aa9559eaa80f0fd9b389ff"}]}
