Change in vdsm[master]: network: add special treatment for dpdk ports
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: network: add special treatment for dpdk ports
......................................................................
network: add special treatment for dpdk ports
When nics are bound to DPDK driver, instead of traditional kernel
driver, they have to be accessed in a different manner.
Netowrk processing driver is a user space driver and packet processing
is not taking place in kernel domain.
Hence, traditional tools for reading nic speed, getting information
about its state and similar actions are irrelevant.
This patch adds fake information for nics which are bound to DPDK.
Change-Id: I9376dd3ddc45a931dc286cc05d4e57016532058b
Signed-off-by: Irit Goihman <igoihman(a)redhat.com>
---
M lib/vdsm/network/ipwrapper.py
M lib/vdsm/network/link/dpdk.py
M lib/vdsm/network/link/iface.py
M lib/vdsm/network/netinfo/nics.py
M lib/vdsm/network/netswitch/configurator.py
M lib/vdsm/network/ovs/switch.py
M tests/network/netinfo_test.py
7 files changed, 58 insertions(+), 2 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Irit Goihman: Verified
Edward Haas: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/74079
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9376dd3ddc45a931dc286cc05d4e57016532058b
Gerrit-PatchSet: 31
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Irit Goihman <igoihman(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years
Change in vdsm[master]: ovs: introduce set_dpdk_bridge and set_dpdk_port commands
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: ovs: introduce set_dpdk_bridge and set_dpdk_port commands
......................................................................
ovs: introduce set_dpdk_bridge and set_dpdk_port commands
These commands set dpdk bridge and port types, enabling
support of ovs-dpdk setup.
Change-Id: Ib68532e59a99380d85f5f1ab27376c24972c3941
Signed-off-by: Irit Goihman <igoihman(a)redhat.com>
---
M lib/vdsm/network/ovs/driver/__init__.py
1 file changed, 6 insertions(+), 0 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Irit Goihman: Verified
Edward Haas: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/74752
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib68532e59a99380d85f5f1ab27376c24972c3941
Gerrit-PatchSet: 14
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Irit Goihman <igoihman(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years
Change in vdsm[master]: ovs: support creating an ovs-dpdk setup
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: ovs: support creating an ovs-dpdk setup
......................................................................
ovs: support creating an ovs-dpdk setup
In case that ovs bridge is created on top of a dpdk
nic, ovs bridge will be created with user space
forwarding plane, having the ability to communicate with
dpdk libraries.
Change-Id: Iadc2d2f5cbdce0c8b91a182414e5f032b81b4556
Signed-off-by: Irit Goihman <igoihman(a)redhat.com>
---
M lib/vdsm/network/ovs/switch.py
M tests/network/func_dpdk_test.py
2 files changed, 18 insertions(+), 5 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Irit Goihman: Verified
Edward Haas: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/71812
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iadc2d2f5cbdce0c8b91a182414e5f032b81b4556
Gerrit-PatchSet: 24
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Irit Goihman <igoihman(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years
Change in vdsm[master]: shell-helper: replace xmlrpc with http package
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: shell-helper: replace xmlrpc with http package
......................................................................
shell-helper: replace xmlrpc with http package
xmplrpc package has been recently replaced with http package.
In order to install vdsm using shell_helper functions we need
to replace xmlrpc with http in the list of rpms to install.
Change-Id: I0aebc910628948f8f29e05f427ffd223e4561435
Signed-off-by: Irit Goihman <igoihman(a)redhat.com>
---
M contrib/shell_helper
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Jenkins CI: Passed CI tests
Irit Goihman: Verified
Edward Haas: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/76930
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0aebc910628948f8f29e05f427ffd223e4561435
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Irit Goihman <igoihman(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years
Change in vdsm[master]: net: Move dump_bonding_opts logic content to sysfs_options_m...
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: net: Move dump_bonding_opts logic content to sysfs_options_mapper
......................................................................
net: Move dump_bonding_opts logic content to sysfs_options_mapper
Leaving under dump_bonding_opts only the tool registration, moving the
logic to the network package.
As part of this patch, toolbonding_test module has been temporary
relocated under the test network package.
Change-Id: Ia5c920ad1d33fe8cde17dc3d000cf9d0c2255160
Signed-off-by: Edward Haas <edwardh(a)redhat.com>
---
M lib/vdsm/network/link/bond/sysfs_options_mapper.py
M lib/vdsm/tool/dump_bonding_opts.py
M tests/Makefile.am
M tests/network/nettestlib.py
R tests/network/toolbonding_test.py
5 files changed, 97 insertions(+), 98 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Petr Horáček: Looks good to me, but someone else must approve
Dan Kenigsberg: Looks good to me, approved
Edward Haas: Verified
--
To view, visit https://gerrit.ovirt.org/76285
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia5c920ad1d33fe8cde17dc3d000cf9d0c2255160
Gerrit-PatchSet: 13
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years
Change in vdsm[master]: net test: Merge toolbonding_test module into link_bond_test
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: net test: Merge toolbonding_test module into link_bond_test
......................................................................
net test: Merge toolbonding_test module into link_bond_test
Change-Id: Ieee8d49ed78f905b66bb7b3e0f8bb693ea048add
Signed-off-by: Edward Haas <edwardh(a)redhat.com>
---
M tests/network/link_bond_test.py
D tests/network/toolbonding_test.py
2 files changed, 22 insertions(+), 54 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Petr Horáček: Looks good to me, but someone else must approve
Dan Kenigsberg: Looks good to me, approved
Edward Haas: Verified
--
To view, visit https://gerrit.ovirt.org/76291
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ieee8d49ed78f905b66bb7b3e0f8bb693ea048add
Gerrit-PatchSet: 13
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years
Change in vdsm[master]: net tool: Merge dump_bonding_opts into the network module
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: net tool: Merge dump_bonding_opts into the network module
......................................................................
net tool: Merge dump_bonding_opts into the network module
Change-Id: Ie960395e8272c79b39d4a3d36492a52eaa910c8a
Signed-off-by: Edward Haas <edwardh(a)redhat.com>
---
M lib/vdsm/tool/Makefile.am
D lib/vdsm/tool/dump_bonding_opts.py
M lib/vdsm/tool/network.py
M vdsm.spec.in
4 files changed, 18 insertions(+), 44 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Petr Horáček: Looks good to me, but someone else must approve
Dan Kenigsberg: Looks good to me, approved
Edward Haas: Verified
--
To view, visit https://gerrit.ovirt.org/76292
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie960395e8272c79b39d4a3d36492a52eaa910c8a
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years
Change in vdsm[master]: net tool: Merge dump_bonding_opts into the network module
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has posted comments on this change.
Change subject: net tool: Merge dump_bonding_opts into the network module
......................................................................
Patch Set 12: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/76292
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie960395e8272c79b39d4a3d36492a52eaa910c8a
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: No
7 years
Change in vdsm[master]: net test: Merge toolbonding_test module into link_bond_test
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has posted comments on this change.
Change subject: net test: Merge toolbonding_test module into link_bond_test
......................................................................
Patch Set 13: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/76291
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ieee8d49ed78f905b66bb7b3e0f8bb693ea048add
Gerrit-PatchSet: 13
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: No
7 years
Change in vdsm[master]: net: Clean netinfo.bonding and create sysfs options mapper
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: net: Clean netinfo.bonding and create sysfs options mapper
......................................................................
net: Clean netinfo.bonding and create sysfs options mapper
Bond options scanning and several leftovers from netinfo.bonding have
been relocated and cleaned out.
Users of netinfo.bonding that used it as a proxy have been updated to
point to the bond sysfs driver.
Change-Id: I70d87506b70b8bd182881d0b3c2cdf57d4497618
Signed-off-by: Edward Haas <edwardh(a)redhat.com>
---
M lib/vdsm/network/configurators/ifcfg.py
M lib/vdsm/network/kernelconfig.py
M lib/vdsm/network/link/bond/sysfs_options.py
A lib/vdsm/network/link/bond/sysfs_options_mapper.py
M lib/vdsm/network/netinfo/bonding.py
M lib/vdsm/tool/dump_bonding_opts.py
M tests/network/link_bond_test.py
M tests/network/netinfo_test.py
8 files changed, 151 insertions(+), 121 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Petr Horáček: Looks good to me, but someone else must approve
Dan Kenigsberg: Looks good to me, approved
Edward Haas: Verified
--
To view, visit https://gerrit.ovirt.org/76260
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I70d87506b70b8bd182881d0b3c2cdf57d4497618
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
7 years