[node-patches] Change in ovirt-node[master]: Don't put output to the screen in ovirt-cleanup

rbarry at redhat.com rbarry at redhat.com
Fri Sep 26 14:30:27 UTC 2014


Ryan Barry has uploaded a new change for review.

Change subject: Don't put output to the screen in ovirt-cleanup
......................................................................

Don't put output to the screen in ovirt-cleanup

We know that `yes` will get a broken pipe and that we
intentionally disabled LVM locking in ovirt-cleanup. We don't need
warnings about it.

Change-Id: Ia8a9f96e8392b469794ae84389ef8c6cbe694ff1
Signed-off-by: Ryan Barry <rbarry at redhat.com>
---
M dracut/ovirt-cleanup.sh
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/08/33408/1

diff --git a/dracut/ovirt-cleanup.sh b/dracut/ovirt-cleanup.sh
index 7a44eff..a850885 100755
--- a/dracut/ovirt-cleanup.sh
+++ b/dracut/ovirt-cleanup.sh
@@ -185,10 +185,10 @@
                 fi
             done
             info "Found and removing vg: $vg"
-            yes | lvm vgremove -ff "$vg"
+            yes 2>/dev/null | lvm vgremove -ff "$vg" 2>/dev/null
         fi
         info "Found and removing pv: $pv"
-        yes | lvm pvremove -ff "$pv"
+        yes 2>/dev/null | lvm pvremove -ff "$pv" 2>/dev/null
     done
     IFS=,
     if [ -f "/etc/lvm/lvm.conf" ]; then


-- 
To view, visit http://gerrit.ovirt.org/33408
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia8a9f96e8392b469794ae84389ef8c6cbe694ff1
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <rbarry at redhat.com>



More information about the node-patches mailing list