Dan Kenigsberg has submitted this change and it was merged.
Change subject: change qosOutbound for a nested 'hostQos':{'out':
bandwidth_params}
......................................................................
change qosOutbound for a nested 'hostQos':{'out': bandwidth_params}
The engine had some semantics associated to the original (and never
used, but shared with vnic traffic shaping) host QoS. That makes it
a good idea to take a sort of a clean-slate approach to this feature
that I did not originally take, but I do here in this patch.
The difference is that it used to be:
{'blue': {'nic': 'em1', 'qosOutbound': {'ls': 8 * 1024 ** 2}}}
and now it will be:
{'blue': {'nic': 'em1', 'hostQos': {'out': {'ls': 8 * 1024 ** 2}}}}
Change-Id: I158af66fb98e2c1384aaf1f9e90b208ab01097a5
Signed-off-by: Antoni S. Puimedon <asegu...@redhat.com>
Reviewed-on: http://gerrit.ovirt.org/33165
Tested-by: Lior Vernia <lver...@redhat.com>
Reviewed-by: Dan Kenigsberg <dan...@redhat.com>
---
M lib/vdsm/tool/unified_persistence.py
M tests/functional/networkTests.py
M vdsm/caps.py
M vdsm/network/api.py
M vdsm/network/configurators/__init__.py
M vdsm/rpc/vdsmapi-schema.json
6 files changed, 49 insertions(+), 32 deletions(-)
Approvals:
Dan Kenigsberg: Looks good to me, approved
Lior Vernia: Verified
--
To view, visit http://gerrit.ovirt.org/33165
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I158af66fb98e2c1384aaf1f9e90b208ab01097a5
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegu...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Ido Barkan <ibar...@redhat.com>
Gerrit-Reviewer: Lior Vernia <lver...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
