Alon Bar-Lev has posted comments on this change.

Change subject: packaging: setup: database: suppress password based on env key
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.ovirt.org/#/c/25347/3/packaging/setup/plugins/ovirt-engine-rename/core/pki.py
File packaging/setup/plugins/ovirt-engine-rename/core/pki.py:

Line 63:         self.environment[
Line 64:             otopicons.CoreEnv.LOG_FILTER_KEYS
Line 65:         ].append(
Line 66:             osetupcons.PKIEnv.STORE_PASS
Line 67:         )
> > this is rename and the other is setup we need this in both.
we do not have common for pki, it looks quite strange to add it just for 
that... but I can do that if you want.

for the suppress, we can use only the log filter keys, but have no time to 
re-verify can you please on "fast track"?
Line 68: 
Line 69:     @plugin.event(
Line 70:         stage=plugin.Stages.STAGE_INIT,
Line 71:     )


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia4d1ad9b0acda23a59f55d1653a542c9ea038754
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <d...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to