-
Notifications
You must be signed in to change notification settings - Fork 100
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
Lvm driver should create volume group and pvc. User should not manually create it. #245
Comments
@yash97 Thanks for raising the requirement. However, creating PV / VG using LVM localPV is not straight forward tasl. We dont have upstream caller like for LV (volume). We would need to create custom resource for them and reconciler as well. In that case also you need to create Custom resources anyway. So it seems easier to preprovision PV / VG as we do now. |
@yash97 Although if you want to contribute, can you please raise a design document for the same so that we can review and plan accordingly? Thanks |
We've just coded the LVM Dynamic auto-provisioning feature for Mayastor, and enabled it as an experimental feature. That new code now auto-provisions all LVM entities (PV, VG and LV) and the Filesystem. - There is no need for manual LVM provisioning by the user. The code is currently being tested as part of Mayastor as an evolution of our DiskGroup. You can now have SPDK or LVM backend managed storage. Now that we have all that auto-provisioning logic coded, our plan is to roll that in to the LVM-LocalPV Data-Engine also. Irrespective of what the CSI spec says, we strongly believe that our user experience strategy should be that the user will never need to manually interact with LVM at all and all LVM backend tasks will be dynamically automated by OpenEBS. note: The current code doesn't allow RAIDed LV's yet. That's definitely coming next. (we're also enabling the same Dynamic auto-provisioning for ZFS-LocalPV too. Also, Issue #164 also relates directly to this issue as #164 Hope this helps. |
Hello! I am the new PM for the OpenEBS project at DataCore and wanted to shed some light on the discussion above. When it comes to managing LVM PVs and VGs, we intend to keep this management outside of the core OpenEBS scope. LocalPV and Mayastor may create LVs on top of pre-created VGs but for now will not create and manage the VGs themselves. This is similar to the approach we have for ZFS where we start acting at the zVol provisioning level but not deal with creation and management of the zPool itself. Hope this helps. |
Hi @Alex130469 - Dave here (@orville-wright ), Maintainer of our OpenEBS CNCF opensource project. Thanks for the Issue comment & update. I think our CNCF OpenEBS opensource community may appreciate a little extra info/clarity...
|
Thanks for your insights, @Alex130469 and @orville-wright . The current issue is a good-to-have enhancement, but not a blocker or must-have for the users. LVM and ZFS are very popular storage technologies, administrators are well-versed in these stacks and can easily configure the required Volume Groups and ZPools as per their needs. This has been the premise on which the localpv-lvm and localpv-zfs came into existence. There will be considerable amount of effort needed in providing the requested functionality on the localpv drivers, as mentioned in this comment (#245 (comment)). For now, this is likely to be of lesser priority. Once the roadmap is drafted, it will be presented to all the maintainers for their review, where this issue can be discussed if need be. Meanwhile, community contributions are always welcome. |
Describe the problem/challenge you have
[A description of the current limitation/problem/challenge that you are experiencing.]
Right now user has to manually create lvm volume as per my understanding. This should not be the case imo. Lvm driver should create it based on config.
The text was updated successfully, but these errors were encountered: