Tal Nisan has submitted this change and it was merged. Change subject: packaging: Update only invalid values in postgres.conf ...................................................................... packaging: Update only invalid values in postgres.conf Motivation: The updater of the conf was changing even valid config item. We want to leave those in place in case someone already took care of that and tweak the system correctly and not override valid values. Change: Make sure valid values are not counted for edition. Result: Valid pg values are not amended. If postgres had autovacuum_max_workers set to 8 and valid value is >= 6 then it is left untouched. Change-Id: Id06bce50b72544e2f9ee72942a3f6344393f87ce Bug-Url: https://bugzilla.redhat.com/1388433 Signed-off-by: Roy Golan <rgolan(a)redhat.com&gt; --- M packaging/setup/ovirt_engine_setup/engine_common/database.py 1 file changed, 2 insertions(+), 9 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, but someone else must approve Roy Golan: Verified -- To view, visit https://gerrit.ovirt.org/74572 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id06bce50b72544e2f9ee72942a3f6344393f87ce Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Roy Golan <rgolan(a)redhat.com&gt; Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com&gt; Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt; Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com&gt; Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com&gt; Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;