Change in vdsm[master]: vdsm.host.uuid: eliminate obsoleted argument "legacy"

danken at redhat.com danken at redhat.com
Sun Mar 6 13:35:57 UTC 2016


Dan Kenigsberg has uploaded a new change for review.

Change subject: vdsm.host.uuid: eliminate obsoleted argument "legacy"
......................................................................

vdsm.host.uuid: eliminate obsoleted argument "legacy"

"legacy" had no meaning since ovirt-3.6; as of commit e3409136 it cannot
even be passed to the function. It's time to eliminate it.

Change-Id: I4113231e46208d79fe53ce0577615832f916012c
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1314705
Signed-off-by: Dan Kenigsberg <danken at redhat.com>
---
M lib/vdsm/host.py
M lib/vdsm/tool/register.py
2 files changed, 2 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/84/54384/1

diff --git a/lib/vdsm/host.py b/lib/vdsm/host.py
index e7f7b86..1cf633a 100644
--- a/lib/vdsm/host.py
+++ b/lib/vdsm/host.py
@@ -29,11 +29,8 @@
 
 
 @memoized
-def uuid(legacy=False):
+def uuid():
     host_UUID = None
-
-    if legacy:
-        raise NotImplementedError
 
     try:
         if os.path.exists(constants.P_VDSM_NODE_ID):
diff --git a/lib/vdsm/tool/register.py b/lib/vdsm/tool/register.py
index 6ed5322..dcd9d19 100644
--- a/lib/vdsm/tool/register.py
+++ b/lib/vdsm/tool/register.py
@@ -257,7 +257,7 @@
         if self.vdsm_uuid:
             self.uuid = self.vdsm_uuid
         else:
-            self.uuid = host.uuid(legacy=False)
+            self.uuid = host.uuid()
 
         self.url_reg += "&uniqueId={u}".format(u=self.uuid)
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4113231e46208d79fe53ce0577615832f916012c
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>


More information about the vdsm-patches mailing list