_ensure_vr_id_and_network is not used anywhere in the code

Bug #1715371 reported by venkata anil
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Undecided
Mamatisa Nurmatov

Bug Description

_ensure_vr_id_and_network is not used anywhere in the code, hence can be removed.

tags: added: low-hanging-fruit
Revision history for this message
Christopher Brown (snecklifter) wrote :

[cbrown2@flash neutron]$ grep -ir "_ensure_vr_id_and_network" *
neutron/tests/unit/db/test_l3_hamode_db.py: net = self.plugin._ensure_vr_id_and_network(
neutron/tests/unit/db/test_l3_hamode_db.py: def test_ensure_vr_id_and_network_net_exists(self):
neutron/tests/unit/db/test_l3_hamode_db.py: self.plugin._ensure_vr_id_and_network(
neutron/tests/unit/db/test_l3_hamode_db.py: def test_ensure_vr_id_and_network_concurrent_create(self):
neutron/tests/unit/db/test_l3_hamode_db.py: net = self.plugin._ensure_vr_id_and_network(
neutron/tests/unit/db/test_l3_hamode_db.py: self.plugin._ensure_vr_id_and_network(
neutron/tests/unit/db/test_l3_hamode_db.py: def test_ensure_vr_id_and_network_interface_failure(self):
neutron/tests/unit/db/test_l3_hamode_db.py: def test_ensure_vr_id_and_network_concurrent_delete(self):
neutron/tests/unit/db/test_l3_hamode_db.py: def test_ensure_vr_id_and_network_concurrent_swap(self):
neutron/db/l3_hamode_db.py: def _ensure_vr_id_and_network(self, context, router_db):

Would you like me to prep a patch which removes this ^^^ ?

Revision history for this message
Sapna Jadhav (sapana45) wrote :

Hi all,
Is this bug still valid? If yes then I would like to work on this.
Shall I remove _ensure_vr_id_and_network from the code?

Changed in neutron:
assignee: nobody → Mamatisa Nurmatov (isabek)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

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

commit ba3d78099f47c3819af0f6eb2083a1df7f67c38d
Author: Nurmatov Mamatisa <email address hidden>
Date: Fri Apr 2 17:07:16 2021 +0300

    Remove unused method _ensure_vr_id_and_network()

    Method _ensure_vr_id_and_network is not used in l3_hamode_db.py,
    hence method and related tests can be removed.

    Closes-Bug: #1715371
    Change-Id: I9a64e683f060573195a88b3c4ee34abbd941fa44

Changed in neutron:
status: In Progress → Fix Released
tags: added: neutron-proactive-backport-potential
tags: removed: neutron-proactive-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 19.0.0.0rc1

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

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.