From 52ad59b7f6cd206a2a31e98bc25528f315828fe5 Mon Sep 17 00:00:00 2001
From: Josh Dover <me@joshdover.com>
Date: Wed, 6 May 2020 14:25:44 -0600
Subject: [PATCH] [7.8] Fix logstash integration with monitoring (#65165)
 (#65512)

---
 x-pack/plugins/logstash/kibana.json                       | 1 +
 x-pack/plugins/logstash/public/application/index.tsx      | 8 ++------
 x-pack/plugins/logstash/public/plugin.ts                  | 3 ++-
 .../public/services/monitoring/monitoring_service.js      | 8 +++++---
 4 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/x-pack/plugins/logstash/kibana.json b/x-pack/plugins/logstash/kibana.json
index 97dbf58865a882..1eb325dcc1610b 100644
--- a/x-pack/plugins/logstash/kibana.json
+++ b/x-pack/plugins/logstash/kibana.json
@@ -9,6 +9,7 @@
   ],
   "optionalPlugins": [
     "home",
+    "monitoring",
     "security"
   ],
   "server": true,
diff --git a/x-pack/plugins/logstash/public/application/index.tsx b/x-pack/plugins/logstash/public/application/index.tsx
index 438038d6c885e0..3588e1f6b2417d 100644
--- a/x-pack/plugins/logstash/public/application/index.tsx
+++ b/x-pack/plugins/logstash/public/application/index.tsx
@@ -31,16 +31,12 @@ import * as Breadcrumbs from './breadcrumbs';
 export const renderApp = async (
   core: CoreStart,
   { basePath, element, setBreadcrumbs }: ManagementAppMountParams,
+  isMonitoringEnabled: boolean,
   licenseService$: Observable<any>
 ) => {
   const logstashLicenseService = await licenseService$.pipe(first()).toPromise();
   const clusterService = new ClusterService(core.http);
-  const monitoringService = new MonitoringService(
-    core.http,
-    // When monitoring is migrated this should be fetched from monitoring's plugin contract
-    core.injectedMetadata.getInjectedVar('monitoringUiEnabled'),
-    clusterService
-  );
+  const monitoringService = new MonitoringService(core.http, isMonitoringEnabled, clusterService);
   const pipelinesService = new PipelinesService(core.http, monitoringService);
   const pipelineService = new PipelineService(core.http, pipelinesService);
   const upgradeService = new UpgradeService(core.http);
diff --git a/x-pack/plugins/logstash/public/plugin.ts b/x-pack/plugins/logstash/public/plugin.ts
index 91d1a39d3970cf..7fbed5b3b86029 100644
--- a/x-pack/plugins/logstash/public/plugin.ts
+++ b/x-pack/plugins/logstash/public/plugin.ts
@@ -49,8 +49,9 @@ export class LogstashPlugin implements Plugin<void, void, SetupDeps> {
       mount: async params => {
         const [coreStart] = await core.getStartServices();
         const { renderApp } = await import('./application');
+        const isMonitoringEnabled = 'monitoring' in plugins;
 
-        return renderApp(coreStart, params, logstashLicense$);
+        return renderApp(coreStart, params, isMonitoringEnabled, logstashLicense$);
       },
     });
 
diff --git a/x-pack/plugins/logstash/public/services/monitoring/monitoring_service.js b/x-pack/plugins/logstash/public/services/monitoring/monitoring_service.js
index d551f4fba61d2e..4db2838cb53549 100755
--- a/x-pack/plugins/logstash/public/services/monitoring/monitoring_service.js
+++ b/x-pack/plugins/logstash/public/services/monitoring/monitoring_service.js
@@ -9,14 +9,14 @@ import { ROUTES, MONITORING } from '../../../common/constants';
 import { PipelineListItem } from '../../models/pipeline_list_item';
 
 export class MonitoringService {
-  constructor(http, monitoringUiEnabled, clusterService) {
+  constructor(http, isMonitoringEnabled, clusterService) {
     this.http = http;
-    this.monitoringUiEnabled = monitoringUiEnabled;
+    this._isMonitoringEnabled = isMonitoringEnabled;
     this.clusterService = clusterService;
   }
 
   isMonitoringEnabled() {
-    return this.monitoringUiEnabled;
+    return this._isMonitoringEnabled;
   }
 
   getPipelineList() {
@@ -27,6 +27,8 @@ export class MonitoringService {
     return this.clusterService
       .loadCluster()
       .then(cluster => {
+        // This API call should live within the Monitoring plugin
+        // https://github.com/elastic/kibana/issues/63931
         const url = `${ROUTES.MONITORING_API_ROOT}/v1/clusters/${cluster.uuid}/logstash/pipeline_ids`;
         const now = moment.utc();
         const body = JSON.stringify({