The region_name was not passed into the the client causing volume commands to fail if there were multiple regions. Change-Id: I066dbbc4852f412e017daeeb16a3f186d3f91d2f Closes-Bug: #1241177
		
			
				
	
	
		
			70 lines
		
	
	
		
			2.0 KiB
		
	
	
	
		
			Python
		
	
	
	
	
	
			
		
		
	
	
			70 lines
		
	
	
		
			2.0 KiB
		
	
	
	
		
			Python
		
	
	
	
	
	
#   Copyright 2012-2013 OpenStack, LLC.
 | 
						|
#
 | 
						|
#   Licensed under the Apache License, Version 2.0 (the "License"); you may
 | 
						|
#   not use this file except in compliance with the License. You may obtain
 | 
						|
#   a copy of the License at
 | 
						|
#
 | 
						|
#        http://www.apache.org/licenses/LICENSE-2.0
 | 
						|
#
 | 
						|
#   Unless required by applicable law or agreed to in writing, software
 | 
						|
#   distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
 | 
						|
#   WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
 | 
						|
#   License for the specific language governing permissions and limitations
 | 
						|
#   under the License.
 | 
						|
#
 | 
						|
 | 
						|
import logging
 | 
						|
 | 
						|
from openstackclient.common import utils
 | 
						|
 | 
						|
 | 
						|
LOG = logging.getLogger(__name__)
 | 
						|
 | 
						|
DEFAULT_VOLUME_API_VERSION = '1'
 | 
						|
API_VERSION_OPTION = 'os_volume_api_version'
 | 
						|
API_NAME = "volume"
 | 
						|
API_VERSIONS = {
 | 
						|
    "1": "cinderclient.v1.client.Client"
 | 
						|
}
 | 
						|
 | 
						|
 | 
						|
def make_client(instance):
 | 
						|
    """Returns a volume service client."""
 | 
						|
    volume_client = utils.get_client_class(
 | 
						|
        API_NAME,
 | 
						|
        instance._api_version[API_NAME],
 | 
						|
        API_VERSIONS
 | 
						|
    )
 | 
						|
 | 
						|
    LOG.debug('instantiating volume client')
 | 
						|
 | 
						|
    # Set client http_log_debug to True if verbosity level is high enough
 | 
						|
    http_log_debug = utils.get_effective_log_level() <= logging.DEBUG
 | 
						|
 | 
						|
    client = volume_client(
 | 
						|
        username=instance._username,
 | 
						|
        api_key=instance._password,
 | 
						|
        project_id=instance._project_name,
 | 
						|
        auth_url=instance._auth_url,
 | 
						|
        cacert=instance._cacert,
 | 
						|
        insecure=instance._insecure,
 | 
						|
        region_name=instance._region_name,
 | 
						|
        http_log_debug=http_log_debug
 | 
						|
    )
 | 
						|
 | 
						|
    return client
 | 
						|
 | 
						|
 | 
						|
def build_option_parser(parser):
 | 
						|
    """Hook to add global options"""
 | 
						|
    parser.add_argument(
 | 
						|
        '--os-volume-api-version',
 | 
						|
        metavar='<volume-api-version>',
 | 
						|
        default=utils.env(
 | 
						|
            'OS_VOLUME_API_VERSION',
 | 
						|
            default=DEFAULT_VOLUME_API_VERSION),
 | 
						|
        help='Volume API version, default=' +
 | 
						|
             DEFAULT_VOLUME_API_VERSION +
 | 
						|
             ' (Env: OS_VOLUME_API_VERSION)')
 | 
						|
    return parser
 |