Fix the total_capacity_gb in CloudByte iSCSI cinder driver

Bug #1420746 reported by yogeshprasad
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
yogeshprasad

Bug Description

CloudByte iSCSI cinder driver doesn't display the proper value for total_capacity_gb.

Please find the logs.

2015-02-11 16:31:58.702 14405 INFO cinder.volume.drivers.cloudbyte.cloudbyte [req-1623bb6b-922e-4779-b2f6-d1e011da4a76 - - - - -] CloudByte API executed successfully for command [listTsm].
2015-02-11 16:31:58.703 14405 DEBUG cinder.volume.drivers.cloudbyte.cloudbyte [req-1623bb6b-922e-4779-b2f6-d1e011da4a76 - - - - -] CloudByte driver stats: [{'volume_backend_name': 'cb-gold', 'free_capacity_gb': 197.0, 'driver_version': '1.0.0', 'total_capacity_gb': '1T', 'reserved_percentage': 0, 'vendor_name': 'CloudByte', 'storage_protocol': 'iSCSI'}]. get_volume_stats /opt/stack/cinder/cinder/volume/drivers/cloudbyte/cloudbyte.py:951

Changed in cinder:
assignee: nobody → yogeshprasad (yogesh-prasad)
Changed in cinder:
status: New → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

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

Reviewed: https://review.openstack.org/154878
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=59c992919064b06daa3c0ac7092d438e527ff7f7
Submitter: Jenkins
Branch: master

commit 59c992919064b06daa3c0ac7092d438e527ff7f7
Author: yogeshprasad <email address hidden>
Date: Wed Feb 11 19:22:33 2015 +0530

    Fixes total_capacity_gb value in CloudByte driver.

    CloudByte storage returns total capacity value in two formats. One is
    numeric and the other is alphanumeric.

    Previously data['total_capacity_gb'] was referring the alphanumeric value
    & now it is referring to the correct numeric value.

    In addition, changes were made to test_cloudbyte.py to unit test
    free_capacity_gb and total_capacity_gb.

    Closes-Bug: 1420746

    Change-Id: I601c2a78c1727bc815771ca26aea928fa2887d09

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