Change in vdsm[ovirt-3.6]: net: Clean running config on unified restoration failure

fromani at redhat.com fromani at redhat.com
Tue Apr 26 08:49:32 UTC 2016


Francesco Romani has submitted this change and it was merged.

Change subject: net: Clean running config on unified restoration failure
......................................................................


net: Clean running config on unified restoration failure

When the unified restoration process fails, running config (file) may
contain networks/bonds that are not in kernel.
When an attempt is made to re-run the unified restoration (without a
boot), the process will fail to remove the mentioned networks/bonds.

In this patch, the running config will be cleared if the restoration
fails, allowing the following attempts to try and setup the network
again.

Change-Id: Ie43da8ca26ef0b0e92ec66e1a481a843aea0487f
Bug-Url: https://bugzilla.redhat.com/1323782
Signed-off-by: Edward Haas <edwardh at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/56257
Continuous-Integration: Jenkins CI
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/56322
---
M vdsm/vdsm-restore-net-config
1 file changed, 25 insertions(+), 13 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved
  Edward Haas: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie43da8ca26ef0b0e92ec66e1a481a843aea0487f
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Edward Haas <edwardh at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list