Remove unused DB API get_rcs_by_bay_uuid

Bug #1489570 reported by Vilobh Meshram
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Magnum
Fix Released
Undecided
Vilobh Meshram

Bug Description

get_rcs_by_bay_uuid is not used by any consumer.

Proposed that we should clean up the dead code. Discussed this with Kennan and hongbin on IRC and hence proposing the fix to clean up dead code. If all the cores agree we can merge this.

Changed in magnum:
assignee: nobody → Vilobh Meshram (vilobhmm)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to magnum (master)

Fix proposed to branch: master
Review: https://review.openstack.org/217865

Changed in magnum:
status: New → In Progress
Revision history for this message
Vilobh Meshram (vilobhmm) wrote :

After further analysis found that get_rcs_by_bay_uuid is not used by any consumer.

Proposed that we should clean up the dead code. Discussed this with Kennan and hongbin on IRC and hence proposing the fix to clean up dead code. If all the cores agree we can merge this.

summary: - Introduce list_by_bay_uuid API to replication controller object
+ Remove unused DB API get_rcs_by_bay_uuid
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to magnum (master)

Reviewed: https://review.openstack.org/217865
Committed: https://git.openstack.org/cgit/openstack/magnum/commit/?id=31dcba92cb6c1a1f0a062b65f96f1bbb650d93ef
Submitter: Jenkins
Branch: master

commit 31dcba92cb6c1a1f0a062b65f96f1bbb650d93ef
Author: Vilobh Meshram <email address hidden>
Date: Thu Aug 27 12:39:27 2015 -0700

    Remove unused DB API get_rcs_by_bay_uuid

    Remove unused DB API 'get_rcs_by_bay_uuid' and associated unit
    tests. There are no consumers of this API in its current state of
    art. Cleaning up dead code.

    Also, added a related bug which will serve as a placeholder
    to re-add the removed code, if needed in future.

    Closes-Bug: #1489570
    Related-Bug: #1499121

    Change-Id: I7c91323e53d7f60a93876197ac1886f8e086d173

Changed in magnum:
status: In Progress → Fix Committed
Adrian Otto (aotto)
Changed in magnum:
milestone: none → mitaka-1
status: Fix Committed → Fix Released
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.