)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36697,"name":"Yihong Jin","display_name":"Yihong Jin","email":"yihongjin404@gmail.com","username":"yihongjin"},"change_message_id":"299ebab135531b89d1231dcac14239c75a5357c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"29e1ba6e_bd547b43","updated":"2024-01-14 07:14:59.000000000","message":"Hi Tim, \n\nI think I found a quick fix to the bug you reported for container-updater account suppression feature. Before writing them to the file, we could encode the account names in a way that avoids newline and space characters. I choosed URL Encoding as it is provided natively by urllib. Also it widely supports languages other than English.\n\nI haven\u0027t figured out how to write tests for my change as it is trival. Thank you in advance for your time. \n\nBest,\nYihong Jin","commit_id":"c551a16db38e52d10e235b1acc7a668c19c822f3"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"e72651ad7f5f82302ce85689c0ded09ccf932ccc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"78050f4f_225f888d","updated":"2024-01-16 17:51:54.000000000","message":"quote seems like a good plan; it seems like these account suppression errors only apply in run_forever mode.\n\nThis change would look better with some tests; I wonder if you would consider squashing in something like these:\n\n905748: sq: some tests for account suppression | https://review.opendev.org/c/openstack/swift/+/905748","commit_id":"c551a16db38e52d10e235b1acc7a668c19c822f3"},{"author":{"_account_id":36697,"name":"Yihong Jin","display_name":"Yihong Jin","email":"yihongjin404@gmail.com","username":"yihongjin"},"change_message_id":"84dd2e1c5c88211c1eb1ce1237e4a22a9642752f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6fe51d24_9021a929","in_reply_to":"371c7d66_57e045e3","updated":"2024-01-24 07:13:05.000000000","message":"Done","commit_id":"c551a16db38e52d10e235b1acc7a668c19c822f3"},{"author":{"_account_id":36697,"name":"Yihong Jin","display_name":"Yihong Jin","email":"yihongjin404@gmail.com","username":"yihongjin"},"change_message_id":"f660bd68022510ec900b1c97136930b8e1456c05","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"371c7d66_57e045e3","in_reply_to":"78050f4f_225f888d","updated":"2024-01-24 07:12:34.000000000","message":"Sorry for the delay, I didn\u0027t receive email notifications from OpenDev (thought it is on by default just like Github). I will add the change 905748 and finish this asap.","commit_id":"c551a16db38e52d10e235b1acc7a668c19c822f3"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"fb7b59db8ff0952016044aa9145ccbec62342f65","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"0b06ed0b_73a20ed3","updated":"2024-01-24 16:54:07.000000000","message":"looks like the py2 issue in my crappy test is still blocking, sorry about that\n\nLet me know if you need some help to get it figured out!","commit_id":"07b591ac86d2d952b411210347084b4dab5f2e5e"}]}