Skip to content
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

allow block devices for SYSTEM_DS images #76

Conversation

atodorov-storpool
Copy link
Contributor

There are block devices that can be used as system datastore like:

CLVM
iSCSI
StorPool

This patch is in need to test and develop our addon against the 'master' branch.
Probably better solution is to have the list of supported devices as list in the configuration files.

Tere are block devices that can be used as system datastore like:

CLVM
iSCSI
StorPool

This patch is in need to test and develop our addon against the 'master' branch.
Probably better solution is to have the list of supported devices as list in the configuration files.
@rsmontero
Copy link
Member

Hi

We have just merge some functionality (branch feature-3987) to use Ceph as a System DS. The patch allow to use volatile disks as Ceph volumes. The changes are basically:

  • Volatile disks get DS attributes (inherited) so you can get DISK_TYPE in the volatile disk to tune the mkfs actions.
  • restore/save actions got TM hooks. So restore.tm_mad is call before restoring and same for save. You can use this to put the checkpoint in storpool
  • System DS monitor has been improved. (shared System DS are monitored once from the front-end/BRIDGE_LIST you need a monitor script in the TM directory)
  • There is a new attribute ds_migrate to enable/disable system ds migrations.

Could you check against the current HEAD at master and see if the changes suits storpool driver. Any feedback will be more than welcome!

Going back to the patch, we introduced the GLUSTER, RBD, SHEEPDOG to deal with deployment file generation. Currently the block devices are managed through links qemu uses the devices through this links... so I am not sure about this patch.

THANKS

Ruben

@atodorov-storpool
Copy link
Contributor Author

Hi Ruben,

Thank you for your prompt response.

Somehow I've missed the possibility to get DS attributes. I'll test it and report back. I was concerned is it a repeat of the disk polling case where I am patching vmm/kvm/poll to enable reporting for StorPool.
Anyway I like the SYSTEM_DS support because it will enlighten some dark magic I am doing in our driver to enable context and volatile disks on StorPool for 4.12+ and we have customer using it :)

Cheers
Anton

rsmontero pushed a commit that referenced this pull request Jul 5, 2020
rsmontero pushed a commit that referenced this pull request Jan 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants