Change in vdsm[master]: janitorial: drop another unused constant

2017-05-29 Thread Code Review 
From Dan Kenigsberg :

Dan Kenigsberg has submitted this change and it was merged.

Change subject: janitorial: drop another unused constant
..


janitorial: drop another unused constant

Prehistoric commit 806420 stopped using this constant.

Change-Id: I1fbdeaf372f8a133f6fad05ac09b83fe739f54c1
Signed-off-by: Dan Kenigsberg 
---
M lib/vdsm/storage/lvm.py
1 file changed, 0 insertions(+), 1 deletion(-)

Approvals:
  Nir Soffer: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Verified



-- 
To view, visit https://gerrit.ovirt.org/77431
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1fbdeaf372f8a133f6fad05ac09b83fe739f54c1
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
___
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Change in vdsm[master]: janitorial: drop another unused constant

2017-05-28 Thread Code Review 
From Dan Kenigsberg :

Dan Kenigsberg has posted comments on this change.

Change subject: janitorial: drop another unused constant
..


Patch Set 1: Verified+1

unit tests passing is enough of a verification for this.

-- 
To view, visit https://gerrit.ovirt.org/77431
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1fbdeaf372f8a133f6fad05ac09b83fe739f54c1
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Change in vdsm[master]: janitorial: drop another unused constant

2017-05-28 Thread Code Review 
From Dan Kenigsberg :

Dan Kenigsberg has uploaded a new change for review.

Change subject: janitorial: drop another unused constant
..

janitorial: drop another unused constant

Prehistoric commit 806420 stopped using this constant.

Change-Id: I1fbdeaf372f8a133f6fad05ac09b83fe739f54c1
Signed-off-by: Dan Kenigsberg 
---
M lib/vdsm/storage/lvm.py
1 file changed, 0 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/31/77431/1

diff --git a/lib/vdsm/storage/lvm.py b/lib/vdsm/storage/lvm.py
index d0e8ca4..b01bb94 100644
--- a/lib/vdsm/storage/lvm.py
+++ b/lib/vdsm/storage/lvm.py
@@ -85,7 +85,6 @@
 # VG states
 VG_OK = "OK"
 VG_PARTIAL = "PARTIAL"
-VG_UNKNOWN = "UNKNOWN"
 
 SEPARATOR = "|"
 LVM_NOBACKUP = ("--autobackup", "n")


-- 
To view, visit https://gerrit.ovirt.org/77431
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1fbdeaf372f8a133f6fad05ac09b83fe739f54c1
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg 
___
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org