[node-patches] Change in ovirt-node[master]: Don't overwrite the entirety of kdump.conf when we save

fabiand at redhat.com fabiand at redhat.com
Tue Aug 5 16:10:45 UTC 2014


Fabian Deutsch has submitted this change and it was merged.

Change subject: Don't overwrite the entirety of kdump.conf when we save
......................................................................


Don't overwrite the entirety of kdump.conf when we save

Be smarter about writing kdump.conf -- only clear and set values
which may be set by the TUI so we don't stomp on other people.

Change-Id: Ic53bab0b3a5602ac71028db5835a69005ac85a0c
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1122923
Signed-off-by: Ryan Barry <rbarry at redhat.com>
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M src/ovirt/node/config/defaults.py
1 file changed, 46 insertions(+), 17 deletions(-)

Approvals:
  Fabian Deutsch: Verified; Looks good to me, approved
  Douglas Schilling Landgraf: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic53bab0b3a5602ac71028db5835a69005ac85a0c
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the node-patches mailing list