_cleanup_running_deleted_instances periodic task failing

Bug #1097464 reported by Chris Behrens
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
Mark McLoughlin

Bug Description

2013-01-08 07:10:31 8040 ERROR nova.manager [-] Error during ComputeManager._cleanup_running_deleted_instances: unsupported operand type(s) for -: 'datetime.datetime' and 'unicode'
2013-01-08 07:10:31 8040 TRACE nova.manager Traceback (most recent call last):
2013-01-08 07:10:31 8040 TRACE nova.manager File "/opt/stack/new/nova/nova/manager.py", line 229, in periodic_tasks
2013-01-08 07:10:31 8040 TRACE nova.manager task(self, context)
2013-01-08 07:10:31 8040 TRACE nova.manager File "/opt/stack/new/nova/nova/compute/manager.py", line 3368, in _cleanup_running_deleted_instances
2013-01-08 07:10:31 8040 TRACE nova.manager for instance in self._running_deleted_instances(context):
2013-01-08 07:10:31 8040 TRACE nova.manager File "/opt/stack/new/nova/nova/compute/manager.py", line 3412, in _running_deleted_instances
2013-01-08 07:10:31 8040 TRACE nova.manager return [i for i in instances if deleted_instance(i)]
2013-01-08 07:10:31 8040 TRACE nova.manager File "/opt/stack/new/nova/nova/compute/manager.py", line 3405, in deleted_instance
2013-01-08 07:10:31 8040 TRACE nova.manager timeout))
2013-01-08 07:10:31 8040 TRACE nova.manager File "/opt/stack/new/nova/nova/openstack/common/timeutils.py", line 76, in is_older_than
2013-01-08 07:10:31 8040 TRACE nova.manager return utcnow() - before > datetime.timedelta(seconds=seconds)
2013-01-08 07:10:31 8040 TRACE nova.manager TypeError: unsupported operand type(s) for -: 'datetime.datetime' and 'unicode'
2013-01-08 07:10:31 8040 TRACE nova.manager

Revision history for this message
Chris Behrens (cbehrens) wrote :

Sounds like we need timeutils.is_older_than() changes synced from oslo.

Changed in nova:
importance: Undecided → Medium
status: New → Incomplete
status: Incomplete → Triaged
Changed in nova:
assignee: nobody → Mark McLoughlin (markmc)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/19201
Committed: http://github.com/openstack/nova/commit/fa2ca8ea7f538669fb32e431eb4c5ea94e6e2e3f
Submitter: Jenkins
Branch: master

commit fa2ca8ea7f538669fb32e431eb4c5ea94e6e2e3f
Author: Flaper Fesp <email address hidden>
Date: Tue Jan 8 16:10:58 2013 +0100

    Import latest timeutils from oslo-incubator

    Fixes bug #1097464

    Changes synced are:

     a08daf1 Use basestring instead of str for type check.
     615d769 Account for tasks duration in LoopingCall delay
     ce80aac Convenience wrapper for datetime.timedelta.total_seconds()

    Change-Id: If4ab7ccd2f4e815e01c41f25fdd541ffcd7693d5

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