commit cd53a181be22291e6f9be1430cab12ba1f381b8a
Author: Matt Riedemann <email address hidden>
Date: Fri May 11 14:44:47 2018 -0400
Don't reschedule on RequestedVRamTooHigh errors
The libvirt driver validates the hw_video_ram image property,
if specified, and the flavor extra spec "hw_video:ram_max_mb"
is set. If validation fails, the libvirt driver raises
RequestedVRamTooHigh which is not handled explicitly in
ComputeManager._build_and_run_instance so it will result in
a RescheduledException to another compute to retry the spawn
but that will always fail because this isn't something that
is per-compute host.
This change adds the error handling in _build_and_run_instance
so that we'll fail and abort the build and not reschedule.
Long-term, this validation should be moved into the API code
since it's not specific to a compute host and would be user
error that should result in a 400 response.
Reviewed: https:/ /review. openstack. org/567929 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=cd53a181be2 2291e6f9be1430c ab12ba1f381b8a
Committed: https:/
Submitter: Zuul
Branch: master
commit cd53a181be22291 e6f9be1430cab12 ba1f381b8a
Author: Matt Riedemann <email address hidden>
Date: Fri May 11 14:44:47 2018 -0400
Don't reschedule on RequestedVRamTo oHigh errors
The libvirt driver validates the hw_video_ram image property, ram_max_ mb" amTooHigh which is not handled explicitly in ger._build_ and_run_ instance so it will result in ption to another compute to retry the spawn
if specified, and the flavor extra spec "hw_video:
is set. If validation fails, the libvirt driver raises
RequestedVR
ComputeMana
a RescheduledExce
but that will always fail because this isn't something that
is per-compute host.
This change adds the error handling in _build_ and_run_ instance
so that we'll fail and abort the build and not reschedule.
Long-term, this validation should be moved into the API code
since it's not specific to a compute host and would be user
error that should result in a 400 response.
Change-Id: I93b409ca2b7b36 400759ee9d2cd5b 71c6df186ab
Partial-Bug: #1770726