← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1781100] Re: TypeError: delete_allocation_for_failed_resize() takes exactly 5 arguments (4 given) in ComputeManager.prep_resize

 

Reviewed:  https://review.openstack.org/581548
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=78af1de1587fb83e09d1c0639fb5a28eb6e51a27
Submitter: Zuul
Branch:    master

commit 78af1de1587fb83e09d1c0639fb5a28eb6e51a27
Author: Matt Riedemann <mriedem.os@xxxxxxxxx>
Date:   Tue Jul 10 22:11:05 2018 -0400

    Fix TypeError in prep_resize allocation cleanup
    
    The CachingScheduler doesn't create allocations in
    placement so when prep_resize fails and it tries
    to revert allocations, the _revert_allocation() method
    returns False and then prep_resize calls
    delete_allocation_for_failed_resize() which since change
    I7891b98f225f97ad47f189afb9110ef31c810717 has taken a
    context argument, but that's not being passed so it
    results in a TypeError.
    
    This fixes the TypeError and adds a functional test case
    which runs a resize reschedule scenario with the
    CachingScheduler.
    
    Change-Id: I0ec0c2d3f7ef8d2274b97a28a175d53921edc9db
    Closes-Bug: #1781100


** Changed in: nova
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1781100

Title:
  TypeError: delete_allocation_for_failed_resize() takes exactly 5
  arguments (4 given) in ComputeManager.prep_resize

Status in OpenStack Compute (nova):
  Fix Released
Status in OpenStack Compute (nova) queens series:
  In Progress

Bug description:
  I found this (without trying) but writing this test:

  http://paste.openstack.org/show/725514/

      2018-07-10 22:03:11,206 INFO [nova.compute.manager] Old-style migration 156eaee2-48f0-49f8-aa8a-daa297864d25 is being reverted; no migration claims found on original node to swap.
      2018-07-10 22:03:11,220 ERROR [nova.compute.manager] Setting instance vm_state to ERROR
      Traceback (most recent call last):
        File "nova/compute/manager.py", line 7634, in _error_out_instance_on_exception
          yield
        File "nova/compute/manager.py", line 4163, in prep_resize
          instance, node, instance_type)
      TypeError: delete_allocation_for_failed_resize() takes exactly 5 arguments (4 given)

  The bug was introduced with this change:

  https://review.openstack.org/#/q/I7891b98f225f97ad47f189afb9110ef31c810717

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1781100/+subscriptions


References