Dan Kenigsberg has posted comments on this change.
Change subject: vdsm-tool: Ensure that the vdsm uuid is a string ......................................................................
Patch Set 1: Code-Review-1
(1 comment)
https://gerrit.ovirt.org/#/c/46970/1/lib/vdsm/tool/register.py File lib/vdsm/tool/register.py:
Line 271: os.unlink(__VDSM_ID) Line 272: Line 273: if not os.path.exists(__VDSM_ID): Line 274: with open(__VDSM_ID, 'wt') as f: Line 275: f.write(str(self.uuid)) how could it ever NOT be a string? whomever attempted to set it to None or whatnot should be fixed, we need to add a blind conversion here. Line 276: Line 277: if utils.isOvirtNode(): Line 278: from ovirt.node.utils.fs import Config Line 279: Config().persist(__VDSM_ID)