[node-patches] Change in ovirt-node[master]: Unpersist and repersist snmpd.conf so it can be moved

[fabiand at redhat.com]

Fabian Deutsch has submitted this change and it was merged.

Change subject: Unpersist and repersist snmpd.conf so it can be moved
......................................................................


Unpersist and repersist snmpd.conf so it can be moved

EL 7.1 (and probably EL6 at some point) now tries to move
/var/lib/net-snmp/snmpd.conf to snmpd.conf.0, probably as a
backup. This fails, since it's persisted (and bind mounted).

Unpersist it, then re-persist it when changes which require
restarting snmpd happen. Also, rework the logic of enable_snmpd
a little so we're not starting and stopping and restarting the
service or touching config files to change a password through
snmpusm

Change-Id: I0ff002717a8bf2744efa995f2e7e09a60b1eaede
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1232412
Signed-off-by: Ryan Barry <rbarry at redhat.com>
---
M src/ovirt/node/setup/snmp/snmp_model.py
1 file changed, 26 insertions(+), 26 deletions(-)

Approvals:
  Fabian Deutsch: Looks good to me, approved; Passed CI tests
  Ryan Barry: Verified



-- 
To view, visit https://gerrit.ovirt.org/42771
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I0ff002717a8bf2744efa995f2e7e09a60b1eaede
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: automation at ovirt.org