Coverage for Reservation pool lacks explicit test of create_computehost with ID for host

Bug #1261788 reported by Sylvain Bauza
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Blazar
Fix Released
Medium
Sylvain Bauza

Bug Description

As Reservation Pool is a wrapper on top of Nova, there is one section of code which checks if an host is part of the freepool.
The host value is directly given as parameter, so we need to explicitely mention we only accept name, not UUID and do the coverage accordingly.

Changed in climate:
importance: Undecided → High
assignee: nobody → Sylvain Bauza (sylvain-bauza)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to climate (master)

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

Changed in climate:
status: New → In Progress
Changed in climate:
importance: High → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to climate (master)

Reviewed: https://review.openstack.org/62685
Committed: https://git.openstack.org/cgit/stackforge/climate/commit/?id=d8a2a1314f3cda205d2a478c84088379b7041a65
Submitter: Jenkins
Branch: master

commit d8a2a1314f3cda205d2a478c84088379b7041a65
Author: Sylvain Bauza <email address hidden>
Date: Tue Dec 17 16:28:32 2013 +0100

    Fix coverage for add_computehost with ID of host

    add_computehost returns HostNotInFreepool if UUID is given for host
    because freepool.hosts only contains names of hosts

    Change-Id: I7ea9b240fa7beb60c532175794da2f30a551a985
    Closes-Bug: #1261788

Changed in climate:
status: In Progress → Fix Committed
Dina Belova (dbelova)
Changed in climate:
milestone: none → 0.1.0
status: Fix Committed → Fix Released
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.