qa/rgw: fix user cleanup in s3tests task

the if condition was backwards, preventing non-keystone users from being
removed after the s3tests task runs

Fixes: https://tracker.ceph.com/issues/69741
Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 4a0ca73f53ad1ddcc22c17c80f153ddd5128649f)

diff --git a/qa/tasks/s3tests.py b/qa/tasks/s3tests.py
index 69ac3591c846ab36b1f8e727220cde0b9e195505..d7ac2b3dbbfef5dfd7f3657392f73d9bb20d00ff 100644 (file)
--- a/qa/tasks/s3tests.py
+++ b/qa/tasks/s3tests.py
@@ -213,7 +213,7 @@ def create_users(ctx, config, s3tests_conf):
         for client in config.keys():
             for user in users.values():
                 # don't need to delete keystone users
-                if not user in keystone_users:
+                if section in keystone_users:
                     continue
                 uid = '{user}.{client}'.format(user=user, client=client)
                 cluster_name, daemon_type, client_id = teuthology.split_role(client)