-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
rbd: support namespaces for image migration #24836
Conversation
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
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.
Actually, I think RefreshReques::get_migration_info
needs update to properly set namespace in parent_md->spec
and migration_info
.
@trociny Good catch -- added |
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.
Don't we want to add the namespace
field to MigrationInfo
structure? It will probably have never been used but the same it true for pool_id
and image_id
.
@trociny fixed |
char *source_image_name; | ||
char *source_image_id; | ||
int64_t dest_pool_id; | ||
char *dest_pool_namespace; |
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.
@dillaman need to update ctypedef struct rbd_image_migration_status_t
in rbd.pyx:
zhuzha:~/ceph/ceph.ci/build% RBD_FEATURES=127 PYTHONPATH=../src/test/pybind:../src/pybind:./lib/cython_modules/lib.2 LD_LIBRARY_PATH=./lib nosetests -v test_rbd:TestMigration.test_migration
test_rbd.TestMigration.test_migration ... ERROR
======================================================================
ERROR: test_rbd.TestMigration.test_migration
----------------------------------------------------------------------
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/nose/case.py", line 197, in runTest
self.test(*self.arg)
File "/ssd/ceph/ceph.ci/src/test/pybind/test_rbd.py", line 2041, in test_migration
status = RBD().migration_status(ioctx, image_name)
File "rbd.pyx", line 1413, in rbd.RBD.migration_status (/home/mgolub/ceph/ceph.ci/build/src/pybind/rbd/rbd.c:11588)
OSError: [errno 34] error getting migration status
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.
Fixed -- and just learned that Cython doesn't validate the structs.
Fixes: http://tracker.ceph.com/issues/26951 Signed-off-by: Jason Dillaman <dillaman@redhat.com>
…ceph#24836 · ceph/ceph · GitHub * wip-26951: rbd: support namespaces for image migration
Fixes: http://tracker.ceph.com/issues/26951
Signed-off-by: Jason Dillaman dillaman@redhat.com