]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
qa/rgw: use 'with-sse-s3' override for s3tests 46232/head
authorCasey Bodley <cbodley@redhat.com>
Wed, 11 May 2022 18:49:49 +0000 (14:49 -0400)
committerCasey Bodley <cbodley@redhat.com>
Wed, 11 May 2022 18:49:51 +0000 (14:49 -0400)
don't rely on the ceph manager task to parse a config file. each rgw
could be using a different config. instead, revert to an s3tests
override called 'with-sse-s3'

this way, the only job that enables sse-s3, vault_transit.yaml, contains
both the 'rgw crypt sse s3' configurables, and the flag to enable the
associated test cases

Signed-off-by: Casey Bodley <cbodley@redhat.com>
qa/suites/rgw/crypt/2-kms/vault_transit.yaml
qa/tasks/s3tests.py

index 4e6d27025153749d9025dc7f24e3c36d6e41d8ea..d20bb52bcbfb3da805d4456876f22f9a4d158189 100644 (file)
@@ -13,6 +13,8 @@ overrides:
   rgw:
     client.0:
       use-vault-role: client.0
+  s3tests:
+    with-sse-s3: true
 
 tasks:
 - vault:
index e5238563f4e385e4d2563e81419fd53b9d71675b..d14eb93fe6c5406e46a4ec16d164ea15cd126966 100644 (file)
@@ -432,7 +432,7 @@ def run_tests(ctx, config):
         attrs = ["!fails_on_rgw", "!lifecycle_expiration", "!fails_strict_rfc2616","!test_of_sts","!webidentity_test"]
         if client_config.get('calling-format') != 'ordinary':
             attrs += ['!fails_with_subdomain']
-        if client_config.get('without-sse-s3'):
+        if not client_config.get('with-sse-s3'):
             attrs += ['!sse-s3']
        
         if 'extra_attrs' in client_config: