yahoo-eng-team team mailing list archive

[OpenStack Infra <2034540@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.opendev.org/c/openstack/neutron/+/893962
Committed: https://opendev.org/openstack/neutron/commit/8cba9a2ee86cb3b65645674ef315c14cfb261143
Submitter: "Zuul (22348)"
Branch:    master

commit 8cba9a2ee86cb3b65645674ef315c14cfb261143
Author: Rodolfo Alonso Hernandez <ralonsoh@xxxxxxxxxx>
Date:   Wed Sep 6 15:58:31 2023 +0000

    Call the "tc qdisc" command for ingress qdisc without parent
    
    The "tc qdisc" command to create an ingress qdisc does not require
    to define a parent qdisc [1]. The ingress qdisc "differs from other
    qdiscs in that it does not occupy the root of a device".
    
    The previous command was replacing the default root egress qdisc:
      root@dev20:~# ip netns exec ns01 tc qdisc show
      qdisc noqueue 0: dev dummy root refcnt 2
    
    This operation is now not permitted with the new kernel.
    
    This patch is also changing how the TC funtional tests interact with
    the namespace interface created. Now the interface is set to UP before
    the test starts. That changes the "tc qdisc show" command because now
    the default qdisc (see the aforementioned CLI output) is now present,
    as it should be in a live environment (where the interfaces are UP).
    
    Closes-Bug: #2034540
    
    [1]https://tldp.org/HOWTO/Adv-Routing-HOWTO/lartc.adv-qdisc.ingress.html
    
    Change-Id: I2f8130dc3cf3244be2a44a4ecbdbaa9c7f865731


** Changed in: neutron
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/2034540

Title:
  [FT] "test_add_tc_filter_policy" failing in master and stable branches

Status in neutron:
  Fix Released

Bug description:
  Test
  "neutron.tests.functional.privileged.agent.linux.test_tc_lib.TcFilterClassTestCase.test_add_tc_filter_policy"
  is failing 100% of the times.

  Error: https://paste.opendev.org/show/bURZiJElPj70CKiGpUbz/

  Log:
  https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_0b4/893655/3/check/neutron-
  functional-with-uwsgi/0b4905d/testr_results.html

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/2034540/+subscriptions