Minor optimization in create_volume in HpSanISCSIDriver

Bug #1074201 reported by Nirmal Ranganathan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Wishlist
Nirmal Ranganathan

Bug Description

On every create_volume there are two additional calls made to the backend, one to collect the volume_info which is needed, the other is for the cluster_info which doesn't change with every create_volume, hence that info can be cached on driver load.

self.cluster_vip = self._cliq_get_cluster_vip(cluster_name)

Changed in cinder:
assignee: nobody → Nirmal Ranganathan (rnirmal)
status: New → In Progress
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/15285

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/15285
Committed: http://github.com/openstack/cinder/commit/88d6be0c66694021847574d2e255d69bcd0bda73
Submitter: Jenkins
Branch: master

commit 88d6be0c66694021847574d2e255d69bcd0bda73
Author: Nirmal Ranganathan <email address hidden>
Date: Fri Nov 2 11:49:17 2012 -0500

    Minor optimization in create_volume in HpSanISCSIDriver

    Removing extra call to getClusterInfo in HpSANISCSIDriver
    on every create_volume call. The cluster info is now
    being lazily stored in a local variable.

    Fixes bug 1074201

    Change-Id: Iadf8383dbf87a8e82c649c312c67978c7dd3df71

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