When claims are rejected indicate specific reason

Bug #1212768 reported by Christopher Lefelhocz
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
jichenjc

Bug Description

In resource_tracker.py upon failure, we raise a generic "ComputeResourcesUnavailable". The logging presents good information as to why there was a failure. We should either provide back data in the ComputeResourcesUnavailble object or raise different exceptions based on what is not available. This may point to a bigger issue of a broken scheduler.

Revision history for this message
Christopher Lefelhocz (christopher-lefelhoc) wrote :

This is a low priority development bug at this point.

Andrew Laski (alaski)
Changed in nova:
importance: Undecided → Low
Revision history for this message
Andrew Laski (alaski) wrote :

Right now Claim.test() returns True/False. It tests memory, disk, and cpu separately and could return either a tuple of bools or a dict of True/False values with the keys indicating the resources. Then the resource tracker can update the ComputeResourcesUnavailable exception with appropriate information. It's probably best to just keep the single exception and stuff data into it because there are many possible combinations of which resources are not available.

Changed in nova:
status: New → Triaged
jichenjc (jichenjc)
Changed in nova:
assignee: nobody → jichencom (jichenjc)
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/61359

Changed in nova:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/61359
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=105d99d4f52fa849e80829dad3ed12b116820118
Submitter: Jenkins
Branch: master

commit 105d99d4f52fa849e80829dad3ed12b116820118
Author: jichen <email address hidden>
Date: Wed Dec 11 11:06:26 2013 -0500

    When a claim is rejected, explain why

    Previously, the Claim class offered #test which would return a Boolean
    based on availability of memory, disk, CPU and PCI devices.

    Now these tests are performed from the constructor and if any test
    fails, an exception is thrown. All failure reasons are described in the
    exception message. EG:

      "Insufficient compute resources: Free memory 8192.00 MB <
      requested 16384 MB; Free disk 45.00 GB < requested 50 GB; Free CPUs
      16.00 VCPUs < requested 17 VCPUs."

    Change-Id: I5bf9c81bba747a6c164144e60a92dc6da2f98f4c
    Closes-Bug: #1212768

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → icehouse-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: icehouse-3 → 2014.1
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.