-
Notifications
You must be signed in to change notification settings - Fork 98
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 archive eats lot of space when creating thousands of volumes #236
Comments
@frasmarco , Thanks for raising this. Can you provide more details on the issue. This seems like a requirement that you were able to handle yourself. |
@abhilashshetty04 , sure I've been able to handle myself but we had to build a custom lvm-driver container, it should be included upstream, who ever cares or will be looking at lvm.conf archives in the localpv driver container? |
@frasmarco , This could be a good add-on to the product. Would like to know if you want to contribute? |
Yes, sure |
@frasmarco , Thanks in advance! |
@frasmarco and @abhilashshetty04 - I am moving this issues / feature request into the LocalPV-LVM repo and it will then get tracked and picked up by the new unified Mayastor project. Which will get activate very soon. It sounds like this feature request will be useful for LocalPV-LVM, and its a great candidate for community contribution since @frasmarco has already built to solution / code. @frasmarco are you still interested in working with us to contribute you code into openebs? |
This doesn't seem to be present on the
In fact, the
|
@frasmarco is this still an issue for you in the latest version? |
Linux LVM by default saves the previus copy of lvm configuration in /etc/lvm/archive/, this is perfectly fine normally but could become a problem when crrating thousand of LVs. As a workaround we built a copy of the container disabling LVM archive feature:
FROM docker.io/openebs/lvm-driver:1.0.0
RUN sed -i -e '/^\sarchive\s=/s/1/0/' /etc/lvm/lvm.conf
The text was updated successfully, but these errors were encountered: