Updates:
        Cc: [email protected]

Comment #3 on issue 1180 by [email protected]: Ceph/RBD support is broken with Ceph 'Jewel' due to incorrect showmapped command
https://code.google.com/p/ganeti/issues/detail?id=1180

I know almost nothing about ceph, but just looking at the sources for hammer and jewel, it does appear that the rbd command line tool in jewel lost the --pool/-p option that hammer had.

I think your fix isn't going to work entirely properly though. What happens if you have two mappings in different pools that happen to have the same name? I think you're going to have to change _ParseRbdShowmappedJson to filter out mappings from the specified pool.

Also, it looks like the rbd subcommands create, rm, map, resize, import and export also have the same problem. In hammer you can do rbd $SUB_CMD -p mypool mymap or rbd $SUB_CMD mypool/mymap (the parsing of the second is implemented by set_pool_image_name(...) in src/rbd.cc).

In jewel only the second syntax appears to be supported (again, just by eyeballing the source code).

I think that you're going to need to update Create, Remove, _MapVolumeToBlockdev, Close, Import and Export in RADOSBlockDevice to use the mypool/mymap syntax.

David, do you have any input on this?

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

Reply via email to