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/suites: Reduce fs combination tests for smoke, use bluestore #14854

Merged
merged 2 commits into from May 9, 2017

Conversation

Projects
None yet
2 participants
@vasukulkarni
Member

vasukulkarni commented Apr 27, 2017

in testing ..

Signed-off-by: Vasu Kulkarni vasu@redhat.com

@vasukulkarni vasukulkarni added the tests label Apr 27, 2017

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

@liewegas

This comment has been minimized.

Member

liewegas commented Apr 27, 2017

see the part where objectstore/ facet is removed, but we also need a symlink to bluestore.yaml to incorporate that?

alternatively, we can use filestore-xfs.yaml for 1node, bluestre.yaml for basic, and filestore-xfs for the systemd facet?

@vasukulkarni

This comment has been minimized.

Member

vasukulkarni commented Apr 27, 2017

now shows 28 jobs with mostly bluestore, systemd and ceph-deploy still need xfs, although i think ceph-deploy is now ready to test bluestore in master now but needs minor changes

http://pulpito.ceph.com/vasu-2017-04-27_22:28:03-smoke-master---basic-vps/

@vasukulkarni

This comment has been minimized.

Member

vasukulkarni commented Apr 27, 2017

@liewegas the top folder in basic already has bluestore.yaml link from objectstore, so i just removed the folder since its redundant and correct 1node and systemd still uses xfs.

@vasukulkarni vasukulkarni changed the title from [DNM] qa/suites: Reduce fs combination tests for smoke, use bluestore to qa/suites: Reduce fs combination tests for smoke, use bluestore Apr 28, 2017

@vasukulkarni

This comment has been minimized.

Member

vasukulkarni commented Apr 28, 2017

@liewegas, I checked one of the logs and it is using the bluestore facet, some fs failure are unrelated http://pulpito.ceph.com/vasu-2017-04-27_22:28:03-smoke-master---basic-vps/

@vasukulkarni

This comment has been minimized.

Member

vasukulkarni commented May 2, 2017

@liewegas this one is ready, is there anything you want me to address here?

@liewegas

This comment has been minimized.

Member

liewegas commented May 2, 2017

@vasukulkarni please add a bluestore.yaml to basic and a filestore-xfs.yaml symlink to each of the other two subdirs (systemd and 1node)

@vasukulkarni

This comment has been minimized.

Member

vasukulkarni commented May 2, 2017

@liewegas, the basic folder already has bluestore.yaml from before(It also existed on top folder above objectstore), but no bluestore-comp.yaml (compression option), systemd is missing filestore-xfs.yaml but its defaults to xfs, I will rework and make both of them clearly visible here.

[vakulkar@vakulkar basic]$ pwd
/home/vakulkar/wip-reduce-fs-smoke/ceph/qa/suites/smoke/basic
[vakulkar@vakulkar basic]$ ls -lt
total 8
drwxrwxr-x. 2 vakulkar vakulkar 4096 Apr 27 15:18 tasks
-rw-rw-r--. 1 vakulkar vakulkar    0 Apr 27 15:18 %
lrwxrwxrwx. 1 vakulkar vakulkar   35 Apr 27 15:18 bluestore.yaml -> ../../../objectstore/bluestore.yaml
drwxrwxr-x. 2 vakulkar vakulkar 4096 Apr 27 15:18 clusters
[vakulkar@vakulkar basic]$ ls -lt ../1node/
total 8
-rw-rw-r--. 1 vakulkar vakulkar    0 Apr 27 15:18 %
drwxrwxr-x. 2 vakulkar vakulkar 4096 Apr 27 15:18 clusters
lrwxrwxrwx. 1 vakulkar vakulkar   39 Apr 27 15:18 filestore-xfs.yaml -> ../../../objectstore/filestore-xfs.yaml
drwxrwxr-x. 2 vakulkar vakulkar 4096 Apr 27 15:18 tasks

vasukulkarni added some commits Apr 27, 2017

Reduce fs combination tests for smoke, use bluestore
for basic and filestore.xfs for 1node and sytemd

Signed-off-by: Vasu Kulkarni <vasu@redhat.com>
Remove bluestore.yaml from top level folder which is redundant
objectstore folder has the link.

Signed-off-by: Vasu Kulkarni <vasu@redhat.com>
@vasukulkarni

This comment has been minimized.

Member

vasukulkarni commented May 2, 2017

@liewegas I made those changes, Do we want the bluestore-comp.yaml option as well?

@vasukulkarni

This comment has been minimized.

Member

vasukulkarni commented May 9, 2017

@liewegas ping ?

@liewegas

lgtm!

@liewegas liewegas merged commit bfcd2ff into master May 9, 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

@vasukulkarni vasukulkarni deleted the wip-reduce-smoke branch May 9, 2017

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