Change in vdsm[master]: vdsm-tool: Add register verb

dougsland at redhat.com dougsland at redhat.com
Tue May 19 16:42:41 UTC 2015


Douglas Schilling Landgraf has posted comments on this change.

Change subject: vdsm-tool: Add register verb
......................................................................


Patch Set 9:

(2 comments)

https://gerrit.ovirt.org/#/c/40966/9/lib/vdsm/tool/register.py
File lib/vdsm/tool/register.py:

Line 355:             os.makedirs(_auth_keys_dir, 0o700)
Line 356:             self._silent_restorecon(_auth_keys_dir)
Line 357:             if utils.isOvirtNode():
Line 358:                 from ovirt.node.utils.fs import Config
Line 359:                 Config().persist(_auth_keys_dir)
os.chwon()
Line 360: 
Line 361:         res = self._execute_http_request(self.url_ssh_key)
Line 362:         with tempfile.NamedTemporaryFile(
Line 363:             dir=_auth_keys_dir,


Line 372:                 if content not in f_w.read():
Line 373:                     f_w.write(content)
Line 374:                     os.chmod(_auth_keys, 0o600)
Line 375:                     self._silent_restorecon(_auth_keys)
Line 376: 
os.chwon()
Line 377:         os.unlink(f.name)
Line 378:         if utils.isOvirtNode():
Line 379:             from ovirt.node.utils.fs import Config
Line 380:             Config().persist(_auth_keys)


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ica800027beec1e5a20165bb5e1e78baf9283c1da
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list