VCDriver - Stats update does not ignore hosts in maintenance mode

Bug #1316674 reported by Sagar Ratnakara Nikam
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
Sagar Ratnakara Nikam
Icehouse
Fix Released
Undecided
Unassigned

Bug Description

Cluster has 2 hosts, with each host having 24 cores.
Both hosts are active.
Stats update is correct

Snippet of nova-compute.log

-05-07 03:14:28.001 AUDIT nova.compute.resource_tracker [-] Free ram (MB): 57582
2014-05-07 03:14:28.001 AUDIT nova.compute.resource_tracker [-] Free disk (GB): 99
2014-05-07 03:14:28.002 AUDIT nova.compute.resource_tracker [-] Free VCPUS: 48
2014-05-07 03:14:28.021 INFO nova.compute.resource_tracker [-] Compute_service record updated for sagar-devstack:domain-c162(Demo-Pulsar-Cluster-DRS)
2014-05-07 03:14:28.021 DEBUG nova.openstack.common.lockutils [-] Semaphore / lock released "update_available_resource" from (pid=28072) inner /opt/stack/nova/nova/openstack/common/lockutils.py:252

Put one of the hosts in Maintenance mode, stats update does not ignore host in maintenance mode and the free vCPUS is still 48.
It should be 24

Changed in nova:
assignee: nobody → Sagar Ratnakara Nikam (sagar-r-nikam)
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix proposed to nova (master)

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

Changed in nova:
status: New → In Progress
Tracy Jones (tjones-i)
Changed in nova:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/92583
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=4cb139fb99814f04b9e4595d7e2efa34e7992826
Submitter: Jenkins
Branch: master

commit 4cb139fb99814f04b9e4595d7e2efa34e7992826
Author: Sagar Ratnakara Nikam <email address hidden>
Date: Wed May 7 18:47:45 2014 +0530

    VCDriver - Ignore host in Maintenance mode in stats update

    The VCDriver during stats update checks only for connected hosts.
    However hosts which are in connected state to the Cluster can
    be set to Maintenance mode.
    The cluster does not boot instances to hosts which are in
    maintenance mode.
    Hence the VCDriver should check for hosts which are in
    maintenance mode and ignore them for stats update.
    VISDK API returns True for the property inMaintenanceMode for
    Hosts which are in maintenance mode

    Change-Id: Ieaf7530ed2d98c59c5450e8cb75e209823193d49
    Closes-Bug: #1316674

Changed in nova:
status: In Progress → Fix Committed
tags: added: icehouse-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/icehouse)

Fix proposed to branch: stable/icehouse
Review: https://review.openstack.org/98009

Thierry Carrez (ttx)
Changed in nova:
milestone: none → juno-1
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/icehouse)

Reviewed: https://review.openstack.org/98009
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=c75430580b0e2aa7108658bf4d4f9153ab1befd7
Submitter: Jenkins
Branch: stable/icehouse

commit c75430580b0e2aa7108658bf4d4f9153ab1befd7
Author: Sagar Ratnakara Nikam <email address hidden>
Date: Wed May 7 18:47:45 2014 +0530

    VCDriver - Ignore host in Maintenance mode in stats update

    The VCDriver during stats update checks only for connected hosts.
    However hosts which are in connected state to the Cluster can
    be set to Maintenance mode.
    The cluster does not boot instances to hosts which are in
    maintenance mode.
    Hence the VCDriver should check for hosts which are in
    maintenance mode and ignore them for stats update.
    VISDK API returns True for the property inMaintenanceMode for
    Hosts which are in maintenance mode

    Change-Id: Ieaf7530ed2d98c59c5450e8cb75e209823193d49
    Closes-Bug: #1316674
    (cherry picked from commit 4cb139fb99814f04b9e4595d7e2efa34e7992826)

tags: added: in-stable-icehouse
Chuck Short (zulcss)
tags: removed: icehouse-backport-potential
Thierry Carrez (ttx)
Changed in nova:
milestone: juno-1 → 2014.2
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.