Hello community,
here is the log from the commit of package lvm2 for openSUSE:Factory checked in
at 2017-02-04 15:18:41
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/lvm2 (Old)
and /work/SRC/openSUSE:Factory/.lvm2.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "lvm2"
Changes:
--------
--- /work/SRC/openSUSE:Factory/lvm2/device-mapper.changes 2017-01-24
10:27:54.413148588 +0100
+++ /work/SRC/openSUSE:Factory/.lvm2.new/device-mapper.changes 2017-02-04
18:01:27.871985763 +0100
@@ -1,0 +2,6 @@
+Fri Jan 20 09:46:27 UTC 2017 - tchva...@suse.com
+
+- Remove special case for md (bsc#1012973)
+ * remove-special-case-for-md-in-69-dm-lvm-metadata.rul.patch
+
+-------------------------------------------------------------------
lvm2-clvm.changes: same change
lvm2.changes: same change
New:
----
remove-special-case-for-md-in-69-dm-lvm-metadata.rul.patch
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ device-mapper.spec ++++++
--- /var/tmp/diff_new_pack.fIiBej/_old 2017-02-04 18:01:28.495896565 +0100
+++ /var/tmp/diff_new_pack.fIiBej/_new 2017-02-04 18:01:28.495896565 +0100
@@ -78,6 +78,8 @@
Patch2010: 69-dm-lvm-metad.rules-Do-not-process-rules-for-multi.patch
#PATCH-FIX-SUSE, bnc#950089
Patch2011: lvm2-testsuite.patch
+# Remove special case for the md rules
+Patch2012: remove-special-case-for-md-in-69-dm-lvm-metadata.rul.patch
### COMMON-PATCH-END ###
BuildRequires: gcc-c++
BuildRequires: pkgconfig
@@ -154,6 +156,7 @@
%patch2009 -p1
%patch2010 -p1
%patch2011 -p1
+%patch2012 -p1
### COMMON-PREP-END ###
%build
lvm2-clvm.spec: same change
++++++ lvm2.spec ++++++
--- /var/tmp/diff_new_pack.fIiBej/_old 2017-02-04 18:01:28.559887417 +0100
+++ /var/tmp/diff_new_pack.fIiBej/_new 2017-02-04 18:01:28.563886845 +0100
@@ -79,6 +79,8 @@
Patch2010: 69-dm-lvm-metad.rules-Do-not-process-rules-for-multi.patch
#PATCH-FIX-SUSE, bnc#950089
Patch2011: lvm2-testsuite.patch
+# Remove special case for the md rules
+Patch2012: remove-special-case-for-md-in-69-dm-lvm-metadata.rul.patch
### COMMON-PATCH-END ###
BuildRequires: gcc-c++
BuildRequires: libaio-devel
@@ -173,6 +175,7 @@
%patch2009 -p1
%patch2010 -p1
%patch2011 -p1
+%patch2012 -p1
### COMMON-PREP-END ###
%build
++++++ remove-special-case-for-md-in-69-dm-lvm-metadata.rul.patch ++++++
>From 2759c9fa04af726d2777db142806287256bd9cc5 Mon Sep 17 00:00:00 2001
From: NeilBrown <ne...@suse.com>
Date: Wed, 4 Jan 2017 14:30:52 +1100
Subject: [PATCH] Remove special-case for md in 69-dm-lvm-metadata.rules
This special casing brings no value. It appears to attempt to
determine if the array is active yet or not, and to skip
processing if the array has not yet been started.
However, if the array hasn't been started, then "blkid" will
not have been able to read a signature, so:
ENV{ID_FS_TYPE}!="LVM2_member|LVM1_member", GOTO="lvm_end"
will have caused all this code to be skipped.
Further, this code causes incorrect behaviour in at least one case.
It assumes that the first "add" event should be ignored, as it will be
followed by a "change" event which indicates the array coming on line.
This is consistent with how the kernel sends events, but not always
consistent with how this script sees event.
Specifically: if the initrd has "mdadm" support installed, but not
"lvm2" support, then the initial "add" and "change" events will
happen while the initrd is in charge and this file is not available.
Once the root filesystem is mountd, this file will be available
and "udevadm trigger --action=add" will be run.
So the first and only event seen by this script for an md device will be
"add", and it will incorrectly ignore it.
It is probable that the special handling for "loop" should be removed as
well, but I have not actually seen that cause a problem, so I'm
leaving it unchanged.
Signed-off-by: NeilBrown <ne...@suse.com>
---
udev/69-dm-lvm-metad.rules.in | 10 ----------
1 file changed, 10 deletions(-)
diff --git a/udev/69-dm-lvm-metad.rules.in b/udev/69-dm-lvm-metad.rules.in
index bd75fc8..db213ed 100644
--- a/udev/69-dm-lvm-metad.rules.in
+++ b/udev/69-dm-lvm-metad.rules.in
@@ -50,16 +50,6 @@ KERNEL!="dm-[0-9]*", GOTO="next"
ENV{DM_UDEV_PRIMARY_SOURCE_FLAG}=="1", ENV{DM_ACTIVATION}=="1", GOTO="lvm_scan"
GOTO="lvm_end"
-# MD device:
-LABEL="next"
-KERNEL!="md[0-9]*", GOTO="next"
-IMPORT{db}="LVM_MD_PV_ACTIVATED"
-ACTION=="add", ENV{LVM_MD_PV_ACTIVATED}=="1", GOTO="lvm_scan"
-ACTION=="change", ENV{LVM_MD_PV_ACTIVATED}!="1", TEST=="md/array_state",
ENV{LVM_MD_PV_ACTIVATED}="1", GOTO="lvm_scan"
-ACTION=="add", KERNEL=="md[0-9]*p[0-9]*", GOTO="lvm_scan"
-ENV{LVM_MD_PV_ACTIVATED}!="1", ENV{SYSTEMD_READY}="0"
-GOTO="lvm_end"
-
# Loop device:
LABEL="next"
KERNEL!="loop[0-9]*", GOTO="next"
--
2.10.2
