Lennart Poettering <[email protected]> writes:

> On Wed, 01.04.15 14:33, Jan Synacek ([email protected]) wrote:
>
>> Creating subvolumes in chrooted environments makes them
>> undeletable and breaks mock.
>
> Humm, I am not convinced that this is a good idea.
>
> The chroot environments are hardly "undeletable", they just require
> you to delete them explicitly. There's work going on to tech
> btrfs-progs recursive deleting of subvolumes. I am pretty sure that's
> the right fix and mock should really be updated to deal with that...

"undeletable" was a bad wording from my side, sorry for that. What I
really meant is that mock simply couldn't deal with it... 

> I am also against this since chrooting is an implementation detail of
> mock, nothing more, and the fact that mock's recursive deletion logic
> cannot handle removal of subvolumes is not directly connected to the
> fact that mock uses chroot.
>
> Sorry, but we need to find a different solution for this.
>
> Maybe mock should use seccomp to make the subvolume creation ioctls
> unavailable, or it should be updated to deal with subvolumes properly.

I agree that mock should be enhanced to cope with subvolumes, but I also
think that systemd shouldn't create them where it doesn't make
sense. I don't think that that's achievable with the current logic. Am I
missing something?

Cheers,
-- 
Jan Synacek
Software Engineer, Red Hat

Attachment: signature.asc
Description: PGP signature

_______________________________________________
systemd-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to