Change in vdsm[master]: QoS: Remove broken libvirt-based host network QoS

danken at redhat.com danken at redhat.com
Mon Sep 15 10:50:47 UTC 2014


Dan Kenigsberg has submitted this change and it was merged.

Change subject: QoS: Remove broken libvirt-based host network QoS
......................................................................


QoS: Remove broken libvirt-based host network QoS

Back when the current host network level QoSa was added, it was
thought that libvirt had a way of configuring and honoring the
Quality of Service definitions that we were giving it. Unfortunately
that was an unimplemented part of their api, which forced us to
design a new implementation with different semantics.

This patch removes most of the host based network QoS implementation
from the codebase to prepare vdsm for the following patches which
will use the new tc package to provide outbound host network QoS.

Change-Id: I7b82cc6b656fb6af8caa2940c9ee2d14287f3526
Signed-off-by: Antoni S. Puimedon <asegurap at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/30198
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M lib/vdsm/netinfo.py
M lib/vdsm/tool/unified_persistence.py
M tests/functional/networkTests.py
M tests/functional/utils.py
M tests/netconfTests.py
M tests/netinfoTests.py
M vdsm/network/api.py
M vdsm/network/configurators/__init__.py
M vdsm/network/configurators/ifcfg.py
M vdsm/network/configurators/iproute2.py
M vdsm/network/configurators/libvirt.py
M vdsm/rpc/vdsmapi-schema.json
12 files changed, 18 insertions(+), 174 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7b82cc6b656fb6af8caa2940c9ee2d14287f3526
Gerrit-PatchSet: 12
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: Livnat Peer <lpeer at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list