ui: [BUGFIX] Ensure we show the correct count of instances for each n…

…ode (#9749) * Add MeshServiceInstances property to node model * Use MeshServiceInstances property * Make sure we show the 'No * checks' if Checks.length is zero
hashicorp · Apr 21, 2021 · f58d97b · f58d97b
1 parent fe65a42
commit f58d97b
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 5 deletions.
diff --git a/.changelog/9749.txt b/.changelog/9749.txt
@@ -0,0 +1,3 @@
+```release-note:bug
+ui: Exclude proxies when showing the total number of instances on a node.
+```
diff --git a/ui/packages/consul-ui/app/components/consul/instance-checks/index.hbs b/ui/packages/consul-ui/app/components/consul/instance-checks/index.hbs
@@ -1,4 +1,3 @@
-{{#if (gt @items.length 0)}}
   {{#if (eq this.healthCheck.check 'empty') }}
     <dl class={{this.healthCheck.check}}>
       <dt>
@@ -28,5 +27,4 @@
       <dd>{{this.healthCheck.count}}/{{@items.length}} {{@type}} checks {{this.healthCheck.status}}</dd>
     </dl>
     {{/if}}
-  {{/if}}
-{{/if}}
+  {{/if}}
diff --git a/ui/packages/consul-ui/app/components/consul/node/list/index.hbs b/ui/packages/consul-ui/app/components/consul/node/list/index.hbs
@@ -30,7 +30,7 @@ as |item index|>
     <span class="leader" data-test-leader={{@leader.Address}}>Leader</span>
   {{/if}}
     <span>
-      {{format-number item.Services.length}} {{pluralize item.Services.length 'Service' without-count=true}}
+      {{format-number item.MeshServiceInstances.length}} {{pluralize item.MeshServiceInstances.length 'Service' without-count=true}}
     </span>
     <dl>
       <dt>

diff --git a/ui/packages/consul-ui/app/models/node.js b/ui/packages/consul-ui/app/models/node.js
@@ -1,5 +1,6 @@
 import Model, { attr, hasMany } from '@ember-data/model';
 import { computed } from '@ember/object';
+import { filter } from '@ember/object/computed';
 import { fragmentArray } from 'ember-data-model-fragments/attributes';
 
 export const PRIMARY_KEY = 'uid';
@@ -18,8 +19,13 @@ export default class Node extends Model {
   @attr() meta; // {}
   @attr() Meta; // {}
   @attr() TaggedAddresses; // {lan, wan}
+  // Services are reshaped to a different shape to what you sometimes get from
+  // the response, see models/node.js
   @hasMany('service-instance') Services; // TODO: Rename to ServiceInstances
   @fragmentArray('health-check') Checks;
+  // MeshServiceInstances are all instances that aren't connect-proxies this
+  // currently includes gateways as these need to show up in listings
+  @filter('Services', item => item.Service.Kind !== 'connect-proxy') MeshServiceInstances;
 
   @computed('Checks.[]', 'ChecksCritical', 'ChecksPassing', 'ChecksWarning')
   get Status() {

diff --git a/ui/packages/consul-ui/app/templates/dc/nodes/show/services.hbs b/ui/packages/consul-ui/app/templates/dc/nodes/show/services.hbs
@@ -24,7 +24,7 @@
     )
   )
 
-  (reject-by 'Service.Kind' 'connect-proxy' item.Services)
+  item.MeshServiceInstances
 
 as |sort filters items|}}
 <div class="tab-section">