[Bug 1046591] New: [lvm test] thin-overprovisioning.sh failed - openSUSE Bugs - openSUSE Mailing Lists

newer
[Bug 1047729] Plasma does not...

[Bug 1046591] New: [lvm test] thin-overprovisioning.sh failed

older
[Bug 1047729] Plasma does not...

bugzilla_noreply＠novell.com

29 Jun 2017 29 Jun '17

08:32

http://bugzilla.suse.com/show_bug.cgi?id=1046591 Bug ID: 1046591 Summary: [lvm test] thin-overprovisioning.sh failed Classification: openSUSE Product: openSUSE Tumbleweed Version: Current Hardware: Other OS: Other Status: NEW Severity: Normal Priority: P5 - None Component: Basesystem Assignee: bnc-team-screening@forge.provo.novell.com Reporter: zren@suse.com QA Contact: qa-bugs@suse.de Found By: --- Blocker: --- Created attachment 730693 --> http://bugzilla.suse.com/attachment.cgi?id=730693&action=edit log for thin-overprovisioning.sh # lvm2-testsuite --flavours udev-lvmetad --only thin-overprovisioning.sh ... ### 1 tests: 0 passed, 0 skipped, 0 timed out, 0 warned, 1 failed -- You are receiving this mail because: You are on the CC list for the bug.

Attachments:

attachment.htm (text/html — 2.5 KB)

Reply

Sign in to reply online Use email software

Show replies by date

bugzilla_noreply＠novell.com

29 Jun 29 Jun

08:34

New subject: [Bug 1046591] [lvm test] thin-overprovisioning.sh failed

http://bugzilla.suse.com/show_bug.cgi?id=1046591 zhen ren changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CONFIRMED Assignee|bnc-team-screening@forge.pr |zren@suse.com |ovo.novell.com | -- You are receiving this mail because: You are on the CC list for the bug.

Reply

Sign in to reply online Use email software

bugzilla_noreply＠novell.com

30 Jun 30 Jun

08:00

New subject: [Bug 1046591] [lvm test] thin-overprovisioning.sh failed

http://bugzilla.suse.com/show_bug.cgi?id=1046591 http://bugzilla.suse.com/show_bug.cgi?id=1046591#c1 --- Comment #1 from zhen ren --- The pool devnode link is expected to be gone, after the pool is being used, i.e., a thin volume is created in this pool. """ [ 0:04] not grep "WARNING: Sum" out [ 0:04] #thin-overprovisioning.sh:35+ not grep 'WARNING: Sum' out [ 0:04] # check again link node is now gone for a USED thin-pool [ 0:04] test ! -e "$DM_DEV_DIR/$vg/pool" [ 0:04] #thin-overprovisioning.sh:37+ test '!' -e /dev/@PREFIX@vg/pool [ 0:04] set +vx; STACKTRACE; set -vx [ 0:04] ##thin-overprovisioning.sh:37+ set +vx [ 0:04] ## - /usr/share/lvm2-testsuite/shell/thin-overprovisioning.sh:125 [ 0:04] ## 0 stacktrace() called from lib/utils:125 [ 0:04] ## 1 STACKTRACE() called from lib/utils:37 """ -- You are receiving this mail because: You are on the CC list for the bug.

Reply

Sign in to reply online Use email software

bugzilla_noreply＠novell.com

3 Jul 3 Jul

07:26

New subject: [Bug 1046591] [lvm test] thin-overprovisioning.sh failed

http://bugzilla.suse.com/show_bug.cgi?id=1046591 http://bugzilla.suse.com/show_bug.cgi?id=1046591#c2 --- Comment #2 from zhen ren --- Created attachment 730974 --> http://bugzilla.suse.com/attachment.cgi?id=730974&action=edit This patch causes this testing failure - Testing with this patch: # vgcreate testvg /dev/loop0 Volume group "testvg" successfully created # lvcreate -L30 -T testvg/pool Logical volume "pool" created. # ll /dev/testvg/ total 0 lrwxrwxrwx 1 root root 7 Jul 3 15:16 pool -> ../dm-2 lrwxrwxrwx 1 root root 7 Jul 3 15:16 pool_tdata -> ../dm-1 lrwxrwxrwx 1 root root 7 Jul 3 15:16 pool_tmeta -> ../dm-0 # lvcreate -n thin1 -V30 testvg/pool Logical volume "thin1" created. # ll /dev/testvg/ total 0 lrwxrwxrwx 1 root root 7 Jul 3 15:17 pool -> ../dm-3 lrwxrwxrwx 1 root root 7 Jul 3 15:17 pool_tdata -> ../dm-1 lrwxrwxrwx 1 root root 7 Jul 3 15:17 pool_tmeta -> ../dm-0 lrwxrwxrwx 1 root root 7 Jul 3 15:17 thin1 -> ../dm-4 - Testing without this patch # vgcreate testvg /dev/loop0 Volume group "testvg" successfully created # lvcreate -L30 -T testvg/pool Logical volume "pool" created. # ll /dev/testvg/ total 0 lrwxrwxrwx 1 root root 7 Jul 3 15:24 pool -> ../dm-2 # lvcreate -n thin1 -V30 testvg/pool Logical volume "thin1" created. # ll /dev/testvg/ total 0 lrwxrwxrwx 1 root root 7 Jul 3 15:24 thin1 -> ../dm-4 -- You are receiving this mail because: You are on the CC list for the bug.

Reply

Sign in to reply online Use email software

bugzilla_noreply＠novell.com

07:30

New subject: [Bug 1046591] [lvm test] thin-overprovisioning.sh failed

http://bugzilla.suse.com/show_bug.cgi?id=1046591 zhen ren changed: What |Removed |Added ---------------------------------------------------------------------------- Status|CONFIRMED |IN_PROGRESS CC| |hare@suse.com -- You are receiving this mail because: You are on the CC list for the bug.

Reply

Sign in to reply online Use email software

bugzilla_noreply＠novell.com

4 Jul 4 Jul

06:17

New subject: [Bug 1046591] [lvm test] thin-overprovisioning.sh failed

http://bugzilla.suse.com/show_bug.cgi?id=1046591 http://bugzilla.suse.com/show_bug.cgi?id=1046591#c4 zhen ren changed: What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(hare@suse.com) --- Comment #4 from zhen ren --- DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG is set to indicate not creating symlink like /dev/${vg_name}/${lv_name} for LV that should not be visible to user, like a thin pool LV. lib/activate/dev_manager.c : _get_udev_flags() """ 1665 /* 1666 * Is this top-level and visible device? 1667 * If not, create just the /dev/mapper content. 1668 */ 1669 /* FIXME: add target's method for this */ 1670 if (lv_is_new_thin_pool(lv)) 1671 /* New thin-pool is regular LV with -tpool UUID suffix. */ 1672 udev_flags |= DM_UDEV_DISABLE_DISK_RULES_FLAG | 1673 DM_UDEV_DISABLE_OTHER_RULES_FLAG; 1674 else if (layer || !lv_is_visible(lv) || lv_is_thin_pool(lv)) 1675 udev_flags |= DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG | 1676 DM_UDEV_DISABLE_DISK_RULES_FLAG | 1677 DM_UDEV_DISABLE_OTHER_RULES_FLAG; """ udev/11-dm-lvm.rules.in: """ 38 ENV{DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG}=="1", GOTO="lvm_end" ... 46 # Create symlinks for top-level devices only. 47 ENV{DM_VG_NAME}=="?*", ENV{DM_LV_NAME}=="?*", SYMLINK+="$env{DM_VG_NAME}/$env{DM_LV_NAME}", GOTO="lvm_end" """ But, this patch "Subject: [PATCH] 10-dm.rules: Reset state variables for spurious events" cleared DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG by: """ +ENV{DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG}="" """ @Hannes, this testcase will pass if I remove the line above from your patch. But, I'm not sure if it's safe. What do you think? -- You are receiving this mail because: You are on the CC list for the bug.

Reply

Sign in to reply online Use email software

bugzilla_noreply＠novell.com

5 Jul 5 Jul

06:09

New subject: [Bug 1046591] [lvm test] thin-overprovisioning.sh failed

http://bugzilla.suse.com/show_bug.cgi?id=1046591 http://bugzilla.suse.com/show_bug.cgi?id=1046591#c5 Hannes Reinecke changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(hare@suse.com) | --- Comment #5 from Hannes Reinecke --- (In reply to zhen ren from comment #4)

DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG is set to indicate not creating symlink like /dev/${vg_name}/${lv_name} for LV that should not be visible to user, like a thin pool LV.

lib/activate/dev_manager.c : _get_udev_flags() """ 1665 /* 1666 * Is this top-level and visible device? 1667 * If not, create just the /dev/mapper content. 1668 */ 1669 /* FIXME: add target's method for this */ 1670 if (lv_is_new_thin_pool(lv)) 1671 /* New thin-pool is regular LV with -tpool UUID suffix. */ 1672 udev_flags |= DM_UDEV_DISABLE_DISK_RULES_FLAG | 1673 DM_UDEV_DISABLE_OTHER_RULES_FLAG; 1674 else if (layer || !lv_is_visible(lv) || lv_is_thin_pool(lv)) 1675 udev_flags |= DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG | 1676 DM_UDEV_DISABLE_DISK_RULES_FLAG | 1677 DM_UDEV_DISABLE_OTHER_RULES_FLAG; """

udev/11-dm-lvm.rules.in: """ 38 ENV{DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG}=="1", GOTO="lvm_end" ... 46 # Create symlinks for top-level devices only. 47 ENV{DM_VG_NAME}=="?*", ENV{DM_LV_NAME}=="?*", SYMLINK+="$env{DM_VG_NAME}/$env{DM_LV_NAME}", GOTO="lvm_end" """

But, this patch "Subject: [PATCH] 10-dm.rules: Reset state variables for spurious events" cleared DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG by: """ +ENV{DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG}="" """

@Hannes, this testcase will pass if I remove the line above from your patch. But, I'm not sure if it's safe. What do you think?

Hmm. Should be okay, indeed. The DM_UDEV_* variables are magic thingies anyway, so I'm not _that_ surprised that this error happened. -- You are receiving this mail because: You are on the CC list for the bug.

Reply

Sign in to reply online Use email software

bugzilla_noreply＠novell.com

06:13

New subject: [Bug 1046591] [lvm test] thin-overprovisioning.sh failed

http://bugzilla.suse.com/show_bug.cgi?id=1046591 http://bugzilla.suse.com/show_bug.cgi?id=1046591#c6 --- Comment #6 from zhen ren ---

...
But, this patch "Subject: [PATCH] 10-dm.rules: Reset state variables for spurious events" cleared DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG by: """ +ENV{DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG}="" """

@Hannes, this testcase will pass if I remove the line above from your patch. But, I'm not sure if it's safe. What do you think?

Hmm. Should be okay, indeed.

Just remove the DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG line, or drop this patch?

The DM_UDEV_* variables are magic thingies anyway, so I'm not _that_ surprised that this error happened.

Yes. Thanks! -- You are receiving this mail because: You are on the CC list for the bug.

Reply

Sign in to reply online Use email software

bugzilla_noreply＠novell.com

11 Jul 11 Jul

06:29

New subject: [Bug 1046591] [lvm test] thin-overprovisioning.sh failed

http://bugzilla.suse.com/show_bug.cgi?id=1046591 http://bugzilla.suse.com/show_bug.cgi?id=1046591#c7 zhen ren changed: What |Removed |Added ---------------------------------------------------------------------------- Status|IN_PROGRESS |RESOLVED Resolution|--- |FIXED --- Comment #7 from zhen ren --- Dropped 10-dm.rules-Reset-state-variable-for-spurious-events.patch in factory LVM. -- You are receiving this mail because: You are on the CC list for the bug.

Reply

Sign in to reply online Use email software

2491

Age (days ago)

2503

Last active (days ago)

Download

8 comments

1 participants

tags

participants (1)

bugzilla_noreply＠novell.com