]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
qa/rgw: Run s3-tests from within the Ceph repo
authorAdam C. Emerson <aemerson@redhat.com>
Mon, 29 Sep 2025 21:11:09 +0000 (17:11 -0400)
committerAdam C. Emerson <aemerson@redhat.com>
Mon, 22 Jun 2026 23:33:47 +0000 (19:33 -0400)
Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
17 files changed:
qa/rgw/s3tests-branch.yaml [deleted file]
qa/suites/crimson-rados/rgw/lifecycle/s3tests-branch.yaml [deleted symlink]
qa/suites/crimson-rados/rgw/multifs/s3tests-branch.yaml [deleted symlink]
qa/suites/crimson-rados/rgw/sts/s3tests-branch.yaml [deleted symlink]
qa/suites/rgw/bucket-logging/s3tests-branch.yaml [deleted symlink]
qa/suites/rgw/cloud-transition/s3tests-branch.yaml [deleted symlink]
qa/suites/rgw/crypt/s3tests-branch.yaml [deleted symlink]
qa/suites/rgw/dbstore/s3tests-branch.yaml [deleted symlink]
qa/suites/rgw/lifecycle/s3tests-branch.yaml [deleted symlink]
qa/suites/rgw/multifs/s3tests-branch.yaml [deleted symlink]
qa/suites/rgw/s3control/s3tests-branch.yaml [deleted symlink]
qa/suites/rgw/sts/s3tests-branch.yaml [deleted symlink]
qa/suites/rgw/tempest/s3tests-branch.yaml [deleted symlink]
qa/suites/rgw/thrash/s3tests-branch.yaml [deleted symlink]
qa/suites/rgw/verify/s3tests-branch.yaml [deleted symlink]
qa/suites/rgw/website/s3tests-branch.yaml [deleted symlink]
qa/tasks/s3tests.py

diff --git a/qa/rgw/s3tests-branch.yaml b/qa/rgw/s3tests-branch.yaml
deleted file mode 100644 (file)
index 8710ce3..0000000
+++ /dev/null
@@ -1,4 +0,0 @@
-overrides:
-  s3tests:
-     force-branch: ceph-master
-     # git_remote: https://github.com/ceph/
diff --git a/qa/suites/crimson-rados/rgw/lifecycle/s3tests-branch.yaml b/qa/suites/crimson-rados/rgw/lifecycle/s3tests-branch.yaml
deleted file mode 120000 (symlink)
index bdcaca4..0000000
+++ /dev/null
@@ -1 +0,0 @@
-.qa/rgw/s3tests-branch.yaml
\ No newline at end of file
diff --git a/qa/suites/crimson-rados/rgw/multifs/s3tests-branch.yaml b/qa/suites/crimson-rados/rgw/multifs/s3tests-branch.yaml
deleted file mode 120000 (symlink)
index bdcaca4..0000000
+++ /dev/null
@@ -1 +0,0 @@
-.qa/rgw/s3tests-branch.yaml
\ No newline at end of file
diff --git a/qa/suites/crimson-rados/rgw/sts/s3tests-branch.yaml b/qa/suites/crimson-rados/rgw/sts/s3tests-branch.yaml
deleted file mode 120000 (symlink)
index bdcaca4..0000000
+++ /dev/null
@@ -1 +0,0 @@
-.qa/rgw/s3tests-branch.yaml
\ No newline at end of file
diff --git a/qa/suites/rgw/bucket-logging/s3tests-branch.yaml b/qa/suites/rgw/bucket-logging/s3tests-branch.yaml
deleted file mode 120000 (symlink)
index bdcaca4..0000000
+++ /dev/null
@@ -1 +0,0 @@
-.qa/rgw/s3tests-branch.yaml
\ No newline at end of file
diff --git a/qa/suites/rgw/cloud-transition/s3tests-branch.yaml b/qa/suites/rgw/cloud-transition/s3tests-branch.yaml
deleted file mode 120000 (symlink)
index bdcaca4..0000000
+++ /dev/null
@@ -1 +0,0 @@
-.qa/rgw/s3tests-branch.yaml
\ No newline at end of file
diff --git a/qa/suites/rgw/crypt/s3tests-branch.yaml b/qa/suites/rgw/crypt/s3tests-branch.yaml
deleted file mode 120000 (symlink)
index bdcaca4..0000000
+++ /dev/null
@@ -1 +0,0 @@
-.qa/rgw/s3tests-branch.yaml
\ No newline at end of file
diff --git a/qa/suites/rgw/dbstore/s3tests-branch.yaml b/qa/suites/rgw/dbstore/s3tests-branch.yaml
deleted file mode 120000 (symlink)
index bdcaca4..0000000
+++ /dev/null
@@ -1 +0,0 @@
-.qa/rgw/s3tests-branch.yaml
\ No newline at end of file
diff --git a/qa/suites/rgw/lifecycle/s3tests-branch.yaml b/qa/suites/rgw/lifecycle/s3tests-branch.yaml
deleted file mode 120000 (symlink)
index bdcaca4..0000000
+++ /dev/null
@@ -1 +0,0 @@
-.qa/rgw/s3tests-branch.yaml
\ No newline at end of file
diff --git a/qa/suites/rgw/multifs/s3tests-branch.yaml b/qa/suites/rgw/multifs/s3tests-branch.yaml
deleted file mode 120000 (symlink)
index bdcaca4..0000000
+++ /dev/null
@@ -1 +0,0 @@
-.qa/rgw/s3tests-branch.yaml
\ No newline at end of file
diff --git a/qa/suites/rgw/s3control/s3tests-branch.yaml b/qa/suites/rgw/s3control/s3tests-branch.yaml
deleted file mode 120000 (symlink)
index bdcaca4..0000000
+++ /dev/null
@@ -1 +0,0 @@
-.qa/rgw/s3tests-branch.yaml
\ No newline at end of file
diff --git a/qa/suites/rgw/sts/s3tests-branch.yaml b/qa/suites/rgw/sts/s3tests-branch.yaml
deleted file mode 120000 (symlink)
index bdcaca4..0000000
+++ /dev/null
@@ -1 +0,0 @@
-.qa/rgw/s3tests-branch.yaml
\ No newline at end of file
diff --git a/qa/suites/rgw/tempest/s3tests-branch.yaml b/qa/suites/rgw/tempest/s3tests-branch.yaml
deleted file mode 120000 (symlink)
index bdcaca4..0000000
+++ /dev/null
@@ -1 +0,0 @@
-.qa/rgw/s3tests-branch.yaml
\ No newline at end of file
diff --git a/qa/suites/rgw/thrash/s3tests-branch.yaml b/qa/suites/rgw/thrash/s3tests-branch.yaml
deleted file mode 120000 (symlink)
index bdcaca4..0000000
+++ /dev/null
@@ -1 +0,0 @@
-.qa/rgw/s3tests-branch.yaml
\ No newline at end of file
diff --git a/qa/suites/rgw/verify/s3tests-branch.yaml b/qa/suites/rgw/verify/s3tests-branch.yaml
deleted file mode 120000 (symlink)
index bdcaca4..0000000
+++ /dev/null
@@ -1 +0,0 @@
-.qa/rgw/s3tests-branch.yaml
\ No newline at end of file
diff --git a/qa/suites/rgw/website/s3tests-branch.yaml b/qa/suites/rgw/website/s3tests-branch.yaml
deleted file mode 120000 (symlink)
index bdcaca4..0000000
+++ /dev/null
@@ -1 +0,0 @@
-.qa/rgw/s3tests-branch.yaml
\ No newline at end of file
index 3a9db1d543ca8f06b7cf9284a7cccc3f110b4949..45e36f2bc227b224dfcb07386eae12555d9ba34b 100644 (file)
@@ -32,31 +32,29 @@ def download(ctx, config):
     assert isinstance(config, dict)
     log.info('Downloading s3-tests...')
     testdir = teuthology.get_testdir(ctx)
+    s3tests_branch = ctx.config.get('suite_branch') or ctx.config.get('branch')
+    s3tests_repo = ctx.config.get('suite_repo') or teuth_config.get_ceph_qa_suite_git_url()
+    s3tests_sha1 = ctx.config.get('suite_sha1')
+    if not s3tests_branch:
+        raise ValueError(
+            "Could not determine what branch to use for ceph suite.")
     for (client, client_config) in config.items():
-        s3tests_branch = client_config.get('force-branch', None)
-        if not s3tests_branch:
-            raise ValueError(
-                "Could not determine what branch to use for s3-tests. Please add 'force-branch: {s3-tests branch name}' to the .yaml config for this s3tests task.")
-
         log.info("Using branch '%s' for s3tests", s3tests_branch)
-        sha1 = client_config.get('sha1')
-        git_remote = client_config.get('git_remote', teuth_config.ceph_git_base_url)
         ctx.cluster.only(client).run(
             args=[
-                'git', 'clone',
-                '-b', s3tests_branch,
-                git_remote + 's3-tests.git',
-                '{tdir}/s3-tests-{client}'.format(tdir=testdir, client=client),
+                'git', 'clone', '-b', s3tests_branch,
+                s3tests_repo, '{tdir}/s3-tests-{client}'.format(tdir=testdir, client=client),
                 ],
             )
-        if sha1 is not None:
+        if s3tests_sha1 is not None:
             ctx.cluster.only(client).run(
                 args=[
                     'cd', '{tdir}/s3-tests-{client}'.format(tdir=testdir, client=client),
                     run.Raw('&&'),
-                    'git', 'reset', '--hard', sha1,
+                    'git', 'reset', '--hard', s3tests_sha1,
                     ],
                 )
+
         if client_config.get('boto3_extensions'):
             ctx.cluster.only(client).run(
                     args=['mkdir',
@@ -464,7 +462,7 @@ def run_tests(ctx, config):
         client_config = client_config or {}
         (remote,) = ctx.cluster.only(client).remotes.keys()
         args = [
-            'cd', '{tdir}/s3-tests-{client}'.format(tdir=testdir, client=client), run.Raw('&&'),
+            'cd', '{tdir}/s3-tests-{client}/src/test/rgw/s3-tests'.format(tdir=testdir, client=client), run.Raw('&&'),
             'S3TEST_CONF={tdir}/archive/s3-tests.{client}.conf'.format(tdir=testdir, client=client),
             'BOTO_CONFIG={tdir}/boto-{client}.cfg'.format(tdir=testdir, client=client)
             ]