)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"859d38a55e53b6a89a36570646d8425d91501744","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4e6b930d_2dfbd62e","updated":"2022-10-21 09:34:42.000000000","message":"how did you encounter this? there are unit tests which (should) test the _get_keystone_config method - so i wonder why they don\u0027t fail","commit_id":"33d4f0696419da84c64a2d1fdcbbd18d48a1cd07"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"10afb2c0993851de61ce6bef1510ab758117bbeb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0344664b_58729982","in_reply_to":"4e6b930d_2dfbd62e","updated":"2022-10-21 11:05:27.000000000","message":"I guess we probably have an error in the unit tests. We expect failure when we read [identity] values from tempest.conf [1][2]. But reading [identity] values from tempest.conf is only deprecated, it is not removed [3]. \n\nWe can either remove support for loading of the [identity] values or fix the unit test. I will add a fix for the unit test here for now. \n\n[1] https://opendev.org/openinfra/refstack-client/src/commit/d320c6c60abc2cf2e78990952a9a747d1dbb545a/refstack_client/tests/unit/test_client.py#L201\n[2] https://opendev.org/openinfra/refstack-client/commit/b066bd68a8faad6a12e13f248507d146a9d066e4\n[3] https://opendev.org/openinfra/refstack-client/src/commit/d320c6c60abc2cf2e78990952a9a747d1dbb545a/refstack_client/refstack_client.py#L214","commit_id":"33d4f0696419da84c64a2d1fdcbbd18d48a1cd07"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"0f2465d58f0b6a4842b2e8015d283a9597c2d9d9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"695d44fc_fdba818e","updated":"2022-11-01 17:36:00.000000000","message":"makes sense, tenant_name is an old name and was replaced by project_name, it makes sense to ignore missing tenant_name and try to read project_name instead, thanks for fixing the unit test as well","commit_id":"340344526bff3dc2b298eb55f99ce5eb7bff1579"}]}