)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36227,"name":"Katarina Strenkova","email":"kstrenko@redhat.com","username":"kstrenko"},"change_message_id":"c2235c805c0505b652e046ee6c389938a3364280","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"431cc424_30957a81","updated":"2024-06-19 09:33:19.000000000","message":"For now, this patch shows the thought behind the solution. The current user doesn\u0027t list other user\u0027s keypairs, just their own for some reason. It will take a more in depth look into the issue to resolve this problem.","commit_id":"cd96bf40557bfdf71d1c614db1b12e288a1a1674"},{"author":{"_account_id":36227,"name":"Katarina Strenkova","email":"kstrenko@redhat.com","username":"kstrenko"},"change_message_id":"196f02c9bd7442bb2b4a998574345b7a005e36ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"219bd885_f00f536a","updated":"2024-07-09 09:36:34.000000000","message":"I have changed the logic of when the keypairs are deleted as @mkopec@redhat.com has suggested. Now keypairs are deleted before each user is deleted. \n\nThe same issue remains, though. The user, even when he is supposed to be admin, can\u0027t list other users keypairs. I really can\u0027t seem to grasp why this issue persists, if anyone has some ideas, please do share.","commit_id":"08819657eff6ec2779b435fb8ad2d7a834814f04"}],"tempest/cmd/cleanup_service.py":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"d2866eef4c6fdacbd2be0e155902a597cf8cde99","unresolved":true,"context_lines":[{"line_number":1063,"context_line":"        for user in users:"},{"line_number":1064,"context_line":"            LOG.debug(\"Deleting keypairs for user with id %s\", user[\u0027id\u0027])"},{"line_number":1065,"context_line":"            self.keypair_svc.set_uid(user[\u0027id\u0027])"},{"line_number":1066,"context_line":"            self.keypair_svc.run()"},{"line_number":1067,"context_line":""},{"line_number":1068,"context_line":"            try:"},{"line_number":1069,"context_line":"                LOG.debug(\"Deleting user with id %s\", user[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"50f9c815_11df82a4","line":1066,"range":{"start_line":1066,"start_character":29,"end_line":1066,"end_character":32},"updated":"2024-07-10 14:32:26.000000000","message":"list?","commit_id":"08819657eff6ec2779b435fb8ad2d7a834814f04"},{"author":{"_account_id":36227,"name":"Katarina Strenkova","email":"kstrenko@redhat.com","username":"kstrenko"},"change_message_id":"6706402abfb5f21fa03ee35d751d717c177a5ba9","unresolved":true,"context_lines":[{"line_number":1063,"context_line":"        for user in users:"},{"line_number":1064,"context_line":"            LOG.debug(\"Deleting keypairs for user with id %s\", user[\u0027id\u0027])"},{"line_number":1065,"context_line":"            self.keypair_svc.set_uid(user[\u0027id\u0027])"},{"line_number":1066,"context_line":"            self.keypair_svc.run()"},{"line_number":1067,"context_line":""},{"line_number":1068,"context_line":"            try:"},{"line_number":1069,"context_line":"                LOG.debug(\"Deleting user with id %s\", user[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"8e8818d0_7bbbb66c","line":1066,"range":{"start_line":1066,"start_character":29,"end_line":1066,"end_character":32},"in_reply_to":"50f9c815_11df82a4","updated":"2024-07-10 14:52:22.000000000","message":"If it was just list, then the keypairs wouldn\u0027t get deleted. I am not entirely sure, but when calling run() it should contain list and also delete. \n\nI tried to delete one specific admin keypair using tempest cleanup --resource-list and added this key into resource_list.json by hand. It took some commenting, because by default we can\u0027t delete admin user, but I was able to delete this keypair using the code I proposed. So I think the structure is okay. \n\nMain issue remains: User lists his own keypairs and not keypairs of other users.","commit_id":"08819657eff6ec2779b435fb8ad2d7a834814f04"}]}