methods succeed and fail in Task class reference non-existent var

Bug #1284975 reported by Nikhil Komawar
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Fix Released
Undecided
Nikhil Komawar

Bug Description

https://github.com/openstack/glance/blob/master/glance/domain/__init__.py line 376 <-- result is referenced however, does not exist in the class definition
and
https://github.com/openstack/glance/blob/master/glance/domain/__init__.py line 382 <-- message is referenced however, does not exist in the class definition

Revision history for this message
Nikhil Komawar (nikhil-komawar) wrote :
Changed in glance:
assignee: nobody → nikhil komawar (nikhil-komawar)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to glance (master)

Reviewed: https://review.openstack.org/76445
Committed: https://git.openstack.org/cgit/openstack/glance/commit/?id=0118103eec9f770fd9013d61f17865598db3737e
Submitter: Jenkins
Branch: master

commit 0118103eec9f770fd9013d61f17865598db3737e
Author: Nikhil Komawar <email address hidden>
Date: Tue Feb 25 23:28:04 2014 -0600

    Adds TaskStub class

    Partial Task patch, Id I4fbadc9a97e3147128c7c733384c7bb50918806f
    removed the result and message vars of the Tasks class. However,
    the succeed and fail methods were using those to update them
    respectively. This patch introduces a new TaskStub class which
    enables the PartialTask functionality without looking result
    and message attrs of the Task class.

    Fixes bug 1284975

    Change-Id: I0b406e4662cfd3cb496b71be77ad10a90c178baa

Changed in glance:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in glance:
milestone: none → juno-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in glance:
milestone: juno-1 → 2014.2
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.