)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"aba24ca4dd1293474a1fb7c7ebf193195f41c691","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Emma Foley \u003cefoley@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2026-04-02 12:50:17 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[WIP] Replace inline mocks with FakeKeystoneClient"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The keystoneclient.v3.client.Client was mocked with bare mock.MagicMock()"},{"line_number":10,"context_line":"in discovery, polling, and gnocchi tests, with return values set inline"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"afdf8414_967623cb","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":2},"updated":"2026-04-09 19:25:52.000000000","message":"Most of this change should be rolled into the parent review, IMHO","commit_id":"05704c6d9c2b6491dce2ccce04858aeccf61a3ff"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"aba24ca4dd1293474a1fb7c7ebf193195f41c691","unresolved":true,"context_lines":[{"line_number":15,"context_line":"so tests can assert on calls (domains.list, projects.list) while still"},{"line_number":16,"context_line":"exercising real manager logic rather than configured return values."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Also fixes test_manager.py to patch keystoneclient.v3.client.Client"},{"line_number":19,"context_line":"instead of the removed v2_0 client, and corrects test_when_keystone_fail"},{"line_number":20,"context_line":"to reset the cached keystone state before simulating an unavailable"},{"line_number":21,"context_line":"keystone, matching the actual code path."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Assisted-By: Claude Sonnet 4.6 \u003cnoreply@anthropic.com\u003e"},{"line_number":24,"context_line":"Change-Id: I7be5c139e47ef2049a0bc1010d904af2cbfbaeae"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"8acab8bf_219c7d43","line":21,"range":{"start_line":18,"start_character":0,"end_line":21,"end_character":40},"updated":"2026-04-09 19:25:52.000000000","message":"I think this should be addressed separately to the FakeKeystoneClient change. So I will try to reduce this review to cover this part of the change, and see if it makes sense on its own.","commit_id":"05704c6d9c2b6491dce2ccce04858aeccf61a3ff"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"03af396679617fa0c9a28743939558dce874b927","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0968e238_c3ba4f3c","updated":"2026-04-13 07:50:53.000000000","message":"It looks good to me once the rebase on master is done.","commit_id":"f9239b7e2fc2affdb2b1180c6a4391999756f804"}]}