Only show actions and primary label on primary node; hide error if healthy

app/assets/javascripts/geo_nodes.js

@@ -37,7 +37,9 @@ class GeoNodeStatus {
       this.$repositoriesFailed.html(status.repositories_failed_count);
       this.$lfsObjectsSynced.html(`${status.lfs_objects_synced_count}/${status.lfs_objects_count} (${status.lfs_objects_synced_in_percentage})`);
       this.$attachmentsSynced.html(`${status.attachments_synced_count}/${status.attachments_count} (${status.attachments_synced_in_percentage})`);
-      if (status.health !== 'Healthy') {
+      if (status.health === 'Healthy') {
+        this.$health.html('');
+      } else {
         this.$health.html(`<code class="geo-health">${status.health}</code>`);
       }
 

app/helpers/ee/geo_helper.rb

@@ -18,7 +18,7 @@ module EE
     def node_class(node)
       klass = []
       klass << 'js-geo-secondary-node' if node.secondary?
-      klass << ' node-disabled' unless node.enabled?
+      klass << 'node-disabled' unless node.enabled?
       klass
     end
 

app/views/admin/geo_nodes/index.html.haml

@@ -20,10 +20,10 @@
           .node-block
             = node_status_icon(node)
             %strong= node.url
-            - if node.primary?
-              .node-badge.primary-node Primary
             - if current_node?(node)
               .node-badge.current-node Current node
+            - if node.primary?
+              .node-badge.primary-node Primary
               %span.help-block Primary node
             - else
               .js-geo-node-status{ style: 'display: none' }
@@ -51,6 +51,7 @@
                 %p
                   .js-health
 
+          - if current_node?(node) == node.primary?
             .node-actions
               - if Gitlab::Geo.license_allows?
                 - if node.missing_oauth_application?