Yeela Kaplan has uploaded a new change for review.
Change subject: lvm: remove unused add/remove tags
......................................................................
lvm: remove unused add/remove tags
Change-Id: Ibffb9138cb28e150f18e2283c24a60c165cde337
Signed-off-by: Yeela Kaplan <ykaplan(a)redhat.com>
---
M vdsm/storage/lvm.py
M vdsm/storage/storage_exception.py
2 files changed, 0 insertions(+), 35 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/13/25013/1
diff --git a/vdsm/storage/lvm.py b/vdsm/storage/lvm.py
index 24d023d..7e7ad06 100644
--- a/vdsm/storage/lvm.py
+++ b/vdsm/storage/lvm.py
@@ -1339,33 +1339,8 @@
(vgName, ", ".join(delTags), ", ".join(addTags),
err[-1]))
-def addVGTag(vgName, tag):
- _lvminfo._invalidatevgs(vgName)
- cmd = ["vgchange", "--addtag", tag, vgName]
- rc, out, err = _lvminfo.cmd(cmd, _lvminfo._getVGDevs((vgName, )))
- if rc != 0:
- raise se.VolumeGroupAddTagError("Failed adding tag %s to VG %s." %
- (tag, vgName))
-
-
-def remVGTag(vgName, tag):
- _lvminfo._invalidatevgs(vgName)
- cmd = ["vgchange", "--deltag", tag, vgName]
- rc, out, err = _lvminfo.cmd(cmd, _lvminfo._getVGDevs((vgName, )))
- if rc != 0:
- raise se.VolumeGroupRemoveTagError(vgName)
-
-
def replaceVGTag(vg, oldTag, newTag):
changeVGTags(vg, [oldTag], [newTag])
-
-
-def addVGTags(vgName, tags):
- changeVGTags(vgName, addTags=tags)
-
-
-def remVGTags(vgName, tags):
- changeVGTags(vgName, delTags=tags)
def getFirstExt(vg, lv):
diff --git a/vdsm/storage/storage_exception.py b/vdsm/storage/storage_exception.py
index ae68a6f..0317794 100644
--- a/vdsm/storage/storage_exception.py
+++ b/vdsm/storage/storage_exception.py
@@ -1249,16 +1249,6 @@
message = "Read Volume Group's tag error"
-class VolumeGroupAddTagError(StorageException):
- code = 511
- message = "Add tag to Volume Group error"
-
-
-class VolumeGroupRemoveTagError(StorageException):
- code = 512
- message = "Remove tag from Volume Group error"
-
-
class VolumeGroupScanError(StorageException):
code = 513
message = "Volume Group scanning error"
--
To view, visit
http://gerrit.ovirt.org/25013
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibffb9138cb28e150f18e2283c24a60c165cde337
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan <ykaplan(a)redhat.com>