Skip to content
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/tests: fix volume size when running in ovh #21961

Merged
merged 2 commits into from
May 20, 2018
Merged

qa/tests: fix volume size when running in ovh #21961

merged 2 commits into from
May 20, 2018

Conversation

vasukulkarni
Copy link
Contributor

a) fix volume size when running tests in ovh
b) remove rhel from support distros

Testing here: http://pulpito.ceph.com/vasu-2018-05-12_17:33:03-ceph-deploy-master-distro-basic-ovh/

…b as per conf

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

Signed-off-by: Vasu Kulkarni <vasu@redhat.com>
@@ -1 +0,0 @@
../../../distros/supported
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why do we want to remove RHEL from supported distros when running ceph-deploy tests?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tchaikov ceph-deploy and ceph-ansible work differently when rhel comes into picture(tools are configured with different assumptions)

eg: http://qa-proxy.ceph.com/teuthology/teuthology-2018-05-16_05:55:02-ceph-deploy-mimic-distro-basic-mira/2538001/teuthology.log

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2018-05-16T06:11:08.108 INFO:teuthology.orchestra.run.mira081.stderr:[mira081][INFO  ] Running command: sudo yum -y install ceph ceph-osd ceph-
mds ceph-mon ceph-radosgw
2018-05-16T06:11:08.377 INFO:teuthology.orchestra.run.mira081.stderr:[mira081][DEBUG ] Loaded plugins: fastestmirror, langpacks, priorities, pr
oduct-id, search-
2018-05-16T06:11:08.377 INFO:teuthology.orchestra.run.mira081.stderr:[mira081][DEBUG ]               : disabled-repos, subscription-manager
2018-05-16T06:11:08.641 INFO:teuthology.orchestra.run.mira081.stderr:[mira081][DEBUG ] Determining fastest mirrors
2018-05-16T06:11:08.706 INFO:teuthology.orchestra.run.mira081.stderr:[mira081][DEBUG ]  * epel: ftp.linux.ncsu.edu
2018-05-16T06:11:41.000 INFO:teuthology.orchestra.run.mira081.stderr:[mira081][DEBUG ] No package ceph available.
2018-05-16T06:11:42.719 INFO:teuthology.orchestra.run.mira081.stderr:[mira081][DEBUG ] No package ceph-osd available.
2018-05-16T06:11:44.589 INFO:teuthology.orchestra.run.mira081.stderr:[mira081][DEBUG ] No package ceph-mds available.
2018-05-16T06:11:46.308 INFO:teuthology.orchestra.run.mira081.stderr:[mira081][DEBUG ] No package ceph-mon available.
2018-05-16T06:11:48.178 INFO:teuthology.orchestra.run.mira081.stderr:[mira081][DEBUG ] No package ceph-radosgw available.
2018-05-16T06:11:49.145 INFO:teuthology.orchestra.run.mira081.stderr:[mira081][WARNING] Error: Nothing to do
2018-05-16T06:11:49.310 INFO:teuthology.orchestra.run.mira081.stderr:[mira081][ERROR ] RuntimeError: command returned non-zero exit status: 1

is it possible that we can setup a ceph repo for RHEL ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tchaikov I am not sure its worth enhancing ceph-deploy to support rhel and there will be obviously more work on the build side too, unless someone is going to signup on the build/rpm items it would be wasted effort for little to no gain.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there will be obviously more work on the build side too

i think we are already building packages for RHEL, see http://docs.ceph.com/docs/master/install/get-packages/#add-ceph

by "setting up a ceph repo for RHEL", i mean we can update the yum settings so it can find the ceph packages.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It has to be something like this, RHEL was more configured for downstream so didnt get much attention. https://github.com/ceph/ceph-deploy/blob/master/ceph_deploy/hosts/centos/install.py#L69-L124

Its not just released buids we check, it has to match the dev commit sha and there could be other things from build side we might encounter.

@vasukulkarni
Copy link
Contributor Author

vasukulkarni commented May 16, 2018

Copy link
Contributor

@yuriw yuriw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgt

@tchaikov tchaikov merged commit 5ceee88 into master May 20, 2018
@tchaikov tchaikov deleted the wip-cd-ovh-vol branch May 20, 2018 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants