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

[Rodolfo Alonso]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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)

[OpenStack Infra (hudson-openstack)]

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)

[OpenStack Infra (hudson-openstack)]

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)

[OpenStack Infra (hudson-openstack)]

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)

[OpenStack Infra (hudson-openstack)]

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)

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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)

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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