yahoo-eng-team team mailing list archive

[Devananda van der Veen <1369317@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

https://review.openstack.org/#/c/118391/
Change-id I012ee5c118265e044ff41fb58b732728946ee85a

included a change to the definition of nova/scheduler/host_manager.py
HostState.consume_from_instance(), changing the method from requiring 2
to 3 parameters.

This change did not update the ironic_host_manager's HostState class,
thus breaking it.

This has broken Ironic's tempest tests, resulting in all our CI tests
failing.

** Affects: nova
     Importance: Undecided
     Assignee: Devananda van der Veen (devananda)
         Status: New


** Tags: ironic

** Tags added: ironic

** Description changed:

- Change-id  I012ee5c118265e044ff41fb58b732728946ee85a included a change
- to the definition of nova/scheduler/host_manager.py
+ https://review.openstack.org/#/c/118391/
+ Change-id I012ee5c118265e044ff41fb58b732728946ee85a
+ 
+ included a change to the definition of nova/scheduler/host_manager.py
  HostState.consume_from_instance(), changing the method from requiring 2
  to 3 parameters.
  
  This change did not update the ironic_host_manager's HostState class,
  thus breaking it.
  
  This has broken Ironic's tempest tests, resulting in all our CI tests
  failing.

** Changed in: nova
     Assignee: (unassigned) => Devananda van der Veen (devananda)

-- 
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/1369317

Title:
  ironic_host_manager's consume_from_instance() takes exactly 2
  arguments (3 given)

Status in OpenStack Compute (Nova):
  New

Bug description:
  https://review.openstack.org/#/c/118391/
  Change-id I012ee5c118265e044ff41fb58b732728946ee85a

  included a change to the definition of nova/scheduler/host_manager.py
  HostState.consume_from_instance(), changing the method from requiring
  2 to 3 parameters.

  This change did not update the ironic_host_manager's HostState class,
  thus breaking it.

  This has broken Ironic's tempest tests, resulting in all our CI tests
  failing.

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