Created attachment 84713 [details] [review]
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.