powervm: get_host_uptime output doesn't match convention

Bug #1196630 reported by Mathew Odden
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Matt Riedemann

Bug Description

Change I67aaf6a5: Implement get_host_uptime for powervm driver was just merged, but reports the uptime in a format that isn't consistent with other hypervisors. The driver should follow convention and report uptime as the full returned string. An example of this from the libvirt driver:

root@idpx31:~# nova hypervisor-uptime 1
+---------------------+-----------------------------------------------------------------------+
| Property | Value |
+---------------------+-----------------------------------------------------------------------+
| uptime | 13:56:17 up 10 days, 23:07, 1 user, load average: 2.36, 2.23, 2.25 |
| | |
| id | 1 |
| hypervisor_hostname | ************ |
+---------------------+-----------------------------------------------------------------------+

Tags: powervm
Mathew Odden (locke105)
tags: added: powervm
Matt Riedemann (mriedem)
Changed in nova:
importance: Undecided → Low
status: New → Confirmed
assignee: nobody → Matt Riedemann (mriedem)
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/35169

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

Reviewed: https://review.openstack.org/35169
Committed: http://github.com/openstack/nova/commit/84c5e4a08a668a3d691374a1f396d4e761094ec4
Submitter: Jenkins
Branch: master

commit 84c5e4a08a668a3d691374a1f396d4e761094ec4
Author: Matt Riedemann <email address hidden>
Date: Mon Jul 1 11:20:26 2013 -0700

    powervm: make get_host_uptime output consistent with other virt drivers

    The powervm driver was filtering out part of the uptime information so
    that only system uptime was returned, not the number of users, load,
    etc. The hypervisor API sample and the libvirt driver provide the full
    uptime output, so this patch makes the powervm driver follow that
    convention.

    Fixes bug 1196630

    Change-Id: I24134270cab9df97ff59c06e057406b4aa09992a

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