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: rados: move cephtool.yaml to new singleton/bluestore subsuite #14847

Merged
merged 1 commit into from Apr 29, 2017

Conversation

Projects
None yet
3 participants
@smithfarm
Contributor

smithfarm commented Apr 27, 2017

@smithfarm smithfarm requested review from liewegas and jdurgin Apr 27, 2017

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

@liewegas

This comment has been minimized.

Member

liewegas commented Apr 27, 2017

that works!

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Apr 27, 2017

teuthology-suite --priority 101 --suite rados --machine-type smithi --email ncutler@suse.com --ceph wip-19797 --filter="rados/singleton/{all/cephtool.yaml msgr-failures/few.yaml msgr/simple.yaml objectstore/filestore-btrfs.yaml rados.yaml}"

fail http://pulpito.ceph.com/smithfarm-2017-04-28_00:53:12-rados-wip-19797---basic-smithi/

@jdurgin

thanks! not sure how I missed this one

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Apr 28, 2017

Oddly, this patch doesn't make the error go away.

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Apr 28, 2017

This is a different problem than I thought. The rados/singleton suite runs over a matrix of objectstores: bluestore-comp.yaml bluestore.yaml filestore-btrfs.yaml filestore-xfs.yaml - in this case, the "osd objectstore: bluestore" override introduced in cephtool.yaml was clobbered by the "osd objectstore: filestore" override in filestore-btrfs.yaml.

Looks to me like the cephtool.yaml needs to be split out into a bluestore-only subsuite.

@smithfarm smithfarm changed the title from tests: cephtool: force bluestore to [DNM] tests: cephtool: force bluestore Apr 28, 2017

@smithfarm smithfarm changed the title from [DNM] tests: cephtool: force bluestore to tests: rados: move cephtool.yaml to new singleton/bluestore subsuite Apr 28, 2017

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Apr 28, 2017

teuthology-suite --priority 101 --suite rados --machine-type smithi --email ncutler@suse.com --ceph wip-19797 --filter="rados/singleton/bluestore"

waiting for build https://shaman.ceph.com/builds/ceph/wip-19797/e69a9763a00f78168bdaea0956e0d1ce98812950/

tests: rados: move cephtool.yaml to new singleton/bluestore subsuite
cephtool.yaml is bluestore-only, yet it was in singleton/ which runs against a
generalized objectstore matrix.

Fixes: http://tracker.ceph.com/issues/19797
Signed-off-by: Nathan Cutler <ncutler@suse.com>
@liewegas

This comment has been minimized.

Member

liewegas commented Apr 28, 2017

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Apr 28, 2017

Weird, the trusty notcmalloc build keeps failing. That must be a problem in master itself.

@liewegas liewegas merged commit 4c41537 into ceph:master Apr 29, 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

@smithfarm smithfarm deleted the smithfarm:wip-19797 branch Apr 29, 2017

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