Dan Kenigsberg has submitted this change and it was merged.
Change subject: NetReload: Moved addNetwork logic to the configurator
......................................................................
NetReload: Moved addNetwork logic to the configurator
This change starts moving the logic of configNetwork API endpoints
towards the configurators.
configureBridge, configureVlan, configureBond and configureNic
after this patch have to take care of the underlying devices
explicitly in the configurator modules. However, the underlying
device handling will be abstracted to the network models in a
future patch. Thus, the code of configureBond for any configurator
such as Ifcfg is gonna be something like:
self.configWriter.addBonding(params)
for slave in bond.slaves:
slave.configure(params)
Change-Id: I877eddc07a17f0bdd376df7bab8b19ffe52f72ba
Signed-off-by: Antoni S. Puimedon <[email protected]>
---
M vdsm/configNetwork.py
M vdsm/netconf/ifcfg.py
2 files changed, 128 insertions(+), 74 deletions(-)
Approvals:
Antoni Segura Puimedon: Verified
Dan Kenigsberg: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/13268
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I877eddc07a17f0bdd376df7bab8b19ffe52f72ba
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <[email protected]>
Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Mark Wu <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
