Sandro Bonazzola has posted comments on this change.

Change subject: packaging: setup: suppress logging of VM password
......................................................................


Patch Set 2: Verified-1 -Code-Review

(1 comment)

http://gerrit.ovirt.org/#/c/27591/2/src/plugins/ovirt-hosted-engine-setup/vm/runvm.py
File src/plugins/ovirt-hosted-engine-setup/vm/runvm.py:

Line 51:         self.environment.setdefault(
Line 52:             ohostedcons.VMEnv.VM_PASSWD,
Line 53:             self._generateTempVncPassword()
Line 54:         )
Line 55:         self.environment[otopicons.CoreEnv.LOG_FILTER_KEYS].append(
missing import for otopicons
Line 56:             ohostedcons.VMEnv.VM_PASSWD
Line 57:         )
Line 58:         self.environment.setdefault(
Line 59:             ohostedcons.VMEnv.VM_PASSWD_VALIDITY_SECS,


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I92ef0a1516d7a6dd70d3fd53ceac3c71ce1eaf24
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to