rename/remove osapi_compute_link_prefix config option

Bug #1046451 reported by Dan Prince
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Sathish Nagappan

Bug Description

The osapi_compute_link_prefix in Cinder is poorly named. It only seems to be used in the base ViewBuilder class which I'm not sure we actually use in Cinder (in which case we could just remove those methods and remove this option entirely).

In any case we should either remove or rename this option so that it sounds more 'volumish'.

Changed in cinder:
assignee: nobody → Sathish Nagappan (sathish-nagappan)
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/15797

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/15797
Committed: http://github.com/openstack/cinder/commit/842521cb1267413025c47248dae8cd622d1d104c
Submitter: Jenkins
Branch: master

commit 842521cb1267413025c47248dae8cd622d1d104c
Author: sathish-nagappan <email address hidden>
Date: Fri Nov 30 14:53:21 2012 -0800

    Rename Config osapi_compute_link_prefix to osapi_volume_base_URL

    fixes Bug #1046451

    Renamed the config option.

    Change-Id: Ia6eb057a11828bdc216e0bddbd29855285ba938c

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