)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"9c04e56ea3804e07bbee198ff2a4f75270be39e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"442ba7d2_d79a3ca6","updated":"2026-03-24 15:56:05.000000000","message":"LGTM!","commit_id":"2102492e29b028be18fcf017dcad20224bb47534"}],"swift/container/sync.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"9c04e56ea3804e07bbee198ff2a4f75270be39e2","unresolved":true,"context_lines":[{"line_number":209,"context_line":"            self.logger.warning("},{"line_number":210,"context_line":"                \u0027Configuration option internal_client_conf_path not \u0027"},{"line_number":211,"context_line":"                \u0027defined. Using default configuration, See \u0027"},{"line_number":212,"context_line":"                \u0027internal-client.conf-sample for options\u0027)"},{"line_number":213,"context_line":"            internal_client_conf \u003d ConfigString(ic_conf_body)"},{"line_number":214,"context_line":"        else:"},{"line_number":215,"context_line":"            internal_client_conf \u003d internal_client_conf_path"}],"source_content_type":"text/x-python","patch_set":2,"id":"983ca3a2_b862afe5","line":212,"updated":"2026-03-24 15:56:05.000000000","message":"I guess maintaining the deprecated config is a better drive-by than adding \"stop supporting the decade old upgrade path\" 😊","commit_id":"2102492e29b028be18fcf017dcad20224bb47534"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"01775821049426ee0574c99544ed285bfe0cb215","unresolved":true,"context_lines":[{"line_number":209,"context_line":"            self.logger.warning("},{"line_number":210,"context_line":"                \u0027Configuration option internal_client_conf_path not \u0027"},{"line_number":211,"context_line":"                \u0027defined. Using default configuration, See \u0027"},{"line_number":212,"context_line":"                \u0027internal-client.conf-sample for options\u0027)"},{"line_number":213,"context_line":"            internal_client_conf \u003d ConfigString(ic_conf_body)"},{"line_number":214,"context_line":"        else:"},{"line_number":215,"context_line":"            internal_client_conf \u003d internal_client_conf_path"}],"source_content_type":"text/x-python","patch_set":2,"id":"a043d975_bb0cc511","line":212,"in_reply_to":"983ca3a2_b862afe5","updated":"2026-04-24 18:53:03.000000000","message":"https://review.opendev.org/c/openstack/swift/+/986163 will at least start telling ops that continuing to rely on the hardcoded config will be an error in some future release. *Maybe* there\u0027s some hope we could finally actually remove it?","commit_id":"2102492e29b028be18fcf017dcad20224bb47534"}]}
