)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f8614f619f1ea0810cf9ae1be61f68bf6fcdfbfd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6c8d0e51_8757c855","updated":"2026-05-13 12:12:24.000000000","message":"Nice work. A few requests:\n\n* Can we add docs for these new commands to `docs/cli/command-objects` and also add a release note?\n* I\u0027d much prefer if we could use SDK rather than cinderclient for this (to avoid needing to rewrite it later: we\u0027re trying to replace all use of cinderclient here). I won\u0027t block on this but if you could do this, it\u0027d save some student/me from doing that work in the future.\n* I\u0027d appreciate a review from another cinder core, if you can rope someone in, just to sanity check the general behavior.","commit_id":"dba1c69ee8b93f2b6564e3c20812d2d43b95ab27"}],"openstackclient/tests/unit/volume/v3/test_block_storage_pool.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f8614f619f1ea0810cf9ae1be61f68bf6fcdfbfd","unresolved":true,"context_lines":[{"line_number":213,"context_line":"            \u0027--disable-reason\u0027,"},{"line_number":214,"context_line":"            \u0027reason\u0027,"},{"line_number":215,"context_line":"            \u0027host1@backend1#pool1\u0027,"},{"line_number":216,"context_line":"        ]"},{"line_number":217,"context_line":"        # --enable and --disable-reason are not mutually exclusive at parser"},{"line_number":218,"context_line":"        # level, but the command logic should reject this"},{"line_number":219,"context_line":"        # Actually --enable and --disable are mutually exclusive"},{"line_number":220,"context_line":"        # This test verifies that --disable-reason without --disable raises"},{"line_number":221,"context_line":"        # We need to set disabled\u003dFalse (from --enable) and have a reason"},{"line_number":222,"context_line":"        verifylist \u003d ["},{"line_number":223,"context_line":"            (\u0027pool\u0027, \u0027host1@backend1#pool1\u0027),"},{"line_number":224,"context_line":"            (\u0027disabled\u0027, False),"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a533657_894c864f","line":221,"range":{"start_line":216,"start_character":9,"end_line":221,"end_character":73},"updated":"2026-05-13 12:12:24.000000000","message":"nit: this reads a bit oddly, like a train of thought that contradicts itself 😅 Did you mean to include it?","commit_id":"dba1c69ee8b93f2b6564e3c20812d2d43b95ab27"}]}