[RFC] rewrite RBD provisioning to use go-ceph instead of rbd command #729
Conversation
nixpanic
force-pushed
the
wip/rbd/go-ceph
branch
2 times, most recently
from
cf0ba84
to
7d7963f
Compare
|
@nixpanic thanks ! one question, instead of referring to your fork of ceph-go, cant we refer directly to go-ceph master branch? is it the case that, there are unmerged PRs in master against your fork ? |
nixpanic
force-pushed
the
wip/rbd/go-ceph
branch
from
7d7963f
to
cc3c927
Compare
|
There are currently changes under review in the go-ceph project that this PR takes advantage of. Hence the |
nixpanic
force-pushed
the
wip/rbd/go-ceph
branch
5 times, most recently
from
4d8d98e
to
5d2fe62
Compare
nixpanic
force-pushed
the
wip/rbd/go-ceph
branch
3 times, most recently
from
c7770e4
to
6722436
Compare
|
@nixpanic as we are tracking this for |
nixpanic
force-pushed
the
wip/rbd/go-ceph
branch
5 times, most recently
from
99484fa
to
49daaf6
Compare
|
A cleaned-up version, using the upstream available go-ceph API is available at #777. That PR only implements CreateVolume, this PR will get updated with more functionality while go-ceph gets extended. |
nixpanic
force-pushed
the
wip/rbd/go-ceph
branch
from
6461075
to
2178ec4
Compare
nixpanic
force-pushed
the
wip/rbd/go-ceph
branch
2 times, most recently
from
aee8423
to
2cee603
Compare
nixpanic
force-pushed
the
wip/rbd/go-ceph
branch
3 times, most recently
from
5d7c4e4
to
1ab621e
Compare
nixpanic
force-pushed
the
wip/rbd/go-ceph
branch
from
fe7f6bb
to
ca44f22
Compare
Make it easier to open an rbd.Image by hiding the rados.IOContext for functions that only need operate on an rbd.Image. Signed-off-by: Niels de Vos <[email protected]>
Signed-off-by: Niels de Vos <[email protected]>
Reduce the number of calls to the 'rbd' command, and replace it with the go-ceph API. While at it, modernize the calls to the functions to be more object oriented. Signed-off-by: Niels de Vos <[email protected]>
Signed-off-by: Niels de Vos <[email protected]>
Signed-off-by: Niels de Vos <[email protected]>
Signed-off-by: Niels de Vos <[email protected]>
Signed-off-by: Niels de Vos <[email protected]>
Signed-off-by: Niels de Vos <[email protected]>
Signed-off-by: Niels de Vos <[email protected]>
Signed-off-by: Niels de Vos <[email protected]>
Signed-off-by: Niels de Vos <[email protected]>
Signed-off-by: Niels de Vos <[email protected]>
Signed-off-by: Niels de Vos <[email protected]>
nixpanic
force-pushed
the
wip/rbd/go-ceph
branch
from
ca44f22
to
0c620eb
Compare
|
This pull request now has conflicts with the target branch. Could you please resolve conflicts and force push the corrected changes? 🙏 |
|
The majority of calls has been replaced by a go-ceph implementation. Follow the Use go-ceph for volume management operations project for an updated status. |
Describe what this PR does
This is a PR that should not be merged as is, but is used for Request For Comments.
There is a need to extend go-ceph with certain functions so that the current RBD provisioner can be rewritten to use the librbd.so library directly. This should increase performance.
Initial changes, and planned work, in no particular order:
createVolume()deleteVolume(), foreground only, not as a background taskrbdVolumeobject that can have functionsThe changes in the branch used to create this PR depend on a work-in-progress version of go-ceph. Hopefully it will be possible to merge parts of this [RFC] PR when go-ceph functionality gets merged,
Related issues
See-also: #449