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

mon: fix crush testing for new pools #5979

Merged
merged 3 commits into from
Sep 20, 2015
Merged

mon: fix crush testing for new pools #5979

merged 3 commits into from
Sep 20, 2015

Conversation

liewegas
Copy link
Member

only test the relevant ruleset

http://tracker.ceph.com/issues/13400

Signed-off-by: Sage Weil <sage@redhat.com>
Otherwise, we test *all* crush rules.. which might be a lot, and
which is a big waste of time and effort.

Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
@liewegas liewegas added this to the infernalis milestone Sep 18, 2015
@liewegas
Copy link
Member Author

@jecluis can you take a look?

@ghost
Copy link

ghost commented Sep 18, 2015

Reviewed-by: Loic Dachary <ldachary@redhat.com>

once the make check bot approves.

@ghost ghost self-assigned this Sep 18, 2015
@tchaikov
Copy link
Contributor

@dachary spotted again:

2015-09-18 15:21:15.233992 7f6ea07f5700 -1 asok(0x7f6e98000fd0) AdminSocketConfigObs::init: failed: AdminSocket::bind_and_listen: The UNIX domain socket path /home/jenkins/workspace/ceph/LABELS/ceph-centos-7-jenkins/src/testdir/test-7202/out/client.xx-profile-rw.1613.asok is too long! The maximum length on this system is 107

@ghost
Copy link

ghost commented Sep 18, 2015

@tchaikov ack. I'm tempted to wait it out until jenkins goes away.

liewegas added a commit that referenced this pull request Sep 20, 2015
mon: fix crush testing for new pools

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@liewegas liewegas merged commit 782855e into master Sep 20, 2015
@liewegas liewegas deleted the wip-crushtest branch September 20, 2015 17:50
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