Created attachment 84713 [details] [review] LVM2 support Here is a patch to add LVM2 support to UDisks2.
Known shortcomings are: - RAID, mirrors and striped volumes are not supported yet. - "pvmove" jobs can not be cancelled, interrupted pvmoves can not be resumed, and externally started pvmoves do not show up as jobs. - Clients must explicitly poll to keep certain properties up-to-date. - No automated tests. - Not really checked/reviewed yet for memory leaks and race conditions... I'll continue to work on this and will update this bug.
As I said a couple of months ago when we discussed this via email, I think this is a bad idea. Please move this to its own daemon/project or something; I don't want it in udisks. Also - preemptively - I would advise you to not just stash the patch in the Fedora/RHEL udisks RPM; doing so is tantamount to a fork and if you do this (and I can understand that you have a need), you should do it properly which includes renaming the project so there is no confusion that this is your code/ideas and not mine. Thanks for your consideration.
(In reply to comment #2) > As I said a couple of months ago when we discussed this via email, I think > this is a bad idea. Please move this to its own daemon/project or something; > I don't want it in udisks. Any chance you could document your reasoning/opinion/details in a public place so the questions don't keep coming up? Please forgive me if I've missed it ... I did search around for more details on mailing list, git logs and such.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.