Dan Kenigsberg has submitted this change and it was merged. Change subject: `tc qdisc del device x ingress` changed err code ......................................................................
`tc qdisc del device x ingress` changed err code In new kernel, operation `tc qdisc del device dummy_x ingress` changed error code of 'no ingress exists' from EINVAL to ENOENT. CentOS 6.5 $ ip link add dummy_ni type dummy $ tc qdisc del dev dummy_ni ingress RTNETLINK answers: Invalid argument Fedora 20 $ ip link add dummy_ni type dummy $ tc qdisc del dev dummy_ni ingress RTNETLINK answers: No such file or directory Now qos.py:_fresh_qdisc_conf_out() ignores ENOENT too. Change-Id: Ic40d7bdd34212cbd63850142211e544cfe1bc97b Signed-off-by: Petr Horáček <[email protected]> Reviewed-on: http://gerrit.ovirt.org/33973 Reviewed-by: Antoni Segura Puimedon <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M vdsm/network/configurators/qos.py 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Antoni Segura Puimedon: Looks good to me, but someone else must approve Petr Horáček: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33973 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic40d7bdd34212cbd63850142211e544cfe1bc97b Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Petr Horáček <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Petr Horáček <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
