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
crimson/osd: add --mkkey support #28534
Conversation
src/common/buffer.cc
Outdated
@@ -1838,6 +1844,33 @@ ssize_t buffer::list::read_fd(int fd, size_t len) | |||
return ret; | |||
} | |||
|
|||
#ifdef WITH_SEASTAR |
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.
i can move this helper into src/crimson/common
so we don't need to taint buffer.h
and buffer.cc
in this way.
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.
We might also consider src/common/buffer_seastar.cc
.
derr << "Unable to get a Ceph keyring." << dendl; | ||
forker.exit(1); | ||
} | ||
KeyRing keyring; |
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.
ACK.
src/common/buffer.cc
Outdated
@@ -1838,6 +1844,33 @@ ssize_t buffer::list::read_fd(int fd, size_t len) | |||
return ret; | |||
} | |||
|
|||
#ifdef WITH_SEASTAR |
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.
We might also consider src/common/buffer_seastar.cc
.
i will move the read/write file helpers for crimson to |
933404b
to
9f58998
Compare
it never fails unless out of memory. and it would be simpler to just allocate it on stack. Signed-off-by: Kefu Chai <kchai@redhat.com>
9f58998
to
b9fe760
Compare
Signed-off-by: Kefu Chai <kchai@redhat.com>
instead of using the blocking calls so we can write to file without blocking the reactor Signed-off-by: Kefu Chai <kchai@redhat.com>
see qa/tasks/ceph.py for how it is used, and why we need it. Signed-off-by: Kefu Chai <kchai@redhat.com>
OSD::stop() is a top-level method in the sense that it will be called directly when engine stops. so it should not throw any exceptions. instead, we will print out the error and ignore the exception if any. Signed-off-by: Kefu Chai <kchai@redhat.com>
b9fe760
to
5461047
Compare
@rzarzynski fixed and repushed. |
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.
No description provided.