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
pybind: Implementation of rados_ioctx_snapshot_rollback #6878
Conversation
e3ccfb8
to
a32d90b
Compare
could you add your Signed-off-by to the commit message? it's just in the PR description right now. Use 'git commit --amend -s' to edit the commit message, adding the signed-off-by automatically, then force push with 'git push origin +pybind_rados_ioctx_snap_rollback' |
a32d90b
to
64a8f7b
Compare
@manens you can ignore the bot failure, it is a temporary network failure. You will get another run after rebasing & repushing. |
64a8f7b
to
1e995cb
Compare
""" | ||
self.require_ioctx_open() | ||
ret = run_in_thread(self.librados.rados_ioctx_snap_rollback, | ||
(self.io, cstr(oid), snap.name)) |
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.
snap.name is populated as a python string, so we should encode it as a byte string via cstr(snap.name) when passing it to the c function here
1e995cb
to
3ac716c
Compare
Signed-off-by: Florent Manens <florent@beezim.fr>
3ac716c
to
2b50a75
Compare
pybind: Implementation of rados_ioctx_snapshot_rollback Reviewed-by: Josh Durgin <jdurgin@redhat.com>
Signed-off-by: Florent Manens florent@beezim.fr