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

libcephfs: cleanups #12830

Merged
merged 4 commits into from Jan 20, 2017

Conversation

Projects
None yet
4 participants
@renhwztetecs
Member

renhwztetecs commented Jan 9, 2017

No description provided.

renhwztetecs added some commits Dec 9, 2016

libcephfs: fixup spelling mistakes
Signed-off-by: huanwen ren <ren.huanwen@zte.com.cn>
libcephfs: move ceph_fstatx() for readability
Signed-off-by: huanwen ren <ren.huanwen@zte.com.cn>
@xiexingguo

This comment has been minimized.

Member

xiexingguo commented Jan 10, 2017

retest this please

1 similar comment
@xiexingguo

This comment has been minimized.

Member

xiexingguo commented Jan 10, 2017

retest this please

@tchaikov tchaikov changed the title from libcephfs: libcephfs misc to libcephfs: cleanups Jan 10, 2017

@@ -999,7 +999,13 @@ TEST(LibCephFS, BadFileDesc) {
ASSERT_EQ(ceph_get_file_stripe_unit(cmount, -1), -EBADF);
ASSERT_EQ(ceph_get_file_pool(cmount, -1), -EBADF);
char poolname[80];
ASSERT_EQ(ceph_get_file_pool_name(cmount, -1, poolname, sizeof(poolname));

This comment has been minimized.

@jcsp

jcsp Jan 10, 2017

Contributor

missing second argument to ASSERT_EQ, does not compile

This comment has been minimized.

@renhwztetecs

renhwztetecs Jan 11, 2017

Member

thanks,I fix it

renhwztetecs added some commits Dec 13, 2016

test/libcephfs: add the missing test cases
Signed-off-by: huanwen ren <ren.huanwen@zte.com.cn>
libcephfs: add extern "C" in ceph_ll_walk()
Signed-off-by: huanwen ren <ren.huanwen@zte.com.cn>
@jcsp

jcsp approved these changes Jan 20, 2017

@jcsp jcsp merged commit 2d253c3 into ceph:master Jan 20, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@renhwztetecs renhwztetecs deleted the renhwztetecs:renhw-wip-libcephfs-misc branch Feb 16, 2017

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