Yaniv Bronhaim has posted comments on this change.

Change subject: [WIP] Improving vdsm-tool commands - retrieving host uuid
......................................................................


Patch Set 3: (2 inline comments)

....................................................
File lib/vdsm/tool/vdsm-id.py
Line 20: from vdsm.utils import getHostUUID
Line 21: from vdsm.tool import expose
Line 22: 
Line 23: 
Line 24: @expose("vdsm-id")
When performing this command you should be already connected to the host by 
ssh, ssl certs or other parameters are not required in that case due to the 
local run of the command. All vdsm-tool commands are separated from vdsm itself 
and run locally on the host, the only remote part in this specific use is the 
ssh connection to the host..
Line 25: def getUUID():
Line 26:     """
Line 27:     Printing host uuid
Line 28:     """


Line 25: def getUUID():
Line 26:     """
Line 27:     Printing host uuid
Line 28:     """
Line 29:     print getHostUUID()
I forgot about the mac .. Well, legacy it is, I'll add it. Thanks.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia8cab7f55697e07479a4f35c161bd836925374e8
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to