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

mimic: qa/suites/rgw/verify/tasks/cls_rgw: test cls_rgw #23197

Merged
merged 3 commits into from Oct 17, 2018

Conversation

Projects
None yet
5 participants
@pdvian
Copy link

pdvian commented Jul 24, 2018

@smithfarm smithfarm added this to the mimic milestone Jul 24, 2018

@smithfarm smithfarm requested a review from cbodley Aug 20, 2018

@yuriw

This comment has been minimized.

Copy link
Contributor

yuriw commented Aug 20, 2018

@cbodley

This comment has been minimized.

Copy link
Contributor

cbodley commented Aug 22, 2018

it looks like this one is causing a lot of test failures, because it depends on changes in #22249 that weren't backported to mimic. could you please pull in the commit from that pr?

@yuriw

This comment has been minimized.

Copy link
Contributor

yuriw commented Aug 22, 2018

pls tags when fixed

@smithfarm smithfarm added the DNM label Aug 23, 2018

@smithfarm smithfarm changed the title mimic: qa/suites/rgw/verify/tasks/cls_rgw: test cls_rgw [DNM] mimic: qa/suites/rgw/verify/tasks/cls_rgw: test cls_rgw Aug 23, 2018

@smithfarm

This comment has been minimized.

Copy link
Contributor

smithfarm commented Sep 7, 2018

@pdvian ping, could you please respond to #23197 (comment)

@pdvian

This comment has been minimized.

Copy link

pdvian commented Sep 12, 2018

@cbodley @smithfarm Let me pull #22249 changes.

@smithfarm

This comment has been minimized.

Copy link
Contributor

smithfarm commented Sep 13, 2018

@pdvian The changes from #22249 depend on an earlier commit. See #24070 (it hasn't been tested yet, but it looks sane).

@pdvian pdvian force-pushed the pdvian:wip-25024-mimic branch from 8a602cf to d010444 Sep 14, 2018

@pdvian

This comment has been minimized.

Copy link

pdvian commented Sep 14, 2018

@smithfarm Agree. @cbodley have pulled #22249 changes. Kindly review.

@smithfarm

This comment has been minimized.

Copy link
Contributor

smithfarm commented Sep 14, 2018

@pdvian Could you please model this PR after #24070 - i.e. three cherry-picked commits from master, and change ceph-master branch to ceph-mimic.

@pdvian

This comment has been minimized.

Copy link

pdvian commented Sep 18, 2018

@smithfarm sure. Let me do it.

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)

@pdvian pdvian force-pushed the pdvian:wip-25024-mimic branch from d010444 to 0f0cf71 Sep 19, 2018

cbodley added some commits Mar 16, 2018

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/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/0-install.yaml
      qa/suites/rgw/verify/tasks/rgw_s3tests.yaml
      qa/suites/rgw/verify/tasks/rgw_tests.yaml

@pdvian pdvian force-pushed the pdvian:wip-25024-mimic branch from 0f0cf71 to f5d0acf Sep 19, 2018

@pdvian

This comment has been minimized.

Copy link

pdvian commented Sep 19, 2018

@smithfarm done with changes. Kindly review. Shall we add extra commit log for changing force-branch to ceph-mimic ?

@smithfarm smithfarm removed the DNM label Oct 4, 2018

@smithfarm smithfarm changed the title [DNM] mimic: qa/suites/rgw/verify/tasks/cls_rgw: test cls_rgw mimic: qa/suites/rgw/verify/tasks/cls_rgw: test cls_rgw Oct 4, 2018

@smithfarm

This comment has been minimized.

Copy link
Contributor

smithfarm commented Oct 4, 2018

This looks good to me now. @cbodley Do you agree?

@cbodley

This comment has been minimized.

Copy link
Contributor

cbodley commented Oct 4, 2018

@smithfarm yep 👍

@yuriw

This comment has been minimized.

Copy link
Contributor

yuriw commented Oct 15, 2018

@yuriw yuriw merged commit 47939ee into ceph:mimic Oct 17, 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment