From 5303539f5b165f7b7a228d6e70e658c1fca3383f Mon Sep 17 00:00:00 2001
From: neptunian <sandra.gonzales@elastic.co>
Date: Tue, 21 Apr 2020 13:05:09 -0400
Subject: [PATCH 1/2] remove endpoint handles unremovable packages

---
 x-pack/plugins/ingest_manager/common/types/models/epm.ts    | 1 +
 x-pack/plugins/ingest_manager/server/saved_objects.ts       | 1 +
 .../ingest_manager/server/services/epm/packages/install.ts  | 6 +++++-
 .../ingest_manager/server/services/epm/packages/remove.ts   | 4 +++-
 4 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/x-pack/plugins/ingest_manager/common/types/models/epm.ts b/x-pack/plugins/ingest_manager/common/types/models/epm.ts
index 53ad0310ea6134..fea9da95529efd 100644
--- a/x-pack/plugins/ingest_manager/common/types/models/epm.ts
+++ b/x-pack/plugins/ingest_manager/common/types/models/epm.ts
@@ -57,6 +57,7 @@ export interface RegistryPackage {
   icons?: RegistryImage[];
   assets?: string[];
   internal?: boolean;
+  removable?: boolean;
   format_version: string;
   datasets?: Dataset[];
   datasources?: RegistryDatasource[];
diff --git a/x-pack/plugins/ingest_manager/server/saved_objects.ts b/x-pack/plugins/ingest_manager/server/saved_objects.ts
index dc0b4695603e41..0a7229b1f28072 100644
--- a/x-pack/plugins/ingest_manager/server/saved_objects.ts
+++ b/x-pack/plugins/ingest_manager/server/saved_objects.ts
@@ -150,6 +150,7 @@ export const savedObjectMappings = {
       name: { type: 'keyword' },
       version: { type: 'keyword' },
       internal: { type: 'boolean' },
+      removable: { type: 'boolean' },
       es_index_patterns: {
         dynamic: false,
         type: 'object',
diff --git a/x-pack/plugins/ingest_manager/server/services/epm/packages/install.ts b/x-pack/plugins/ingest_manager/server/services/epm/packages/install.ts
index 0a7642752b3e98..be3982746ff663 100644
--- a/x-pack/plugins/ingest_manager/server/services/epm/packages/install.ts
+++ b/x-pack/plugins/ingest_manager/server/services/epm/packages/install.ts
@@ -90,7 +90,7 @@ export async function installPackage(options: {
   // TODO: change epm API to /packageName/version so we don't need to do this
   const [pkgName, pkgVersion] = pkgkey.split('-');
   const registryPackageInfo = await Registry.fetchInfo(pkgName, pkgVersion);
-  const { internal = false } = registryPackageInfo;
+  const { internal = false, removable = true } = registryPackageInfo;
 
   const installKibanaAssetsPromise = installKibanaAssets({
     savedObjectsClient,
@@ -127,6 +127,7 @@ export async function installPackage(options: {
     pkgName,
     pkgVersion,
     internal,
+    removable,
     toSaveAssetRefs,
     toSaveESIndexPatterns,
   });
@@ -158,6 +159,7 @@ export async function saveInstallationReferences(options: {
   pkgName: string;
   pkgVersion: string;
   internal: boolean;
+  removable: boolean;
   toSaveAssetRefs: AssetReference[];
   toSaveESIndexPatterns: Record<string, string>;
 }) {
@@ -166,6 +168,7 @@ export async function saveInstallationReferences(options: {
     pkgName,
     pkgVersion,
     internal,
+    removable,
     toSaveAssetRefs,
     toSaveESIndexPatterns,
   } = options;
@@ -191,6 +194,7 @@ export async function saveInstallationReferences(options: {
       name: pkgName,
       version: pkgVersion,
       internal,
+      removable,
     },
     { id: pkgName, overwrite: true }
   );
diff --git a/x-pack/plugins/ingest_manager/server/services/epm/packages/remove.ts b/x-pack/plugins/ingest_manager/server/services/epm/packages/remove.ts
index a30acb97b99cf0..7c8a57160fc0a9 100644
--- a/x-pack/plugins/ingest_manager/server/services/epm/packages/remove.ts
+++ b/x-pack/plugins/ingest_manager/server/services/epm/packages/remove.ts
@@ -20,7 +20,9 @@ export async function removeInstallation(options: {
   // TODO:  the epm api should change to /name/version so we don't need to do this
   const [pkgName] = pkgkey.split('-');
   const installation = await getInstallation({ savedObjectsClient, pkgName });
-  const installedObjects = installation?.installed || [];
+  if (!installation) throw new Error('integration does not exist');
+  if (installation.removable === false) throw new Error('this integration is not removable');
+  const installedObjects = installation.installed || [];
 
   // Delete the manager saved object with references to the asset objects
   // could also update with [] or some other state

From d921fab8effeb9fccaff8322228a6b4d6e369a5c Mon Sep 17 00:00:00 2001
From: neptunian <sandra.gonzales@elastic.co>
Date: Tue, 21 Apr 2020 14:05:03 -0400
Subject: [PATCH 2/2] adjust UI to disallow removing of unremovable packages

---
 .../sections/epm/screens/detail/content.tsx   | 12 +++++--
 .../epm/screens/detail/settings_panel.tsx     | 36 ++++++++++++++-----
 .../server/services/epm/packages/remove.ts    |  3 +-
 3 files changed, 39 insertions(+), 12 deletions(-)

diff --git a/x-pack/plugins/ingest_manager/public/applications/ingest_manager/sections/epm/screens/detail/content.tsx b/x-pack/plugins/ingest_manager/public/applications/ingest_manager/sections/epm/screens/detail/content.tsx
index 0d4b3958953225..a3d24e7806f343 100644
--- a/x-pack/plugins/ingest_manager/public/applications/ingest_manager/sections/epm/screens/detail/content.tsx
+++ b/x-pack/plugins/ingest_manager/public/applications/ingest_manager/sections/epm/screens/detail/content.tsx
@@ -50,10 +50,18 @@ export function Content(props: ContentProps) {
 
 type ContentPanelProps = PackageInfo & Pick<DetailParams, 'panel'>;
 export function ContentPanel(props: ContentPanelProps) {
-  const { panel, name, version, assets, title } = props;
+  const { panel, name, version, assets, title, removable } = props;
   switch (panel) {
     case 'settings':
-      return <SettingsPanel name={name} version={version} assets={assets} title={title} />;
+      return (
+        <SettingsPanel
+          name={name}
+          version={version}
+          assets={assets}
+          title={title}
+          removable={removable}
+        />
+      );
     case 'data-sources':
       return <DataSourcesPanel name={name} version={version} />;
     case 'overview':
diff --git a/x-pack/plugins/ingest_manager/public/applications/ingest_manager/sections/epm/screens/detail/settings_panel.tsx b/x-pack/plugins/ingest_manager/public/applications/ingest_manager/sections/epm/screens/detail/settings_panel.tsx
index ff7ecf97714b6b..f947466caf4b09 100644
--- a/x-pack/plugins/ingest_manager/public/applications/ingest_manager/sections/epm/screens/detail/settings_panel.tsx
+++ b/x-pack/plugins/ingest_manager/public/applications/ingest_manager/sections/epm/screens/detail/settings_panel.tsx
@@ -13,8 +13,14 @@ import { InstallStatus, PackageInfo } from '../../../../types';
 import { InstallationButton } from './installation_button';
 import { useGetDatasources } from '../../../../hooks';
 
+const NoteLabel = () => (
+  <FormattedMessage
+    id="xpack.ingestManager.integrations.settings.packageUninstallNoteDescription.packageUninstallNoteLabel"
+    defaultMessage="Note:"
+  />
+);
 export const SettingsPanel = (
-  props: Pick<PackageInfo, 'assets' | 'name' | 'title' | 'version'>
+  props: Pick<PackageInfo, 'assets' | 'name' | 'title' | 'version' | 'removable'>
 ) => {
   const getPackageInstallStatus = useGetPackageInstallStatus();
   const { data: datasourcesData } = useGetDatasources({
@@ -22,10 +28,9 @@ export const SettingsPanel = (
     page: 1,
     kuery: `datasources.package.name:${props.name}`,
   });
-  const { name, title } = props;
+  const { name, title, removable } = props;
   const packageInstallStatus = getPackageInstallStatus(name);
   const packageHasDatasources = !!datasourcesData?.total;
-
   return (
     <EuiText>
       <EuiTitle>
@@ -89,12 +94,12 @@ export const SettingsPanel = (
           <p>
             <InstallationButton
               {...props}
-              disabled={!datasourcesData ? true : packageHasDatasources}
+              disabled={!datasourcesData || removable === false ? true : packageHasDatasources}
             />
           </p>
         </EuiFlexItem>
       </EuiFlexGroup>
-      {packageHasDatasources && (
+      {packageHasDatasources && removable === true && (
         <p>
           <FormattedMessage
             id="xpack.ingestManager.integrations.settings.packageUninstallNoteDescription.packageUninstallNoteDetail"
@@ -103,10 +108,23 @@ export const SettingsPanel = (
               title,
               strongNote: (
                 <strong>
-                  <FormattedMessage
-                    id="xpack.ingestManager.integrations.settings.packageUninstallNoteDescription.packageUninstallNoteLabel"
-                    defaultMessage="Note:"
-                  />
+                  <NoteLabel />
+                </strong>
+              ),
+            }}
+          />
+        </p>
+      )}
+      {removable === false && (
+        <p>
+          <FormattedMessage
+            id="xpack.ingestManager.integrations.settings.packageUninstallNoteDescription.packageUninstallUninstallableNoteDetail"
+            defaultMessage="{strongNote} The {title} integration is installed by default and cannot be removed."
+            values={{
+              title,
+              strongNote: (
+                <strong>
+                  <NoteLabel />
                 </strong>
               ),
             }}
diff --git a/x-pack/plugins/ingest_manager/server/services/epm/packages/remove.ts b/x-pack/plugins/ingest_manager/server/services/epm/packages/remove.ts
index 7c8a57160fc0a9..e5fac9b38a3c19 100644
--- a/x-pack/plugins/ingest_manager/server/services/epm/packages/remove.ts
+++ b/x-pack/plugins/ingest_manager/server/services/epm/packages/remove.ts
@@ -21,7 +21,8 @@ export async function removeInstallation(options: {
   const [pkgName] = pkgkey.split('-');
   const installation = await getInstallation({ savedObjectsClient, pkgName });
   if (!installation) throw new Error('integration does not exist');
-  if (installation.removable === false) throw new Error('this integration is not removable');
+  if (installation.removable === false)
+    throw new Error(`The ${pkgName} integration is installed by default and cannot be removed`);
   const installedObjects = installation.installed || [];
 
   // Delete the manager saved object with references to the asset objects