)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"49bfcbba9a5083568b7a846558f5d5cd13fcb45e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"efa7b6dc_71b85e9f","updated":"2022-06-22 08:44:02.000000000","message":"Thanks for the patch.\n\nSorry for the early review comments. I am still reviewing changes, I would like to know whether we are planning to modify the below function with the proposed changes, please refer to [1]. Or was the file left intentionally.\n\nFile: tacker/tests/functional/clients.py\nFunction: \n    def keystone_session(self):\n        return self.keystone.session\n        \nAs per my knowledge, this function is called in the \"BaseTackerTest\" class while registering heatclient [2].\n\n[1] https://opendev.org/openstack/tacker/src/branch/master/tacker/tests/functional/clients.py#L37\n[2] https://opendev.org/openstack/tacker/src/branch/master/tacker/tests/functional/base.py#L176","commit_id":"4324e3b706cccc9019822cd611cf6f58eb82c808"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"c5131564146c9554566fdafd1eb7795228c95717","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f93a6416_3eac041d","in_reply_to":"efa7b6dc_71b85e9f","updated":"2022-09-13 16:05:57.000000000","message":"Sorry for late reply.\nI changed tests [1] to use `tacker.common.clients` [2] instead and removed `tests.functional.clients`.\n\n[1] https://opendev.org/openstack/tacker/src/branch/master/tacker/tests/functional/base.py#L176\n[2] https://review.opendev.org/c/openstack/tacker/+/846966/6/tacker/common/clients.py\n[3] https://opendev.org/openstack/tacker/src/branch/master/tacker/tests/functional/clients.py#L37","commit_id":"4324e3b706cccc9019822cd611cf6f58eb82c808"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"de3c0ff5e25baf8c7aad12e56073a0e7f0f4fcbe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"7e36dbe9_eb01f150","updated":"2022-09-14 07:05:06.000000000","message":"Thanks for re-work, yes this approach looks better.","commit_id":"f41cee0c2cbf6f28f26fafc5bd8814c5e50a4e2d"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"01d03e33d4b7ae2413efe8b41a13a976fda5ffa5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"59ec027f_a5a2b164","updated":"2022-09-14 08:12:31.000000000","message":"Thanks for your patch.","commit_id":"f41cee0c2cbf6f28f26fafc5bd8814c5e50a4e2d"}]}