openstack
/
ceilometer
Code Issues Proposed changes

Volume discovery minimum 3.43 api version 

Set the volume api version to 3.43 to make
sure we get 3.41 change where it exposes
the user_id for snapshots.

This will fix the volume.snapshot.size metric
that is not working when trying to access the
user_id throws a AttributeError.

Change-Id: I2414717e7c9e64b0092fe93c97a062a48dbf09a8
Closes-Bug: 1794475
This commit is contained in:
Tobias Urdin 2018-09-26 17:25:01 +02:00
parent af18f2be7b
commit be524d2b74
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ceilometer/volume/discovery.py
View File

@ -28,8 +28,10 @@ class _BaseDiscovery(plugin_base.DiscoveryBase):
def __init__(self, conf):
super(_BaseDiscovery, self).__init__(conf)
creds = conf.service_credentials
# NOTE(tobias-urdin): We set 3.43 (the maximum for Pike) because
# we need atleast 3.41 to get user_id on snapshots.
self.client = cinder_client.Client(
version='3',
version='3.43',
session=keystone_client.get_session(conf),
region_name=creds.region_name,
interface=creds.interface,