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

tests: rgw.py: put client roles in a separate list #15913

Merged
merged 1 commit into from Jun 28, 2017

Conversation

Projects
None yet
4 participants
@smithfarm
Contributor

smithfarm commented Jun 26, 2017

@smithfarm smithfarm requested a review from cbodley Jun 26, 2017

@cbodley

This comment has been minimized.

Contributor

cbodley commented Jun 26, 2017

@smithfarm can we also fix the yaml that included a default_idle_timeout override? i thought we got them all in #15420

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Jun 26, 2017

@cbodley Yes, you got them all. It was in our downstream fork of teuthology.

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Jun 26, 2017

But if any override "creeps in" for any reason, the test breaks with the same cryptic error message.

@liewegas liewegas added the needs-qa label Jun 27, 2017

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Jun 27, 2017

Running a random test that exercises rgw.py

teuthology-suite -k distro --machine-type smithi --priority 101 --email ncutler@suse.com --ceph-repo https://github.com/ceph/ceph.git --ceph master --suite-repo https://github.com/smithfarm/ceph.git --suite-branch wip-20417 --suite rgw --filter="multifs/{clusters/fixed-2.yaml frontend/civetweb.yaml objectstore/bluestore-comp.yaml overrides.yaml rgw_pool_type/ec-cache.yaml tasks/rgw_bucket_quota.yaml}"

pass http://pulpito.front.sepia.ceph.com:80/smithfarm-2017-06-27_11:30:32-rgw-master-distro-basic-smithi/

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Jun 27, 2017

@cbodley Noticed that this change renders the config option unused in two of the functions, added a commit to clean that up.

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Jun 27, 2017

tests: rgw.py: put client roles in a separate list
Fixes: http://tracker.ceph.com/issues/20417
Signed-off-by: Nathan Cutler <ncutler@suse.com>
@yuriw

This comment has been minimized.

@yuriw yuriw merged commit ea87f42 into ceph:master Jun 28, 2017

4 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
default Build finished.
Details
make check make check succeeded
Details

@smithfarm smithfarm deleted the smithfarm:wip-20417 branch Jul 18, 2018

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