From a296850d58d9466acea9e4acc4582c073702400c Mon Sep 17 00:00:00 2001
From: Harshavardhana <harsha@minio.io>
Date: Mon, 24 Oct 2022 14:16:55 -0700
Subject: [PATCH] remove cache from configurations tab (#2410)

'cache' is not supported in server deployments
and is not useful to be present in the configurations tab.
---
 .../screens/Console/Configurations/utils.tsx  | 75 -------------------
 portal-ui/tests/permissions-2/settings.ts     |  3 -
 portal-ui/tests/utils/elements.ts             |  4 -
 3 files changed, 82 deletions(-)

diff --git a/portal-ui/src/screens/Console/Configurations/utils.tsx b/portal-ui/src/screens/Console/Configurations/utils.tsx
index 991de6ed6..a1cfa298e 100644
--- a/portal-ui/src/screens/Console/Configurations/utils.tsx
+++ b/portal-ui/src/screens/Console/Configurations/utils.tsx
@@ -15,7 +15,6 @@
 // along with this program.  If not, see <http://www.gnu.org/licenses/>.
 import React from "react";
 import PublicIcon from "@mui/icons-material/Public";
-import SdStorageIcon from "@mui/icons-material/SdStorage";
 import CompressIcon from "@mui/icons-material/Compress";
 import CodeIcon from "@mui/icons-material/Code";
 import LocalHospitalIcon from "@mui/icons-material/LocalHospital";
@@ -33,11 +32,6 @@ export const configurationElements: IElement[] = [
     configuration_id: "region",
     configuration_label: "Region",
   },
-  {
-    icon: <SdStorageIcon />,
-    configuration_id: "cache",
-    configuration_label: "Cache",
-  },
   {
     icon: <CompressIcon />,
     configuration_id: "compression",
@@ -104,75 +98,6 @@ export const fieldsConfigurations: any = {
       placeholder: "Enter custom notes if any",
     },
   ],
-  cache: [
-    {
-      name: "drives",
-      required: true,
-      label: "Drives",
-      tooltip:
-        'Mountpoints e.g. "/optane1" or "/optane2", you can write one per field',
-      type: "csv",
-      placeholder: "Enter Mount Point",
-    },
-    {
-      name: "expiry",
-      required: false,
-      label: "Expiry",
-      tooltip: 'Cache expiry duration in days e.g. "90"',
-      type: "number",
-      placeholder: "Enter Number of Days",
-    },
-    {
-      name: "quota",
-      required: false,
-      label: "Quota",
-      tooltip: 'Limit cache drive usage in percentage e.g. "90"',
-      type: "number",
-      placeholder: "Enter in %",
-    },
-    {
-      name: "exclude",
-      required: false,
-      label: "Exclude",
-      tooltip:
-        'Wildcard exclusion patterns e.g. "bucket/*.tmp" or "*.exe", you can write one per field',
-      type: "csv",
-      placeholder: "Enter Wildcard Exclusion Patterns",
-    },
-    {
-      name: "after",
-      required: false,
-      label: "After",
-      tooltip: "Minimum number of access before caching an object",
-      type: "number",
-      placeholder: "Enter Number of Attempts",
-    },
-    {
-      name: "watermark_low",
-      required: false,
-      label: "Watermark Low",
-      tooltip: "Watermark Low",
-      type: "number",
-      placeholder: "Enter Watermark Low",
-    },
-    {
-      name: "watermark_high",
-      required: false,
-      label: "Watermark High",
-      tooltip: "Watermark High",
-      type: "number",
-      placeholder: "Enter Watermark High",
-    },
-    {
-      name: "comment",
-      required: false,
-      label: "Comment",
-      tooltip: "You can add a comment to this setting",
-      type: "comment",
-      multiline: true,
-      placeholder: "Enter custom notes if any",
-    },
-  ],
   compression: [
     {
       name: "extensions",
diff --git a/portal-ui/tests/permissions-2/settings.ts b/portal-ui/tests/permissions-2/settings.ts
index 9b417ca71..0fe44f4a5 100644
--- a/portal-ui/tests/permissions-2/settings.ts
+++ b/portal-ui/tests/permissions-2/settings.ts
@@ -38,7 +38,6 @@ test("Settings window exists in Settings page", async (t) => {
 
 test("All vertical tab items exist", async (t) => {
   const settingsRegionTabExists = elements.settingsRegionTab.exists;
-  const settingsCacheTabExists = elements.settingsCacheTab.exists;
   const settingsCompressionTabExists = elements.settingsCompressionTab.exists;
   const settingsApiTabExists = elements.settingsApiTab.exists;
   const settingsHealTabExists = elements.settingsHealTab.exists;
@@ -53,8 +52,6 @@ test("All vertical tab items exist", async (t) => {
     .navigateTo("http://localhost:9090/settings/configurations")
     .expect(settingsRegionTabExists)
     .ok()
-    .expect(settingsCacheTabExists)
-    .ok()
     .expect(settingsCompressionTabExists)
     .ok()
     .expect(settingsApiTabExists)
diff --git a/portal-ui/tests/utils/elements.ts b/portal-ui/tests/utils/elements.ts
index 1ff824c38..985500205 100644
--- a/portal-ui/tests/utils/elements.ts
+++ b/portal-ui/tests/utils/elements.ts
@@ -163,10 +163,6 @@ export const settingsRegionTab = Selector(".MuiTab-root").withAttribute(
   "href",
   "/settings/configurations/region"
 );
-export const settingsCacheTab = Selector(".MuiTab-root").withAttribute(
-  "href",
-  "/settings/configurations/cache"
-);
 export const settingsCompressionTab = Selector(".MuiTab-root").withAttribute(
   "href",
   "/settings/configurations/compression"