getClusterInfo is very time-consuming on LHN driver

Bug #1276388 reported by Jay Lee
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Jay Lee

Bug Description

release: master, stable/havana
volume driver: cinder.volume.san.HpSanISCSIDriver

cinder-volume running a _report_driver_status method periodically.
_report_driver_status called get_volume_stats,

self.driver.get_volume_stats(refresh=True)

When using HpSanISCSIDriver,
_get_volume_stats called _update_backend_status.

    def _update_backend_status(self):
        data = {}
        backend_name = self.configuration.safe_get('volume_backend_name')
        data['volume_backend_name'] = backend_name or self.__class__.__name__
        data['driver_version'] = self.VERSION
        data['reserved_percentage'] = 0
        data['storage_protocol'] = 'iSCSI'
        data['vendor_name'] = 'Hewlett-Packard'

        result_xml = self._cliq_run_xml("getClusterInfo", {})

This method call getClusterInfo without searchdepth option.
So It shows every volume, snapshot information, it takes a long long time.
The San storage has more volumes, response time is slower.

In my case, without searchdepth, getClusterInfo output after 546 secs.
with searchdepth=1, output is 5 secs less.

_update_backend_status need "spaceTotal", "unprovisionedSpace" information,
not a volume, snapshot information. so getClusterInfo should be executed with
searchdepth.

Tags: drivers
Jay Lee (hyangii)
Changed in cinder:
assignee: nobody → Jay Lee (hyangii)
summary: - getClusterInfo is very time-consuming
+ getClusterInfo is very time-consuming on LHN driver
tags: added: drivers
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/72041

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

Reviewed: https://review.openstack.org/72041
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=251705354914d4af761965bf32b0e8fd948951c3
Submitter: Jenkins
Branch: master

commit 251705354914d4af761965bf32b0e8fd948951c3
Author: Jay Lee <email address hidden>
Date: Sat Feb 8 01:43:27 2014 +0000

    Add searchDepth when getClusterInfo called.

    Without searchDepth, getClusterInfo shows every volume and
    snapshot. It's an unused information and takes very long time.

    Change-Id: Id590e3023f4446f01b965baf4e8ac61980101067
    Closes-Bug: 1276388

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → icehouse-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: icehouse-3 → 2014.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.