neutron

"TestOvnNbIdlNotifyHandler" uses ML2Plugin instead of OVNMechDriver

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

Bug Description

The testing class "TestOvnNbIdlNotifyHandler" uses self.driver to create the "ovsdb_monitor.OvnNbIdl" instance, instead of self.mech_driver.
* self.driver: ML2Plugin instance
* self.mech_driver: OVNMechDriver instance

The needed object to create the "OvnNbIdl" instance is the "OVNMechDriver" one.

EDIT:
This issue is also affecting "TestOvnSbIdlNotifyHandler".

See original description
Rodolfo Alonso (rodolfo-alonso-hernandez)
Changed in neutron:
assignee: nobody → Rodolfo Alonso (rodolfo-alonso-hernandez)
importance: Undecided → Medium
importance: Medium → Low
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/+/858663

Changed in neutron:
status: New → In Progress
Rodolfo Alonso (rodolfo-alonso-hernandez)
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

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

commit e9b5a27001614836fcf8297dd8510d144816bd02
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Wed Sep 21 12:08:13 2022 +0200

    Use "OVNMechDriver" instance in "TestOvn[Nb|Sb]IdlNotifyHandler"

    The test classes "TestOvn[Nb|Sb]IdlNotifyHandler" now use the
    "OVNMechDriver" instance to create the "OvnNbIdl" instance, that
    is stored in "self.mech_driver", instead of "self.driver".

    Closes-Bug: #1990374
    Change-Id: Ieef6c8a02ca0cf85bc72b9f7d09b46dbd0e073be

Changed in neutron:
status: In Progress → Fix Released
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/+/859084

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/+/859091

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/+/859092

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/+/859093

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

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

commit a9e0dd98fa2068076cc4232b21dc98d5d4f3bead
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Wed Sep 21 12:08:13 2022 +0200

    Use "OVNMechDriver" instance in "TestOvn[Nb|Sb]IdlNotifyHandler"

    The test classes "TestOvn[Nb|Sb]IdlNotifyHandler" now use the
    "OVNMechDriver" instance to create the "OvnNbIdl" instance, that
    is stored in "self.mech_driver", instead of "self.driver".

    Closes-Bug: #1990374
    Change-Id: Ieef6c8a02ca0cf85bc72b9f7d09b46dbd0e073be
    (cherry picked from commit e9b5a27001614836fcf8297dd8510d144816bd02)

tags: added: in-stable-zed
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/+/859091
Committed: https://opendev.org/openstack/neutron/commit/f15a0b66a91756c7d9cb7477251eef423e31e6e7
Submitter: "Zuul (22348)"
Branch: stable/yoga

commit f15a0b66a91756c7d9cb7477251eef423e31e6e7
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Wed Sep 21 12:08:13 2022 +0200

    Use "OVNMechDriver" instance in "TestOvn[Nb|Sb]IdlNotifyHandler"

    The test classes "TestOvn[Nb|Sb]IdlNotifyHandler" now use the
    "OVNMechDriver" instance to create the "OvnNbIdl" instance, that
    is stored in "self.mech_driver", instead of "self.driver".

    Closes-Bug: #1990374
    Change-Id: Ieef6c8a02ca0cf85bc72b9f7d09b46dbd0e073be
    (cherry picked from commit e9b5a27001614836fcf8297dd8510d144816bd02)

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/+/859092
Committed: https://opendev.org/openstack/neutron/commit/794860eb75438e82a3fa3ff5f5d378d20516836e
Submitter: "Zuul (22348)"
Branch: stable/xena

commit 794860eb75438e82a3fa3ff5f5d378d20516836e
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Wed Sep 21 12:08:13 2022 +0200

    Use "OVNMechDriver" instance in "TestOvn[Nb|Sb]IdlNotifyHandler"

    The test classes "TestOvn[Nb|Sb]IdlNotifyHandler" now use the
    "OVNMechDriver" instance to create the "OvnNbIdl" instance, that
    is stored in "self.mech_driver", instead of "self.driver".

    Closes-Bug: #1990374
    Change-Id: Ieef6c8a02ca0cf85bc72b9f7d09b46dbd0e073be
    (cherry picked from commit e9b5a27001614836fcf8297dd8510d144816bd02)

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

Reviewed: https://review.opendev.org/c/openstack/neutron/+/859093
Committed: https://opendev.org/openstack/neutron/commit/00efb17f00a71c75c3ea9286adebf28d49b7a3ab
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit 00efb17f00a71c75c3ea9286adebf28d49b7a3ab
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Wed Sep 21 12:08:13 2022 +0200

    Use "OVNMechDriver" instance in "TestOvn[Nb|Sb]IdlNotifyHandler"

    The test classes "TestOvn[Nb|Sb]IdlNotifyHandler" now use the
    "OVNMechDriver" instance to create the "OvnNbIdl" instance, that
    is stored in "self.mech_driver", instead of "self.driver".

    Closes-Bug: #1990374
    Change-Id: Ieef6c8a02ca0cf85bc72b9f7d09b46dbd0e073be
    (cherry picked from commit e9b5a27001614836fcf8297dd8510d144816bd02)

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

This issue was fixed in the openstack/neutron 21.0.0.0rc2 release candidate.

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

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

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

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

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

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

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

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

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

Other bug subscribers

Remote bug watches

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