Exceptions raised from _prep_block_device not included in instance.fault

Bug #1808248 reported by melanie witt
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
In Progress
Undecided
melanie witt

Bug Description

Noticed this while working on a patch, when boot-from-volume instance build fails during the _prep_block_device step, the message in the instance.fault looks like this during a 'nova show':

 {"message": "Build of instance ec8f00c3-49a5-4ae0-9c93-1a813b17bf4c aborted: Failure prepping block device.", "code": 500, "created": "2018-12-12T22:06:22Z"}

where the underlying exception raised from _prep_block_device is absent.

We can see the error message being squashed here when BuildAbortException is raised:

https://github.com/openstack/nova/blob/c72dafad8011198b78fd906f2fc7d1126e3f79f3/nova/compute/manager.py#L2341-L2343

We can provide more information for users if we include the exception message from the exception raised in _prep_block_device in the BuildAbortException, which will end up in the instance.fault.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/624832

Changed in nova:
status: New → In Progress
melanie witt (melwitt)
tags: added: serviceability volumes
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (master)

Change abandoned by melanie witt (<email address hidden>) on branch: master
Review: https://review.openstack.org/624832
Reason: Ended up handling propagation of the failure reason in the top patch:

https://review.openstack.org/616777

It's better not to propagate everything across the board, to prevent potentially leaking host-specific information out to the user in the case of other exceptions.

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.