Itamar Heim has submitted this change and it was merged.
Change subject: packaging: setup: Do not log passwords
......................................................................
packaging: setup: Do not log passwords
Most of the code already refrains from logging passwords.
This change also makes upgrade from legacy 3.2 code not log the db
password and also makes otopi not log db password and admin password
in dumpEnvironment even if set from an answer file etc.
Depends on
http://gerrit.ovirt.org/19776 .
Bug-Url:
https://bugzilla.redhat.com/1014115
Change-Id: I81c7371eaf930b3cc21861deb620c61891ad4d1f
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
---
M ovirt-engine.spec.in
M packaging/setup/plugins/ovirt-engine-common/db/connection.py
M packaging/setup/plugins/ovirt-engine-setup/config/options.py
M packaging/setup/plugins/ovirt-engine-setup/legacy/database.py
4 files changed, 27 insertions(+), 1 deletion(-)
Approvals:
Itamar Heim: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/19938
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I81c7371eaf930b3cc21861deb620c61891ad4d1f
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3
Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server