yahoo-eng-team team mailing list archive

[Zhenyu Zheng <zhengzhenyu@xxxxxxxxxx>]

Public bug reported:

In patch https://review.openstack.org/#/c/219980 we added a few lines of
code to handle nova-compute failure during a soft reboot.

http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/manager.py#n1059 to 1067
but somehow we missed instance.save() at the end of this logic, so the task state didn't
saved and the logic doesn't work.

** Affects: nova
     Importance: Undecided
     Assignee: Zhenyu Zheng (zhengzhenyu)
         Status: In Progress

** Changed in: nova
     Assignee: (unassigned) => Zhenyu Zheng (zhengzhenyu)

** Description changed:

  In patch https://review.openstack.org/#/c/219980 we added a few lines of
  code to handle nova-compute failure during a soft reboot.
+ 
+ http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/manager.py#n1059 to 1067
+ but somehow we missed instance.save() at the end of this logic, so the task state didn't
+ saved and the logic doesn't work.

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

Title:
  Add instance.save() when handling nova-compute failure during a soft
  reboot

Status in OpenStack Compute (nova):
  In Progress

Bug description:
  In patch https://review.openstack.org/#/c/219980 we added a few lines
  of code to handle nova-compute failure during a soft reboot.

  http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/manager.py#n1059 to 1067
  but somehow we missed instance.save() at the end of this logic, so the task state didn't
  saved and the logic doesn't work.

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