Trunk subports aren't treated as dvr serviced ports

[Slawek Kaplonski]

In case of dvr, for dvr serviced ports, there are openflow rules in br-int installed to translate port's mac address to gateway mac. It's in table 1 of br-int and it looks like:

cookie=0xf8f0be9a44e579e7, duration=351.138s, table=1, n_packets=0, n_bytes=0, idle_age=353, priority=20,dl_vlan=3,dl_dst=fa:16:3e:94:c3:a5 actions=mod_dl_src:fa:16:3e:48:5e:70,resubmit(,60)

But trunk subports aren't included in list of dvr serviced device owners so traffic from those ports isn't going through those rules and it never goes out of the br-int.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/716642

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/716642
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=d0a16522272bce64bceca1ccdb8d98a38d953cf5
Submitter: Zuul
Branch: master

commit d0a16522272bce64bceca1ccdb8d98a38d953cf5
Author: Slawek Kaplonski <email address hidden>
Date: Tue Mar 31 20:39:13 2020 +0200

    Add trunk subports to be one of dvr serviced device owners

    When "trunk:subport" wasn't added to the list of device owners which
    are supported by dvr, there was no proper config in br-int's openflow
    rules for such port, e.g. there was no dvr_to_src_mac rule in table 1
    added and traffic from such port was never going through br-int.

    Trunk ports should be added to this dvr serviced device owners list and
    that patch is adding it there.

    Change-Id: Ic21089adfa32dbf5d0e29a89713e6e2bf28f0f05
    Closes-Bug: #1870114

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/train
Review: https://review.opendev.org/717626

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/stein
Review: https://review.opendev.org/717627

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/rocky
Review: https://review.opendev.org/717628

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/queens
Review: https://review.opendev.org/717629

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/717626
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=d71d5e1f8dd3c240a1e9162463bfe6b9f8bdc6ef
Submitter: Zuul
Branch: stable/train

commit d71d5e1f8dd3c240a1e9162463bfe6b9f8bdc6ef
Author: Slawek Kaplonski <email address hidden>
Date: Tue Mar 31 20:39:13 2020 +0200

    Add trunk subports to be one of dvr serviced device owners

    When "trunk:subport" wasn't added to the list of device owners which
    are supported by dvr, there was no proper config in br-int's openflow
    rules for such port, e.g. there was no dvr_to_src_mac rule in table 1
    added and traffic from such port was never going through br-int.

    Trunk ports should be added to this dvr serviced device owners list and
    that patch is adding it there.

    Change-Id: Ic21089adfa32dbf5d0e29a89713e6e2bf28f0f05
    Closes-Bug: #1870114
    (cherry picked from commit d0a16522272bce64bceca1ccdb8d98a38d953cf5)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/717627
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=08f3633990018a80dfc12dfe8ac8d9e010a8530f
Submitter: Zuul
Branch: stable/stein

commit 08f3633990018a80dfc12dfe8ac8d9e010a8530f
Author: Slawek Kaplonski <email address hidden>
Date: Tue Mar 31 20:39:13 2020 +0200

    Add trunk subports to be one of dvr serviced device owners

    When "trunk:subport" wasn't added to the list of device owners which
    are supported by dvr, there was no proper config in br-int's openflow
    rules for such port, e.g. there was no dvr_to_src_mac rule in table 1
    added and traffic from such port was never going through br-int.

    Trunk ports should be added to this dvr serviced device owners list and
    that patch is adding it there.

    Conflicts:
        neutron/common/utils.py

    Change-Id: Ic21089adfa32dbf5d0e29a89713e6e2bf28f0f05
    Closes-Bug: #1870114
    (cherry picked from commit d0a16522272bce64bceca1ccdb8d98a38d953cf5)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/717629
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=d5fa72c2ae7947803781fb466514242a881e1a6c
Submitter: Zuul
Branch: stable/queens

commit d5fa72c2ae7947803781fb466514242a881e1a6c
Author: Slawek Kaplonski <email address hidden>
Date: Tue Mar 31 20:39:13 2020 +0200

    Add trunk subports to be one of dvr serviced device owners

    When "trunk:subport" wasn't added to the list of device owners which
    are supported by dvr, there was no proper config in br-int's openflow
    rules for such port, e.g. there was no dvr_to_src_mac rule in table 1
    added and traffic from such port was never going through br-int.

    Trunk ports should be added to this dvr serviced device owners list and
    that patch is adding it there.

    Conflicts:
        neutron/common/utils.py

    Change-Id: Ic21089adfa32dbf5d0e29a89713e6e2bf28f0f05
    Closes-Bug: #1870114
    (cherry picked from commit d0a16522272bce64bceca1ccdb8d98a38d953cf5)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/717628
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=687304a9b5ae3affefaa081fcee4975a1d10e947
Submitter: Zuul
Branch: stable/rocky

commit 687304a9b5ae3affefaa081fcee4975a1d10e947
Author: Slawek Kaplonski <email address hidden>
Date: Tue Mar 31 20:39:13 2020 +0200

    Add trunk subports to be one of dvr serviced device owners

    When "trunk:subport" wasn't added to the list of device owners which
    are supported by dvr, there was no proper config in br-int's openflow
    rules for such port, e.g. there was no dvr_to_src_mac rule in table 1
    added and traffic from such port was never going through br-int.

    Trunk ports should be added to this dvr serviced device owners list and
    that patch is adding it there.

    Conflicts:
        neutron/common/utils.py

    Change-Id: Ic21089adfa32dbf5d0e29a89713e6e2bf28f0f05
    Closes-Bug: #1870114
    (cherry picked from commit d0a16522272bce64bceca1ccdb8d98a38d953cf5)

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/neutron queens-eol release.

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/neutron rocky-eol release.