)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"c965bd9f778c23bc3edb9f04a6de01208bfd6b2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e0781f8d_8b5d11f0","updated":"2026-10-12 16:38:58.000000000","message":"adding -1 to highlight my comments.","commit_id":"0f7270aec76496b99f867c8c53446fb7b73d106e"}],"ceilometer/cinder_client.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8f8bf3733524acdccb4fb0011d9e3de45f82c612","unresolved":true,"context_lines":[{"line_number":31,"context_line":"        # NOTE(mnederlof): We set 3.64 (the maximum for Wallaby) because:"},{"line_number":32,"context_line":"        # we need at least 3.41 to get user_id on snapshots."},{"line_number":33,"context_line":"        # we need at least 3.56 for user_id and project_id on backups."},{"line_number":34,"context_line":"        # we need at least 3.63 for volume_type_id on volumes."},{"line_number":35,"context_line":"        # NOTE(fivetime): pass the service type the way the designate and"},{"line_number":36,"context_line":"        # manila clients do. With oslo_conf the SDK only enables a service"},{"line_number":37,"context_line":"        # whose options are registered in a group named after the project"},{"line_number":38,"context_line":"        # ([cinder]), which ceilometer does not have, so block-storage was"},{"line_number":39,"context_line":"        # always disabled and every cinder pollster failed."},{"line_number":40,"context_line":"        self._conn \u003d connection.Connection("},{"line_number":41,"context_line":"            block_storage_api_version\u003d\u00273.64\u0027,"},{"line_number":42,"context_line":"            session\u003dkeystone_client.get_session(conf),"}],"source_content_type":"text/x-python","patch_set":1,"id":"c75de8e5_7a3b5f5a","line":39,"range":{"start_line":34,"start_character":62,"end_line":39,"end_character":59},"updated":"2026-10-11 18:51:42.000000000","message":"Could you remove these ? This may not be quite clear from the code with the change, and recording these in commit message would be enough.","commit_id":"0f7270aec76496b99f867c8c53446fb7b73d106e"},{"author":{"_account_id":39426,"name":"fivetime","display_name":"fivetime","email":"td@fivetime.ltd","username":"fivetime"},"change_message_id":"bb16f5a03930ca86ea4a397954213ce4c1918d6c","unresolved":false,"context_lines":[{"line_number":36,"context_line":"        # manila clients do. With oslo_conf the SDK only enables a service"},{"line_number":37,"context_line":"        # whose options are registered in a group named after the project"},{"line_number":38,"context_line":"        # ([cinder]), which ceilometer does not have, so block-storage was"},{"line_number":39,"context_line":"        # always disabled and every cinder pollster failed."},{"line_number":40,"context_line":"        self._conn \u003d connection.Connection("},{"line_number":41,"context_line":"            block_storage_api_version\u003d\u00273.64\u0027,"},{"line_number":42,"context_line":"            session\u003dkeystone_client.get_session(conf),"}],"source_content_type":"text/x-python","patch_set":1,"id":"d1255bba_6863a8d6","line":39,"in_reply_to":"c75de8e5_7a3b5f5a","updated":"2026-10-20 07:56:49.000000000","message":"Done in patch set 2: the comment is gone; the commit message keeps the explanation.","commit_id":"0f7270aec76496b99f867c8c53446fb7b73d106e"}],"releasenotes/notes/cinder-sdk-service-enabled-7c2e4b1f9a3d5e60.yaml":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8f8bf3733524acdccb4fb0011d9e3de45f82c612","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8dc04a21_b08820a7","line":11,"range":{"start_line":7,"start_character":14,"end_line":11,"end_character":1},"updated":"2026-10-11 18:51:42.000000000","message":"Primary audiences for the release notes are users or operators and they do not really care about internal mechanism of the problem. So please remove this part.\n\nAlso do you mind reporting a bug so that we can track this bug fix properly ? (I think we should backport this).","commit_id":"0f7270aec76496b99f867c8c53446fb7b73d106e"},{"author":{"_account_id":39426,"name":"fivetime","display_name":"fivetime","email":"td@fivetime.ltd","username":"fivetime"},"change_message_id":"bb16f5a03930ca86ea4a397954213ce4c1918d6c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f4ac5947_258a11ba","line":11,"in_reply_to":"8dc04a21_b08820a7","updated":"2026-10-20 07:56:49.000000000","message":"Done in patch set 2: the note now only says what failed and that Cinder resources are polled again. Filed https://bugs.launchpad.net/ceilometer/+bug/2171478 and added Closes-Bug. The regression came with the move to openstacksdk (50230faf), first released in 27.0.0, so stable/2026.2 is the branch to backport to.","commit_id":"0f7270aec76496b99f867c8c53446fb7b73d106e"}]}