)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"f21d87d94b98851aaaa04eca5cf1e449b0fad740","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f503b5b0_3ed691af","updated":"2026-10-30 21:01:44.000000000","message":"Looks OK to me","commit_id":"a865282809c81e3c919ac4013556e5fe7db02daf"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"85f69e48550da1462ec4e409e07cda9539cb50d5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8645d4ff_13c42d96","updated":"2026-10-08 14:49:53.000000000","message":"_get_cinderclient_parameters already uses a few config options so this clearly makes sense.","commit_id":"a865282809c81e3c919ac4013556e5fe7db02daf"}],"nova/volume/cinder.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"60c25f8eec469495e8689bf352fc1ff3529c89d9","unresolved":true,"context_lines":[{"line_number":189,"context_line":"    else:"},{"line_number":190,"context_line":"        url \u003d session.get_endpoint(auth, **service_parameters)"},{"line_number":191,"context_line":""},{"line_number":192,"context_line":"    return auth, service_parameters, url"},{"line_number":193,"context_line":""},{"line_number":194,"context_line":""},{"line_number":195,"context_line":"def is_microversion_supported(context, microversion):"}],"source_content_type":"text/x-python","patch_set":1,"id":"76405b22_0d78197f","line":192,"updated":"2026-10-09 17:56:40.000000000","message":"nit it would be nice to add a small assert for endpoint_override contianing the url in a unit test of this funciton","commit_id":"a865282809c81e3c919ac4013556e5fe7db02daf"}]}