diff options
| author | Akihiro Motoki <amotoki@gmail.com> | 2019-03-21 16:41:37 +0900 |
|---|---|---|
| committer | Vishal Manchanda <manchandavishal143@gmail.com> | 2019-03-27 18:23:39 +0000 |
| commit | a729170c9ff6c56dd2f73a3b899299bbc44ea80e (patch) | |
| tree | 7f83c082a232ceca3f5f88ea110b2963448d2064 | |
| parent | ca1e8f77ccbe502534412f20ff77c5c5033bc62c (diff) | |
| download | horizon-a729170c9ff6c56dd2f73a3b899299bbc44ea80e.tar.gz | |
network topology: handle port AZ correctly
Previously d3 version of the network topology view handles ports with
device_owner 'compute:nova' and 'compute:None' specially. This leads
to the situtaion that neutron ports with non-default AZ are not shown
properly. There is no reason to handle neutron ports with the default
AZ differently. What we would like to do is just to classify neutron
ports attached to nova servers.
This commit changes the logic to check device_owner of ports have
a prefix 'compute:'. Note that we also need to check if device_owner
is a string before checking the prefix.
Change-Id: I472bae9af3d14d8a31efcae8a7610b84c4f09c3d
Closes-Bug: #1820260
(cherry picked from commit ffbe0e2f2f96556f4476ea3873af1bc628c879ca)
| -rw-r--r-- | openstack_dashboard/static/js/horizon.networktopology.js | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/openstack_dashboard/static/js/horizon.networktopology.js b/openstack_dashboard/static/js/horizon.networktopology.js index 5083e1b55..dc0fee176 100644 --- a/openstack_dashboard/static/js/horizon.networktopology.js +++ b/openstack_dashboard/static/js/horizon.networktopology.js @@ -697,7 +697,7 @@ horizon.network_topology = { var device = self.find_by_id(port.device_id); var _network = self.find_by_id(port.network_id); if (angular.isDefined(device) && angular.isDefined(_network)) { - if (port.device_owner == 'compute:nova' || port.device_owner == 'compute:None') { + if (port.device_owner && port.device_owner.startsWith('compute:')) { _network.data.instances++; device.data.networks.push(_network.data); if (port.fixed_ips) { @@ -727,7 +727,8 @@ horizon.network_topology = { } self.new_link(self.find_by_id(port.device_id), self.find_by_id(port.network_id)); change = true; - } else if (angular.isDefined(_network) && port.device_owner == 'compute:nova') { + } else if (angular.isDefined(_network) && + port.device_owner && port.device_owner.startsWith('compute:')) { // Need to add a previously hidden node to the graph because it is // connected to more than 1 network if (_network.data.collapsed) { |