Dan Kenigsberg has posted comments on this change. Change subject: vdsm-tool command that retrives host uuid ......................................................................
Patch Set 1: I would prefer that you didn't submit this (2 inline comments) .................................................... Commit Message Line 3: AuthorDate: 2013-05-19 18:26:02 +0300 Line 4: Commit: Yaniv Bronhaim <ybron...@redhat.com> Line 5: CommitDate: 2013-05-19 18:26:20 +0300 Line 6: Line 7: vdsm-tool command that retrives host uuid retrives -> retrieves but more importantly - could you explain why do we need it? Line 8: Line 9: vdsm-tool vdsm-id returns host uuid without requiring running instance of Line 10: vdsm deamon. Line 11: .................................................... File lib/vdsm/tool/vdsm-id.py Line 1: # Copyright 2013 IBM, Inc. Have you left Red Hat? Line 2: # Line 3: # This program is free software; you can redistribute it and/or modify Line 4: # it under the terms of the GNU General Public License as published by Line 5: # the Free Software Foundation; either version 2 of the License, or -- 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: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@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