* [PATCH] rules: block - add dm devices to whitelist @ 2017-07-05 11:01 David Disseldorp 2017-07-10 8:38 ` Lennart Poettering 0 siblings, 1 reply; 7+ messages in thread From: David Disseldorp @ 2017-07-05 11:01 UTC (permalink / raw) To: systemd-devel; +Cc: David Disseldorp, ceph-devel Ceph relies on by-partuuid symlinks, in order to locate the journal partition from a given OSD partition. For details, see http://tracker.ceph.com/issues/19489. --- rules/60-persistent-storage.rules | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rules/60-persistent-storage.rules b/rules/60-persistent-storage.rules index 5ab03fc27..bc0721f32 100644 --- a/rules/60-persistent-storage.rules +++ b/rules/60-persistent-storage.rules @@ -6,7 +6,7 @@ ACTION=="remove", GOTO="persistent_storage_end" SUBSYSTEM!="block", GOTO="persistent_storage_end" -KERNEL!="loop*|mmcblk*[0-9]|msblk*[0-9]|mspblk*[0-9]|nvme*|sd*|sr*|vd*|xvd*|bcache*|cciss*|dasd*", GOTO="persistent_storage_end" +KERNEL!="loop*|mmcblk*[0-9]|msblk*[0-9]|mspblk*[0-9]|nvme*|sd*|sr*|vd*|xvd*|bcache*|cciss*|dasd*|dm*", GOTO="persistent_storage_end" # ignore partitions that span the entire disk TEST=="whole_disk", GOTO="persistent_storage_end" -- 2.12.3 _______________________________________________ systemd-devel mailing list systemd-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/systemd-devel ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] rules: block - add dm devices to whitelist 2017-07-05 11:01 [PATCH] rules: block - add dm devices to whitelist David Disseldorp @ 2017-07-10 8:38 ` Lennart Poettering 2017-07-10 9:37 ` [systemd-devel] " David Disseldorp 0 siblings, 1 reply; 7+ messages in thread From: Lennart Poettering @ 2017-07-10 8:38 UTC (permalink / raw) To: David Disseldorp; +Cc: ceph-devel, systemd-devel On Wed, 05.07.17 13:01, David Disseldorp (ddiss@suse.de) wrote: > Ceph relies on by-partuuid symlinks, in order to locate the journal > partition from a given OSD partition. For details, see > http://tracker.ceph.com/issues/19489. This appears way too broad, as it would apply to all LVM and all other devices. It appears to me Ceph should do the same as LVM does for this, and ship its own set of rules, and be careful to only match against the actual devices it creates. I hope that makes sense, Lennart -- Lennart Poettering, Red Hat _______________________________________________ systemd-devel mailing list systemd-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/systemd-devel ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [systemd-devel] [PATCH] rules: block - add dm devices to whitelist 2017-07-10 8:38 ` Lennart Poettering @ 2017-07-10 9:37 ` David Disseldorp 2017-07-10 9:53 ` Lennart Poettering 0 siblings, 1 reply; 7+ messages in thread From: David Disseldorp @ 2017-07-10 9:37 UTC (permalink / raw) To: Lennart Poettering; +Cc: systemd-devel, ceph-devel Thanks for the feedback, Lennart... On Mon, 10 Jul 2017 10:38:38 +0200, Lennart Poettering wrote: > On Wed, 05.07.17 13:01, David Disseldorp (ddiss@suse.de) wrote: > > > Ceph relies on by-partuuid symlinks, in order to locate the journal > > partition from a given OSD partition. For details, see > > http://tracker.ceph.com/issues/19489. > > This appears way too broad, as it would apply to all LVM and all other > devices. > > It appears to me Ceph should do the same as LVM does for this, and > ship its own set of rules, and be careful to only match against the > actual devices it creates. We can certainly do this in a Ceph specific manner via the existing 95-ceph-osd.rules, but my impression was that the by-partuuid symlinks are "owned" by 60-persistent-storage.rules . If you don't think the existence of these symlinks for dm paths will be of use to others then I'll go ahead and propose it as a Ceph only change. Cheers, David ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] rules: block - add dm devices to whitelist 2017-07-10 9:37 ` [systemd-devel] " David Disseldorp @ 2017-07-10 9:53 ` Lennart Poettering 2017-07-10 10:14 ` Peter Rajnoha 0 siblings, 1 reply; 7+ messages in thread From: Lennart Poettering @ 2017-07-10 9:53 UTC (permalink / raw) To: David Disseldorp; +Cc: ceph-devel, systemd-devel On Mon, 10.07.17 11:37, David Disseldorp (ddiss@suse.de) wrote: > Thanks for the feedback, Lennart... > > On Mon, 10 Jul 2017 10:38:38 +0200, Lennart Poettering wrote: > > > On Wed, 05.07.17 13:01, David Disseldorp (ddiss@suse.de) wrote: > > > > > Ceph relies on by-partuuid symlinks, in order to locate the journal > > > partition from a given OSD partition. For details, see > > > http://tracker.ceph.com/issues/19489. > > > > This appears way too broad, as it would apply to all LVM and all other > > devices. > > > > It appears to me Ceph should do the same as LVM does for this, and > > ship its own set of rules, and be careful to only match against the > > actual devices it creates. > > We can certainly do this in a Ceph specific manner via the existing > 95-ceph-osd.rules, but my impression was that the by-partuuid symlinks > are "owned" by 60-persistent-storage.rules . > > If you don't think the existence of these symlinks for dm paths will be > of use to others then I'll go ahead and propose it as a Ceph only > change. Hmmm, so thinking about this again: "partuuid" is actually for GPT partition UUIDs if I recall recorrectly, they wouldn't be generated for DM devices anyway, since they are one layer removed from the GPT, so are you even sure this will do what you are asking for? But even if this actually works: DM links so far are created by the LVM/libdevicemapper/ packages, not by udev, and I don't think this should change. Hence, please talk to the LVM/libdevicemapper folks about this and ask them for including it. Lennart -- Lennart Poettering, Red Hat _______________________________________________ systemd-devel mailing list systemd-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/systemd-devel ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] rules: block - add dm devices to whitelist 2017-07-10 9:53 ` Lennart Poettering @ 2017-07-10 10:14 ` Peter Rajnoha 2017-07-10 10:47 ` Peter Rajnoha 0 siblings, 1 reply; 7+ messages in thread From: Peter Rajnoha @ 2017-07-10 10:14 UTC (permalink / raw) To: Lennart Poettering, David Disseldorp; +Cc: ceph-devel, systemd-devel On 07/10/2017 11:53 AM, Lennart Poettering wrote: > On Mon, 10.07.17 11:37, David Disseldorp (ddiss@suse.de) wrote: > >> Thanks for the feedback, Lennart... >> >> On Mon, 10 Jul 2017 10:38:38 +0200, Lennart Poettering wrote: >> >>> On Wed, 05.07.17 13:01, David Disseldorp (ddiss@suse.de) wrote: >>> >>>> Ceph relies on by-partuuid symlinks, in order to locate the journal >>>> partition from a given OSD partition. For details, see >>>> http://tracker.ceph.com/issues/19489. >>> >>> This appears way too broad, as it would apply to all LVM and all other >>> devices. >>> >>> It appears to me Ceph should do the same as LVM does for this, and >>> ship its own set of rules, and be careful to only match against the >>> actual devices it creates. >> >> We can certainly do this in a Ceph specific manner via the existing >> 95-ceph-osd.rules, but my impression was that the by-partuuid symlinks >> are "owned" by 60-persistent-storage.rules . >> >> If you don't think the existence of these symlinks for dm paths will be >> of use to others then I'll go ahead and propose it as a Ceph only >> change. > > Hmmm, so thinking about this again: "partuuid" is actually for GPT > partition UUIDs if I recall recorrectly, they wouldn't be generated > for DM devices anyway, since they are one layer removed from the GPT, > so are you even sure this will do what you are asking for? > > But even if this actually works: DM links so far are created by the > LVM/libdevicemapper/ packages, not by udev, and I don't think this > should change. Hence, please talk to the LVM/libdevicemapper folks > about this and ask them for including it. > Yes, please, any rules for symlinks which should be created under /dev/disk for DM devices (including all its subsystems like LVM, mpath...) should go into 13-dm-disk.rules that is part of LVM/DM source tree: https://sourceware.org/git/?p=lvm2.git;a=blob;f=udev/13-dm-disk.rules.in Now, when we create a partition over a DM device, there's a new mapping created on top for each partition (either by calling kpartx manually or by having it created by partitioning tool directly if it supports that). So in this case, it's not the kernel directly who creates the partitions, but they're simply another DM devices created on top of the underlying DM device to represent these partitions. But I think that doesn't matter - we should still create those symlinks for people to still have a possibility to reference the device by its part uuid - I'll fix 13-dm-disk.rules to include this. -- Peter _______________________________________________ systemd-devel mailing list systemd-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/systemd-devel ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] rules: block - add dm devices to whitelist 2017-07-10 10:14 ` Peter Rajnoha @ 2017-07-10 10:47 ` Peter Rajnoha 2017-07-10 11:18 ` [systemd-devel] " David Disseldorp 0 siblings, 1 reply; 7+ messages in thread From: Peter Rajnoha @ 2017-07-10 10:47 UTC (permalink / raw) To: David Disseldorp; +Cc: ceph-devel, systemd-devel On 07/10/2017 12:14 PM, Peter Rajnoha wrote: > On 07/10/2017 11:53 AM, Lennart Poettering wrote: >> On Mon, 10.07.17 11:37, David Disseldorp (ddiss@suse.de) wrote: >> >>> Thanks for the feedback, Lennart... >>> >>> On Mon, 10 Jul 2017 10:38:38 +0200, Lennart Poettering wrote: >>> >>>> On Wed, 05.07.17 13:01, David Disseldorp (ddiss@suse.de) wrote: >>>> >>>>> Ceph relies on by-partuuid symlinks, in order to locate the journal >>>>> partition from a given OSD partition. For details, see >>>>> http://tracker.ceph.com/issues/19489. >>>> >>>> This appears way too broad, as it would apply to all LVM and all other >>>> devices. >>>> >>>> It appears to me Ceph should do the same as LVM does for this, and >>>> ship its own set of rules, and be careful to only match against the >>>> actual devices it creates. >>> >>> We can certainly do this in a Ceph specific manner via the existing >>> 95-ceph-osd.rules, but my impression was that the by-partuuid symlinks >>> are "owned" by 60-persistent-storage.rules . >>> >>> If you don't think the existence of these symlinks for dm paths will be >>> of use to others then I'll go ahead and propose it as a Ceph only >>> change. >> >> Hmmm, so thinking about this again: "partuuid" is actually for GPT >> partition UUIDs if I recall recorrectly, they wouldn't be generated >> for DM devices anyway, since they are one layer removed from the GPT, >> so are you even sure this will do what you are asking for? >> >> But even if this actually works: DM links so far are created by the >> LVM/libdevicemapper/ packages, not by udev, and I don't think this >> should change. Hence, please talk to the LVM/libdevicemapper folks >> about this and ask them for including it. >> > > Yes, please, any rules for symlinks which should be created under > /dev/disk for DM devices (including all its subsystems like LVM, > mpath...) should go into 13-dm-disk.rules that is part of LVM/DM source > tree: > > https://sourceware.org/git/?p=lvm2.git;a=blob;f=udev/13-dm-disk.rules.in > > Now, when we create a partition over a DM device, there's a new mapping > created on top for each partition (either by calling kpartx manually or > by having it created by partitioning tool directly if it supports that). > So in this case, it's not the kernel directly who creates the > partitions, but they're simply another DM devices created on top of the > underlying DM device to represent these partitions. But I think that > doesn't matter - we should still create those symlinks for people to > still have a possibility to reference the device by its part uuid - I'll > fix 13-dm-disk.rules to include this. > Fixed here: https://sourceware.org/git/?p=lvm2.git;a=commit;h=c48149cf80c6582c2369bc7f8a33d794021d9dae -- Peter _______________________________________________ systemd-devel mailing list systemd-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/systemd-devel ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [systemd-devel] [PATCH] rules: block - add dm devices to whitelist 2017-07-10 10:47 ` Peter Rajnoha @ 2017-07-10 11:18 ` David Disseldorp 0 siblings, 0 replies; 7+ messages in thread From: David Disseldorp @ 2017-07-10 11:18 UTC (permalink / raw) To: Peter Rajnoha; +Cc: ceph-devel, systemd-devel On Mon, 10 Jul 2017 12:47:24 +0200, Peter Rajnoha wrote: > On 07/10/2017 12:14 PM, Peter Rajnoha wrote: ... > > Yes, please, any rules for symlinks which should be created under > > /dev/disk for DM devices (including all its subsystems like LVM, > > mpath...) should go into 13-dm-disk.rules that is part of LVM/DM source > > tree: > > > > https://sourceware.org/git/?p=lvm2.git;a=blob;f=udev/13-dm-disk.rules.in > > > > Now, when we create a partition over a DM device, there's a new mapping > > created on top for each partition (either by calling kpartx manually or > > by having it created by partitioning tool directly if it supports that). > > So in this case, it's not the kernel directly who creates the > > partitions, but they're simply another DM devices created on top of the > > underlying DM device to represent these partitions. But I think that > > doesn't matter - we should still create those symlinks for people to > > still have a possibility to reference the device by its part uuid - I'll > > fix 13-dm-disk.rules to include this. > > > > Fixed here: > > https://sourceware.org/git/?p=lvm2.git;a=commit;h=c48149cf80c6582c2369bc7f8a33d794021d9dae Looks good and works for me - thanks Peter. Cheers, David ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-07-10 11:18 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-07-05 11:01 [PATCH] rules: block - add dm devices to whitelist David Disseldorp 2017-07-10 8:38 ` Lennart Poettering 2017-07-10 9:37 ` [systemd-devel] " David Disseldorp 2017-07-10 9:53 ` Lennart Poettering 2017-07-10 10:14 ` Peter Rajnoha 2017-07-10 10:47 ` Peter Rajnoha 2017-07-10 11:18 ` [systemd-devel] " David Disseldorp
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.