openstack/trove
Code Issues Proposed changes

Merge "Have cluster-show list all ips"

Browse Source
This commit is contained in:
Jenkins
2016-11-21 11:03:38 +00:00
committed by Gerrit Code Review
parent ad51c7dc82 ac2d1bd8bb
commit 66f65ec5d1
2 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
releasenotes/notes/cluster_list_show_all_ips-3547635440.yaml Normal file
View File

@@ -0,0 +1,6 @@
---
fixes:
- The payload for cluster GET now returns ips
for all networks, not just the first one
found for each instance.
Bug 1642695

2
trove/cluster/views.py
View File

@@ -88,7 +88,7 @@ class ClusterView(object):
if self.load_servers and instance_ips:
instance_dict["ip"] = instance_ips
if instance.type in ip_to_be_published_for:
ip_list.append(instance_ips[0])
ip_list.extend(instance_ips)
if instance.type in instance_dict_to_be_published_for:
instances.append(instance_dict)
ip_list.sort()