Add openstack requirements to smoke suite #12913
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Zack Cerza <zack@redhat.com>
|
@zmc did you try running it ? |
yuriw
approved these changes
Jan 12, 2017
|
Hrm. Jobs are failing because of: I don't think this could be related to these changes, but I can't be sure... |
|
Zack, we need to bump the monitors to 3, instead of 2.
…On Thu, Jan 12, 2017 at 3:14 PM, Zack Cerza ***@***.***> wrote:
Hrm. Jobs are failing because of:
2017-01-12T23:06:20.502 INFO:teuthology.orchestra.run.ovh005.stderr:mount.ceph: unrecognized mount option "norequire_active_mds", passing to kernel.
2017-01-12T23:06:20.506 INFO:teuthology.orchestra.run.ovh005.stdout:parsing options: name=0,secretfile=/home/ubuntu/cephtest/ceph.data/client.0.secret,norequire_active_mds
2017-01-12T23:06:20.509 INFO:teuthology.orchestra.run.ovh005.stdout:mount error 22 = Invalid argument
I don't think this could be related to these changes, but I can't be
sure...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#12913 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABuV9doeMclPQbNkgIND9jFyrk7ZK13tks5rRrPigaJpZM4LiSbE>
.
|
|
@tmuthamizhan where? |
|
http://qa-proxy.ceph.com/teuthology/zack-2017-01-12_15:49:01-smoke-master---basic-ovh/713392/teuthology.log
- this test yaml has only 2 mons. and the problem also seems to be that it
is unable to find ceph packages.
…On Thu, Jan 12, 2017 at 3:27 PM, Zack Cerza ***@***.***> wrote:
@tmuthamizhan <https://github.com/tmuthamizhan> where?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#12913 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABuV9RoBTD2e3HdXMY5b3kSlo8VRDyAOks5rRrb_gaJpZM4LiSbE>
.
|
|
@tmuthamizhan ah, you're right. I'm not sure which node to add it to. I'm really just trying to get the openstack files in here. |
|
it should go in here -
https://github.com/ceph/ceph/blob/master/qa/suites/smoke/systemd/tasks/systemd.yaml
like this:
roles:
- [mon.a, osd.0]
- [osd.1, osd.2]
- [mds.a, osd.3]
- [mon.b, mon.c, client.0]
…On Thu, Jan 12, 2017 at 3:38 PM, Zack Cerza ***@***.***> wrote:
@tmuthamizhan <https://github.com/tmuthamizhan> ah, you're right. I'm not
sure which remote to add it to. I'm really just trying to get the openstack
files in here.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#12913 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABuV9TEsJPQNQSSmNAxONJsb5WUBMnj9ks5rRrmNgaJpZM4LiSbE>
.
|
|
3rd monitor is not required for this test, its tests only systemd part, Also tamil you can't overload mds or mon on same node for ceph-deploy tests. |
|
@vasu, i have not had any issues with overloading mons/mds for ceph-deploy
tests in the past.
…On Thu, Jan 12, 2017 at 3:43 PM, vasukulkarni ***@***.***> wrote:
3rd monitor is not required for this test, its tests only systemd part,
Also tamil you can't overload mds or mon on same node for ceph-deploy tests.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#12913 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABuV9ZQzejad1lbVWi8l48VvzvY5yq9jks5rRrqGgaJpZM4LiSbE>
.
|
|
@tmuthamizhan it has no effect in the code, the roles are internally modified for that test. |
|
@ZackC, you are right. the errors appear to be something with mount and not
your change.
2017-01-12T23:06:20.502
INFO:teuthology.orchestra.run.ovh005.stderr:mount.ceph: unrecognized
mount option "norequire_active_mds", passing to
kernel.2017-01-12T23:06:20.506
INFO:teuthology.orchestra.run.ovh005.stdout:parsing options:
name=0,secretfile=/home/ubuntu/cephtest/ceph.data/client.0.secret,norequire_active_mds2017-01-12T23:06:20.509
INFO:teuthology.orchestra.run.ovh005.stdout:mount error 22 = Invalid
argument
…On Thu, Jan 12, 2017 at 3:50 PM, vasukulkarni ***@***.***> wrote:
@tmuthamizhan <https://github.com/tmuthamizhan> it has no effect in the
code, the roles are internally modified for that test.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#12913 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABuV9fYdutwsXVe6RhVIV0sYKoudIz8gks5rRrxNgaJpZM4LiSbE>
.
|
|
@tmuthamizhan - I think you tagged the wrong person. I haven't been involved in this project. Hope it goes well though! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Zack Cerza zack@redhat.com