openstack
/
python-troveclient
Code Issues Proposed changes

Fix getting server id for listing instances 

Just use 'server_id' from the instance response instead.

Story: 2008464
Task: 41496
Change-Id: Ib84dbd5fbd752ab7c7607ccb874663b01bfb7388
This commit is contained in:
Lingxian Kong 2020-12-23 13:21:55 +13:00
parent 77023b350e
commit 961ef31807
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
troveclient/osc/v1/database_instances.py
View File

@ -56,9 +56,6 @@ def get_instances_info(instances):
if 'addresses' not in instance_info:
instance_info['addresses'] = ''
if 'server' in instance_info:
instance_info['server_id'] = instance_info['server'].get('id')
if 'operating_status' not in instance_info:
# In case newer version python-troveclient is talking to older
# version trove.

2
troveclient/tests/osc/v1/test_database_instances.py
View File

@ -109,6 +109,7 @@ class TestInstanceList(TestInstances):
},
"tenant_id": tenant_id,
"access": {"is_public": False, "allowed_cidrs": []},
"server_id": server_id,
'server': {
'id': server_id
}
@ -157,6 +158,7 @@ class TestInstanceList(TestInstances):
},
"tenant_id": tenant_id,
"access": {"is_public": False, "allowed_cidrs": []},
"server_id": server_id,
'server': {
'id': server_id
}