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

qa/rgw: add cluster name to path when s3tests scans rgw log #14845

Merged
merged 1 commit into from May 1, 2017

Conversation

Projects
None yet
2 participants
@cbodley
Contributor

cbodley commented Apr 27, 2017

when support for multiple clusters was added in #12535, the cluster name was added to the radosgw log path. update s3tests task to match

this fixes some radosgw log is leaking encryption keys! failures

qa/rgw: add cluster name to path when s3tests scans rgw log
Signed-off-by: Casey Bodley <cbodley@redhat.com>
@cbodley

This comment has been minimized.

Contributor

cbodley commented Apr 28, 2017

testing branch is currently stalled after 57 passed, 0 failed, 6 dead. but here's a link to a s3test run that shows this check working correctly: http://qa-proxy.ceph.com/teuthology/cbodley-2017-04-27_23:02:56-rgw-wip-cbodley-testing---basic-mira/1076361/teuthology.log

2017-04-28T07:11:34.517 DEBUG:tasks.s3tests:Scanning radosgw logs for leaked encryption keys...
2017-04-28T07:11:34.517 INFO:teuthology.orchestra.run.mira121:Running: 'grep --binary-files=text pO3upElrwuEXSoFwCfnZPdSsmt/xWeFa0N9KgDijwVs= /var/log/ceph/rgw.ceph.client.0.log'
2017-04-28T07:11:34.719 INFO:tasks.s3tests:Cleaning up boto...
@dang

dang approved these changes Apr 28, 2017

@cbodley cbodley merged commit 0e30e3e into ceph:master May 1, 2017

3 checks passed

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

@cbodley cbodley deleted the cbodley:wip-rgw-qa-s3tests branch May 1, 2017

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