Change in vdsm[ovirt-3.5]: network: models: tag non-VDSM devices used by VDSM

ybronhei at redhat.com ybronhei at redhat.com
Thu Mar 19 12:55:31 UTC 2015


Yaniv Bronhaim has submitted this change and it was merged.

Change subject: network: models: tag non-VDSM devices used by VDSM
......................................................................


network: models: tag non-VDSM devices used by VDSM

When we setup network over pre-existing vlaned bond, models.py skip
configuration of pre-existing devices under a vlan (because they are not
to be changed) in order to not ifdown possibly used devices. Due to this
behaviour we don't persist mentioned networks and therefore we are not
able to restore them.

Now we check if device is a part of vdsm, if not, we configure and
persist it.

Change-Id: I4d3ed2b45d618d404bb340100ca07bc57b2f1136
Signed-off-by: Petr Horáček <phoracek at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/38572
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1154399
Reviewed-on: https://gerrit.ovirt.org/38771
Reviewed-by: Ido Barkan <ibarkan at redhat.com>
---
M tests/functional/networkTests.py
M vdsm/network/models.py
2 files changed, 55 insertions(+), 2 deletions(-)

Approvals:
  Ido Barkan: Looks good to me, but someone else must approve
  Petr Horáček: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4d3ed2b45d618d404bb340100ca07bc57b2f1136
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <toni+ovirt at midokura.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list