Change in vdsm[master]: tc: Integrate Host QoS with network addition via setupNetworks

danken at redhat.com danken at redhat.com
Mon Sep 15 10:53:01 UTC 2014


Dan Kenigsberg has submitted this change and it was merged.

Change subject: tc: Integrate Host QoS with network addition via setupNetworks
......................................................................


tc: Integrate Host QoS with network addition via setupNetworks

This patch makes it possible to leverage the network.tc package to
configure outbound Host network Quality of Service. The following
patch should add support for deleting the QoS entities when deleting
the associated network.

It is not legal to configure a shaped network over a device that
backs a non shaped network and vice-versa.

One can configure it just like this:

    In [1]:from vdsm import vdscli
    In [2]: c = vdscli.connect()
    In [3]: c.setupNetworks({
        'bar':{'nic': 'eth1',
        'qosOutbound': {
            'ls': {'m1': 4 * 1000 ** 2,
                   'd': 100 * 1000,
                   'm2': 3 * 1000 **2},
            'ul':{'m2': 8 * 1000 ** 2}},
        'defaultRoute': False,
        'bridged': False,
        'vlan': 11}}, {}, {'connectivityCheck': False})
    Out[3]: {'status': {'code': 0, 'message': 'Done'}}
    In [4]: c.getVdsCapabilities()['info']['networks']['bar']['qosOutbound']
    Out[4]:
    {'ls': {'d': 100000.0, 'm1': 4000000000.0, 'm2': 3000000.0},
     'ul': {'d': 0, 'm1': 0.0, 'm2': 8000000.0}}

Change-Id: I4a5378870a3dac9307a6eef8d9937a9c03a0c819
Signed-off-by: Antoni S. Puimedon <asegurap at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/30467
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M debian/vdsm.install
M tests/functional/networkTests.py
A tests/qosTests.py
M vdsm.spec.in
M vdsm/caps.py
M vdsm/network/api.py
M vdsm/network/configurators/Makefile.am
M vdsm/network/configurators/__init__.py
A vdsm/network/configurators/qos.py
M vdsm/rpc/vdsmapi-schema.json
10 files changed, 347 insertions(+), 10 deletions(-)

Approvals:
  Antoni Segura Puimedon: Verified
  Dan Kenigsberg: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/30467
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4a5378870a3dac9307a6eef8d9937a9c03a0c819
Gerrit-PatchSet: 18
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list