)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"0b99344d638e85586dc4a9cbf94cfca6ba0ed4ba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"63486e8a_91d63833","updated":"2024-05-07 16:28:04.000000000","message":"Thanks Takashi; please see comment inline","commit_id":"73423825d8d0e0bab15ce191caefb95e66d0c4a7"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"32b64e46d030d4437726e83a2aa0c945fb57f5b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"dd6a3c76_448ec974","updated":"2024-05-07 15:11:13.000000000","message":"recheck this change only touches qnap/gluster drivers which are not used in CI.","commit_id":"73423825d8d0e0bab15ce191caefb95e66d0c4a7"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"27b21bc9a2ba19ca44358c6bf5e3142f6cb30f48","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"daa04736_997191ff","updated":"2024-05-10 18:42:22.000000000","message":"recheck\n\nsporadic failure in the DHSS\u003dTrue Dummy driver job; it\u0027s being addressed here: https://review.opendev.org/c/openstack/manila-tempest-plugin/+/918702; lets see if we can get a +1\n\nThanks Takashi","commit_id":"e047b428a5e37b08eff1acdc06c6cf65c2d20ada"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"fd8a2ff7bfa2c20354054e8145a05d6b5bfc987b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0aca5185_43362c77","updated":"2024-05-11 00:55:51.000000000","message":"thanks for your change! LGTM.","commit_id":"e047b428a5e37b08eff1acdc06c6cf65c2d20ada"}],"manila/share/drivers/glusterfs/common.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"0b99344d638e85586dc4a9cbf94cfca6ba0ed4ba","unresolved":true,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import re"},{"line_number":20,"context_line":"import xml.etree.ElementTree as etree"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from oslo_config import cfg"},{"line_number":23,"context_line":"from oslo_log import log"}],"source_content_type":"text/x-python","patch_set":1,"id":"2f0fcee9_7fabffdf","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":37},"updated":"2024-05-07 16:28:04.000000000","message":"we missed these imports; thank you for finding and fixing them.. recently, a few our interns started adding bandit tests to this repo, and they began using this import instead:\n\n```\nfrom defusedxml import ElementTree as ET\n``` \n\nhttps://review.opendev.org/c/openstack/manila/+/914909","commit_id":"73423825d8d0e0bab15ce191caefb95e66d0c4a7"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"27b21bc9a2ba19ca44358c6bf5e3142f6cb30f48","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import re"},{"line_number":20,"context_line":"import xml.etree.ElementTree as etree"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from oslo_config import cfg"},{"line_number":23,"context_line":"from oslo_log import log"}],"source_content_type":"text/x-python","patch_set":1,"id":"53d015c0_d5b356e0","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":37},"in_reply_to":"04c69ff8_52ba8674","updated":"2024-05-10 18:42:22.000000000","message":"No concerns; thanks for fixing it","commit_id":"73423825d8d0e0bab15ce191caefb95e66d0c4a7"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"b88f416eccb9080cdb73ce1334fca30dc97edae6","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import re"},{"line_number":20,"context_line":"import xml.etree.ElementTree as etree"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from oslo_config import cfg"},{"line_number":23,"context_line":"from oslo_log import log"}],"source_content_type":"text/x-python","patch_set":1,"id":"04c69ff8_52ba8674","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":37},"in_reply_to":"2f0fcee9_7fabffdf","updated":"2024-05-07 16:43:42.000000000","message":"I\u0027ve rebased this change on that one. I still prefer longer name (etree to ET) but lmk if you have any concern with using it consistently.","commit_id":"73423825d8d0e0bab15ce191caefb95e66d0c4a7"}]}