)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"6580c7e96e5ee2d164713f3fa98255131b4931ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"22f65f9f_3400735e","updated":"2026-06-01 12:31:14.000000000","message":"I don\u0027t like how the test_discovery.py still reaches the cinderclient through the _client even though we have the abstraction, one of the reason being, that the tests wouldn\u0027t work once the cinderclient switches to openstacksdk. But I see the tests are getting changed during the switch to openstacksdk in https://review.opendev.org/c/openstack/ceilometer/+/986951/7 anyway and this implementation basically keeps the current tests, so I\u0027m fine with this temporarily. +2","commit_id":"7fe39243c4e57ce1910270f09402d27fc4cbafe4"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"93c5658270232d3b462146de65f058042abb2d80","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"75190fd9_1f22c3e6","in_reply_to":"22f65f9f_3400735e","updated":"2026-06-02 15:55:51.000000000","message":"To do those tests correctly, I would need a fixture like I have in BaseTestCase.setup_connection, but I felt that it wasn\u0027t worth adding it in since I was planning on immediately replacing it with the FakeConnection","commit_id":"7fe39243c4e57ce1910270f09402d27fc4cbafe4"}]}