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 <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to