Change in vdsm[master]: net: ignore bondingOptions in persistent networks.

danken at redhat.com danken at redhat.com
Tue Jul 7 16:35:22 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: net: ignore bondingOptions in persistent networks.
......................................................................


net: ignore bondingOptions in persistent networks.

Up until https://gerrit.ovirt.org/#/c/42146/ bondingOptions were
mistakenly part of persistent network attributes. So, if we are
upgrading from an older enough version, we should drop this entry
from the copied persistent networks. If we don't, the comparison
between persisted and kernel config would needlessly fail.

Change-Id: I1c29d4e6c109d4fa7b2d293133b8551e423ec139
Signed-off-by: Ido Barkan <ibarkan at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/43233
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Continuous-Integration: Jenkins CI
---
M lib/vdsm/netconfpersistence.py
1 file changed, 6 insertions(+), 0 deletions(-)

Approvals:
  Ido Barkan: Verified
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved

Objections:
  Petr Horáček: I would prefer that you didn't submit this



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1c29d4e6c109d4fa7b2d293133b8551e423ec139
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list