New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

luminous: tests: cls_rgw test is only run in rados suite: add it to rgw suite as well #24070

Merged
merged 4 commits into from Oct 1, 2018

Conversation

Projects
None yet
4 participants
@smithfarm
Contributor

smithfarm commented Sep 13, 2018

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Sep 13, 2018

Waiting for #24034 to be merged, then rebase.

@smithfarm smithfarm added the DNM label Sep 13, 2018

cbodley and others added some commits Mar 6, 2018

qa/rgw: combine swift, s3tests, ragweed into single verify task
these tests don't need to be split across the job matrix the same way
that we split features like frontend, ssl, objectstore, etc. by
combining them, we can still test the whole matrix of features, but with
only 1/3 of the total jobs

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit b497d9f)

Conflicts:
	qa/suites/rgw/verify/tasks/rgw_tests.yaml
	qa/suites/rgw/verify/tasks/rgw_swift.yaml
qa/rgw: reorganize verify tasks
split the verify suite tasks back into separate yaml files, using + to
combine them into a single job

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 64e20c3)

Conflicts:
	qa/suites/rgw/verify/tasks/0-install.yaml
        qa/suites/rgw/verify/tasks/ragweed.yaml
        qa/suites/rgw/verify/tasks/s3tests.yaml
qa/suites/rgw/verify/tasks/cls_rgw: test cls_rgw
Test this as part of the rgw suite (currently we only cover it in the
rados suite).

Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit a3c2b28)

@smithfarm smithfarm force-pushed the smithfarm:wip-25025-luminous branch from a2616bb to 256cad1 Sep 14, 2018

@smithfarm smithfarm added luminous-batch-1 and removed DNM labels Sep 14, 2018

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Sep 14, 2018

@cbodley This makes the contents of the qa/suites/rgw/verify/tasks/ directory in luminous resemble the same in master, except the branch names are changed from ceph-master to ceph-luminous.

Look right to you?

@yuriw

This comment has been minimized.

Contributor

yuriw commented Sep 17, 2018

@cbodley

a lot of Could not find task 'ragweed' failures in testing

client.0:
force-branch: ceph-luminous
rgw_server: client.0
stages: prepare,check

This comment has been minimized.

@cbodley

cbodley Sep 18, 2018

Contributor

oops - the ragweed task doesn't appear to exist in luminous. can you please remove this ragweed.yaml file?

qa/suites/rgw/verify: drop ragweed.yaml
There is no ragweed task in luminous.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
@yuriw

This comment has been minimized.

Contributor

yuriw commented Sep 28, 2018

@yuriw yuriw merged commit 9c66910 into ceph:luminous Oct 1, 2018

4 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details

@smithfarm smithfarm deleted the smithfarm:wip-25025-luminous branch Oct 1, 2018

@smithfarm smithfarm changed the title from luminous: cls_rgw test is only run in rados suite: add it to rgw suite as well to luminous: tests: cls_rgw test is only run in rados suite: add it to rgw suite as well Oct 26, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment