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/ceph-deploy]: run create mgr nodes as well #16216

Merged
merged 2 commits into from Jul 7, 2017

Conversation

Projects
None yet
2 participants
@vasukulkarni
Member

vasukulkarni commented Jul 7, 2017

Due to #16023, the mgr are no longer started alongside monitor nodes.

Testing at: http://pulpito.ceph.com/vasu-2017-07-07_15:53:08-ceph-deploy-master-distro-basic-vps/

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

create mgr nodes explicitly now, previously they where
started alongside monitor daemons due to systemd rules.

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

@vasukulkarni vasukulkarni added the tests label Jul 7, 2017

@vasukulkarni vasukulkarni requested review from jcsp, ktdreyer and liewegas Jul 7, 2017

@vasukulkarni

This comment has been minimized.

Member

vasukulkarni commented Jul 7, 2017

the tests are working, it failed due to the workunit issue which i believe needs update: .workunit.client.0.vpm141.stderr:rbd: error opening default pool 'rbd'

I will try to add that commit here.

@liewegas

This comment has been minimized.

Member

liewegas commented Jul 7, 2017

ah, yeah, test should just get an exec line that does 'ceph osd pool create rbd 8' or whatever.

create rbd pool since its not created by default anymore
Signed-off-by: Vasu Kulkarni <vasu@redhat.com>
@vasukulkarni

This comment has been minimized.

Member

vasukulkarni commented Jul 7, 2017

http://pulpito.ceph.com/vasu-2017-07-07_16:27:33-ceph-deploy-master-distro-basic-vps/, the one trusty failure with dmcrypt is unrelated, we didnt have trusty working for long time after mgr changes. there is a other trusty run without dmcrypt is passing now.

@vasukulkarni

This comment has been minimized.

Member

vasukulkarni commented Jul 7, 2017

retest this please

@vasukulkarni vasukulkarni merged commit 018e9f1 into master Jul 7, 2017

4 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
make check (arm64) make check succeeded
Details

@vasukulkarni vasukulkarni deleted the wip-mgr-cd-updates branch Jul 7, 2017

@vasukulkarni vasukulkarni restored the wip-mgr-cd-updates branch Jul 14, 2017

@liewegas liewegas deleted the wip-mgr-cd-updates branch Aug 25, 2017

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