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