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

Bug #2034540 reported by Rodolfo Alonso
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Critical
Rodolfo Alonso

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

Changed in neutron:
importance: Undecided → Critical
Changed in neutron:
assignee: nobody → Rodolfo Alonso (rodolfo-alonso-hernandez)
yatin (yatinkarel)
tags: added: functional-tests gate-failure
yatin (yatinkarel)
Changed in neutron:
status: New → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/893962

Changed in neutron:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/2023.1)

Fix proposed to branch: stable/2023.1
Review: https://review.opendev.org/c/openstack/neutron/+/894008

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/zed)

Fix proposed to branch: stable/zed
Review: https://review.opendev.org/c/openstack/neutron/+/894009

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/yoga)

Fix proposed to branch: stable/yoga
Review: https://review.opendev.org/c/openstack/neutron/+/894010

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/xena)

Fix proposed to branch: stable/xena
Review: https://review.opendev.org/c/openstack/neutron/+/894011

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/wallaby)

Fix proposed to branch: stable/wallaby
Review: https://review.opendev.org/c/openstack/neutron/+/894012

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

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 <email address hidden>
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
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/2023.1)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/894008
Committed: https://opendev.org/openstack/neutron/commit/e16a249611dce1a2b39a82431ac6de2a6a39efa8
Submitter: "Zuul (22348)"
Branch: stable/2023.1

commit e16a249611dce1a2b39a82431ac6de2a6a39efa8
Author: Rodolfo Alonso Hernandez <email address hidden>
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
    (cherry picked from commit 8cba9a2ee86cb3b65645674ef315c14cfb261143)

tags: added: in-stable-yoga
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/yoga)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/894010
Committed: https://opendev.org/openstack/neutron/commit/272b3ae7189e04ad2d8a6541d0c00c60337e24dc
Submitter: "Zuul (22348)"
Branch: stable/yoga

commit 272b3ae7189e04ad2d8a6541d0c00c60337e24dc
Author: Rodolfo Alonso Hernandez <email address hidden>
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

    Conflicts:
        neutron/tests/functional/privileged/agent/linux/test_tc_lib.py

    Change-Id: I2f8130dc3cf3244be2a44a4ecbdbaa9c7f865731
    (cherry picked from commit 8cba9a2ee86cb3b65645674ef315c14cfb261143)
    (cherry picked from commit 37d09f95cd58143d0b2de9586158d3a6dfda800b)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/wallaby)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/894012
Committed: https://opendev.org/openstack/neutron/commit/2179c6978d18c74fe6e58f718d842808a01eb4a5
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit 2179c6978d18c74fe6e58f718d842808a01eb4a5
Author: Rodolfo Alonso Hernandez <email address hidden>
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

    Conflicts:
        neutron/tests/functional/privileged/agent/linux/test_tc_lib.py

    Change-Id: I2f8130dc3cf3244be2a44a4ecbdbaa9c7f865731
    (cherry picked from commit 8cba9a2ee86cb3b65645674ef315c14cfb261143)
    (cherry picked from commit 37d09f95cd58143d0b2de9586158d3a6dfda800b)

tags: added: in-stable-wallaby
tags: added: in-stable-xena
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/xena)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/894011
Committed: https://opendev.org/openstack/neutron/commit/03e2b715d0440ba7206894e4c3ab4720d2408639
Submitter: "Zuul (22348)"
Branch: stable/xena

commit 03e2b715d0440ba7206894e4c3ab4720d2408639
Author: Rodolfo Alonso Hernandez <email address hidden>
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

    Conflicts:
        neutron/tests/functional/privileged/agent/linux/test_tc_lib.py

    Change-Id: I2f8130dc3cf3244be2a44a4ecbdbaa9c7f865731
    (cherry picked from commit 8cba9a2ee86cb3b65645674ef315c14cfb261143)
    (cherry picked from commit 37d09f95cd58143d0b2de9586158d3a6dfda800b)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/zed)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/894009
Committed: https://opendev.org/openstack/neutron/commit/37d09f95cd58143d0b2de9586158d3a6dfda800b
Submitter: "Zuul (22348)"
Branch: stable/zed

commit 37d09f95cd58143d0b2de9586158d3a6dfda800b
Author: Rodolfo Alonso Hernandez <email address hidden>
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

    Conflicts:
        neutron/tests/functional/privileged/agent/linux/test_tc_lib.py

    Change-Id: I2f8130dc3cf3244be2a44a4ecbdbaa9c7f865731
    (cherry picked from commit 8cba9a2ee86cb3b65645674ef315c14cfb261143)

tags: added: in-stable-zed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 23.0.0.0rc1

This issue was fixed in the openstack/neutron 23.0.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/victoria)

Fix proposed to branch: stable/victoria
Review: https://review.opendev.org/c/openstack/neutron/+/898567

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/victoria)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/898567
Committed: https://opendev.org/openstack/neutron/commit/e3e4d677680d7f7cd404b40a4fb5e0e984072e89
Submitter: "Zuul (22348)"
Branch: stable/victoria

commit e3e4d677680d7f7cd404b40a4fb5e0e984072e89
Author: Rodolfo Alonso Hernandez <email address hidden>
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

    Conflicts:
        neutron/tests/functional/privileged/agent/linux/test_tc_lib.py

    Change-Id: I2f8130dc3cf3244be2a44a4ecbdbaa9c7f865731
    (cherry picked from commit 8cba9a2ee86cb3b65645674ef315c14cfb261143)
    (cherry picked from commit 37d09f95cd58143d0b2de9586158d3a6dfda800b)
    (cherry picked from commit 2179c6978d18c74fe6e58f718d842808a01eb4a5)

tags: added: in-stable-victoria
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 22.1.0

This issue was fixed in the openstack/neutron 22.1.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 20.5.0

This issue was fixed in the openstack/neutron 20.5.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 21.2.0

This issue was fixed in the openstack/neutron 21.2.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron victoria-eom

This issue was fixed in the openstack/neutron victoria-eom release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron wallaby-eom

This issue was fixed in the openstack/neutron wallaby-eom release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron xena-eom

This issue was fixed in the openstack/neutron xena-eom release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.