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