BuiltInstanceTasks.get_volume_mountpoint is using an unavailable argument

Bug #1263371 reported by Sushil Kumar
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack DBaaS (Trove)
Fix Released
Undecided
Sushil Kumar

Bug Description

At https://github.com/openstack/trove/blob/master/trove/taskmanager/models.py#L593

BuiltInstanceTasks.get_volume_mountpoint is using an unavailable argument is using an un-assigned argument volume_id.

This should be rather self.volume_id which is a part of self.

description: updated
description: updated
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to trove (master)

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

Changed in trove:
assignee: nobody → Sushil Kumar (sushil-kumar2)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to trove (master)

Reviewed: https://review.openstack.org/63589
Committed: https://git.openstack.org/cgit/openstack/trove/commit/?id=f835ad6aca4e9131afbb3c9711c08d8650decb82
Submitter: Jenkins
Branch: master

commit f835ad6aca4e9131afbb3c9711c08d8650decb82
Author: Sushil Kumar <email address hidden>
Date: Sat Dec 21 18:15:10 2013 +0000

    Removes BuiltInstanceTasks.get_volume_mountpoint

    Reasons:
    - Method is not being used in the project.
    - Method has an unassigned variable used volume_id.

    Changes:
    - Removes get_volume_mountpoint from trove.taskmanager.models.

    Change-Id: I2942a4cc952a473dd1ed1d52c8cb2205566b68ae
    Closes-Bug: #1263371

Changed in trove:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in trove:
milestone: none → icehouse-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in trove:
milestone: icehouse-2 → 2014.1
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.