[Monitoring] Address some UI regressions with shard allocation (#29757)

* Address some UI regressions with shard allocation

* Renable the shard tests, and a couple others that are disabled

* PR feedback

x-pack/plugins/monitoring/public/components/elasticsearch/index/index.js

@@ -54,7 +54,7 @@ export const Index = ({
             ))}
           </EuiFlexGrid>
           <EuiSpacer size="m"/>
-          <ShardAllocation scope={scope} {...props} kbnUrl={kbnUrl} type="index" />
+          <ShardAllocation scope={scope} kbnUrl={kbnUrl} type="index" />
         </EuiPageContent>
       </EuiPageBody>
     </EuiPage>

x-pack/plugins/monitoring/public/components/elasticsearch/shard_allocation/_shard_allocation.scss

@@ -61,7 +61,6 @@ monitoring-shard-allocation {
     }
     &.unassigned {
       .title {
-        color: #999;
         display: none;
       }
     }
@@ -82,15 +81,6 @@ monitoring-shard-allocation {
     border-left: 1px solid $euiColorEmptyShade;
     position: relative;
     display: inline-block;
-
-    .shard-tooltip {
-      padding: 5px;
-      bottom: 25px;
-      left: 0;
-      position: absolute;
-      border: 1px solid $euiColorLightShade;
-      white-space: nowrap;
-    }
   }
 
   .legend {

x-pack/plugins/monitoring/public/components/elasticsearch/shard_allocation/components/shard.js

@@ -10,7 +10,7 @@ import React from 'react';
 import { calculateClass } from '../lib/calculate_class';
 import { vents } from '../lib/vents';
 import { i18n } from '@kbn/i18n';
-import { EuiTextColor } from '@elastic/eui';
+import { EuiToolTip, EuiBadge } from '@elastic/eui';
 
 function getColor(classes) {
   return classes.split(' ').reduce((color, cls) => {
@@ -20,7 +20,7 @@ function getColor(classes) {
 
     switch (cls) {
       case 'primary':
-        return 'ghost';
+        return 'hollow';
       case 'replica':
         return 'secondary';
       case 'relocation':
@@ -80,23 +80,24 @@ export class Shard extends React.Component {
 
   render() {
     const shard = this.props.shard;
-    let tooltip;
-    if (this.state.tooltipVisible) {
-      tooltip = (
-        <div
-          className="shard-tooltip"
-          data-test-subj="shardTooltip"
-          data-tooltip-content={this.props.shard.tooltip_message}
-        >
-          {this.props.shard.tooltip_message}
-        </div>
-      );
-    }
-
     const classes = calculateClass(shard);
     const color = getColor(classes);
     const classification = classes + ' ' + shard.shard;
 
+    let shardUi = (
+      <EuiBadge color={color}>
+        {shard.shard}
+      </EuiBadge>
+    );
+
+    if (this.state.tooltipVisible) {
+      shardUi = (
+        <EuiToolTip content={this.props.shard.tooltip_message} position="bottom" data-test-subj="shardTooltip">
+          <p>{shardUi}</p>
+        </EuiToolTip>
+      );
+    }
+
     // data attrs for automated testing verification
     return (
       <div
@@ -107,9 +108,7 @@ export class Shard extends React.Component {
         data-shard-classification={classification}
         data-test-subj="shardIcon"
       >
-        <EuiTextColor color={color}>
-          {tooltip}{shard.shard}
-        </EuiTextColor>
+        {shardUi}
       </div>
     );
   }

x-pack/plugins/monitoring/public/views/elasticsearch/index/index.js

@@ -81,12 +81,13 @@ uiRoutes.when('/elasticsearch/indices/:index', {
         }
 
         const shards = data.shards;
-        data.totalCount = shards.length;
-        data.showing = transformer(shards, data.nodes);
+        $scope.totalCount = shards.length;
+        $scope.showing = transformer(shards, data.nodes);
+        $scope.labels = labels.node;
         if (shards.some((shard) => shard.state === 'UNASSIGNED')) {
-          data.labels = labels.indexWithUnassigned;
+          $scope.labels = labels.indexWithUnassigned;
         } else {
-          data.labels = labels.index;
+          $scope.labels = labels.index;
         }
 
         this.renderReact(

x-pack/test/api_integration/apis/monitoring/elasticsearch/overview.js

@@ -14,7 +14,7 @@ export default function ({ getService }) {
   const supertest = getService('supertest');
   const esArchiver = getService('esArchiver');
 
-  describe.skip('overview', () => {
+  describe('overview', () => {
     describe('with green platinum cluster', () => {
       const archive = 'monitoring/singlecluster-green-platinum';
       const timeRange = {

x-pack/test/api_integration/apis/monitoring/elasticsearch_settings/set_collection_interval.js

@@ -9,7 +9,7 @@ import expect from 'expect.js';
 export default function ({ getService }) {
   const supertest = getService('supertest');
 
-  describe.skip('update collection_interval setting', () => {
+  describe('update collection_interval setting', () => {
     it('should set collection.interval to 10s', async () => {
       const { body } = await supertest
         .put('/api/monitoring/v1/elasticsearch_settings/set/collection_interval')

x-pack/test/functional/apps/monitoring/elasticsearch/shards.js

@@ -28,7 +28,7 @@ export default function ({ getService, getPageObjects }) {
       await tearDown();
     });
 
-    describe.skip('Shard Allocation Per Node', () => {
+    describe('Shard Allocation Per Node', () => {
       before(async () => {
         // start on cluster overview
         await PageObjects.monitoring.clickBreadcrumb('breadcrumbClusters');
@@ -95,7 +95,7 @@ export default function ({ getService, getPageObjects }) {
       });
     });
 
-    describe.skip('Shard Allocation Per Index', () => {
+    describe('Shard Allocation Per Index', () => {
       before(async () => {
         // start on cluster overview
         await PageObjects.monitoring.clickBreadcrumb('breadcrumbClusters');

x-pack/test/functional/services/monitoring/elasticsearch_shards.js

@@ -65,7 +65,7 @@ export function MonitoringElasticsearchShardsProvider({ getService }) {
 
     async showSystemIndices() {
       const checkboxEl = await testSubjects.find(SUBJ_SHOW_SYSTEM_INDICES);
-      const isChecked = await checkboxEl.getSpecAttribute('selected');
+      const isChecked = await checkboxEl.getAttribute('selected');
 
       if (!isChecked) {
         await testSubjects.click(SUBJ_SHOW_SYSTEM_INDICES);