-
Notifications
You must be signed in to change notification settings - Fork 6k
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
ceph-disk: ceph-disk on FreeBSD should not use mpath-code #14837
ceph-disk: ceph-disk on FreeBSD should not use mpath-code #14837
Conversation
08fa4c2
to
9d430af
Compare
src/ceph-disk/ceph_disk/main.py
Outdated
dev, | ||
], | ||
) | ||
if ret: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
EXIT STATUS
The fstyp utility exits 0 on success, and >0 if an error occurs or the filesystem type is not recognized.
see http://www.unix.com/man-page/freebsd/8/fstyp/,
so should be if ret == 0
, i guess?
On 29-04-17 02:42, Sage Weil wrote:
http://pulpito.ceph.com/sage-2017-04-29_00:40:18-ceph-disk-wip-sage-testing---basic-mira/
I'm having trouble interpreting the results there.
Looks like the 2 wrong tests too 5 and 1 hour to complete and the
correct test only 28 min.
And I only see not installed warnings...
Could it be that the faulty tests did not correctly start to begin with?
…--WjW
|
Signed-off-by: Willem Jan Withagen <wjw@digiware.nl>
9d430af
to
6727618
Compare
@wjwithagen i reran the ceph-disk suite at http://pulpito.ceph.com/kchai-2017-04-29_03:01:51-ceph-disk-wip-sage-testing---basic-vps/. and it passed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm. and the test suite passed. the latest change does not affect the change result.
Signed-off-by: Willem Jan Withagen wjw@digiware.nl