From 6d6589b662832d044dc4f4ca872b77ee3d7409af Mon Sep 17 00:00:00 2001
From: suzhou <suzhou@amazon.com>
Date: Tue, 10 Jan 2023 11:10:16 +0800
Subject: [PATCH 01/10] feat: some ui changes

Signed-off-by: suzhou <suzhou@amazon.com>
---
 .../pages/CreateIndex/components/IndexMapping/IndexMapping.tsx  | 2 +-
 .../__snapshots__/CreateIndexTemplate.test.tsx.snap             | 2 +-
 .../containers/TemplateDetail/TemplateDetail.tsx                | 1 -
 3 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/public/pages/CreateIndex/components/IndexMapping/IndexMapping.tsx b/public/pages/CreateIndex/components/IndexMapping/IndexMapping.tsx
index 2fe2cb7a3..d4cfeed79 100644
--- a/public/pages/CreateIndex/components/IndexMapping/IndexMapping.tsx
+++ b/public/pages/CreateIndex/components/IndexMapping/IndexMapping.tsx
@@ -229,7 +229,7 @@ const IndexMapping = (
                   });
                 }}
               >
-                See previous settings
+                See previous mappings
               </EuiButton>
               <EuiSpacer />
             </>
diff --git a/public/pages/CreateIndexTemplate/containers/CreateIndexTemplate/__snapshots__/CreateIndexTemplate.test.tsx.snap b/public/pages/CreateIndexTemplate/containers/CreateIndexTemplate/__snapshots__/CreateIndexTemplate.test.tsx.snap
index 10df6b780..3d704df9a 100644
--- a/public/pages/CreateIndexTemplate/containers/CreateIndexTemplate/__snapshots__/CreateIndexTemplate.test.tsx.snap
+++ b/public/pages/CreateIndexTemplate/containers/CreateIndexTemplate/__snapshots__/CreateIndexTemplate.test.tsx.snap
@@ -23,7 +23,7 @@ exports[`<CreateIndexTemplate /> spec it goes to templates page when click cance
         style="flex-direction: row;"
       >
         <button
-          class="euiButton euiButton--primary euiButton--fill"
+          class="euiButton euiButton--primary"
           style="margin-right: 20px;"
           type="button"
         >
diff --git a/public/pages/CreateIndexTemplate/containers/TemplateDetail/TemplateDetail.tsx b/public/pages/CreateIndexTemplate/containers/TemplateDetail/TemplateDetail.tsx
index 3ac2bf9aa..74d697b47 100644
--- a/public/pages/CreateIndexTemplate/containers/TemplateDetail/TemplateDetail.tsx
+++ b/public/pages/CreateIndexTemplate/containers/TemplateDetail/TemplateDetail.tsx
@@ -134,7 +134,6 @@ const TemplateDetail = (props: TemplateDetailProps, ref: Ref<FieldInstance>) =>
         {readonly ? (
           <EuiFlexItem grow={false} style={{ flexDirection: "row" }}>
             <EuiButton
-              fill
               style={{ marginRight: 20 }}
               onClick={() => {
                 const showValue: TemplateItemRemote = {

From df47c6b8bececc2fffba36db9e4fea582577c6c3 Mon Sep 17 00:00:00 2001
From: suzhou <suzhou@amazon.com>
Date: Tue, 10 Jan 2023 12:21:03 +0800
Subject: [PATCH 02/10] feat: update system index/alias wording

Signed-off-by: suzhou <suzhou@amazon.com>
---
 public/pages/Aliases/containers/CreateAlias/index.tsx         | 4 +++-
 .../Aliases/containers/DeleteAliasModal/DeleteAliasModal.tsx  | 4 +++-
 .../pages/CreateIndex/components/IndexDetail/IndexDetail.tsx  | 4 +++-
 .../Indices/components/CloseIndexModal/CloseIndexModal.tsx    | 2 +-
 .../__snapshots__/CloseIndexModal.test.tsx.snap               | 2 +-
 .../components/DeleteIndexModal/DeleteIndexModal.test.tsx     | 4 +++-
 .../Indices/components/DeleteIndexModal/DeleteIndexModal.tsx  | 4 +++-
 .../__snapshots__/DeleteIndexModal.test.tsx.snap              | 2 +-
 8 files changed, 18 insertions(+), 8 deletions(-)

diff --git a/public/pages/Aliases/containers/CreateAlias/index.tsx b/public/pages/Aliases/containers/CreateAlias/index.tsx
index 5003d8d12..3978a5467 100644
--- a/public/pages/Aliases/containers/CreateAlias/index.tsx
+++ b/public/pages/Aliases/containers/CreateAlias/index.tsx
@@ -123,7 +123,9 @@ export default function CreateAlias(props: ICreateAliasProps) {
       <EuiModalBody>
         {isEdit && filterByMinimatch(props.alias?.alias || "", SYSTEM_ALIAS) ? (
           <>
-            <EuiCallOut color="warning">You are editing a system-like alias, please be careful before you do any change to it.</EuiCallOut>
+            <EuiCallOut color="warning">
+              This alias may contain critical system data. Changing system aliases may break OpenSearch.
+            </EuiCallOut>
             <EuiSpacer />
           </>
         ) : null}
diff --git a/public/pages/Aliases/containers/DeleteAliasModal/DeleteAliasModal.tsx b/public/pages/Aliases/containers/DeleteAliasModal/DeleteAliasModal.tsx
index 5b3b320e0..727c1f3ee 100644
--- a/public/pages/Aliases/containers/DeleteAliasModal/DeleteAliasModal.tsx
+++ b/public/pages/Aliases/containers/DeleteAliasModal/DeleteAliasModal.tsx
@@ -74,7 +74,9 @@ export default function DeleteAliasModal(props: DeleteAliasModalProps) {
       <EuiModalBody>
         {hasSystemIndex ? (
           <>
-            <EuiCallOut color="warning">You are trying to delete system-like alias, please be careful.</EuiCallOut>
+            <EuiCallOut color="warning">
+              These aliases may contain critical system data. Deleting system aliases may break OpenSearch.
+            </EuiCallOut>
             <EuiSpacer />
           </>
         ) : null}
diff --git a/public/pages/CreateIndex/components/IndexDetail/IndexDetail.tsx b/public/pages/CreateIndex/components/IndexDetail/IndexDetail.tsx
index a55f1ba11..b0eb9b2fa 100644
--- a/public/pages/CreateIndex/components/IndexDetail/IndexDetail.tsx
+++ b/public/pages/CreateIndex/components/IndexDetail/IndexDetail.tsx
@@ -329,7 +329,9 @@ const IndexDetail = (
     <>
       {isEdit && !readonly && filterByMinimatch(value?.index as string, SYSTEM_INDEX) ? (
         <>
-          <EuiCallOut color="warning">You are editing a system-like index, please be careful before you do any change to it.</EuiCallOut>
+          <EuiCallOut color="warning">
+            This index may contain critical system data. Changing system indexes may break OpenSearch.
+          </EuiCallOut>
           <EuiSpacer />
         </>
       ) : null}
diff --git a/public/pages/Indices/components/CloseIndexModal/CloseIndexModal.tsx b/public/pages/Indices/components/CloseIndexModal/CloseIndexModal.tsx
index 6e3335482..8da8bee1a 100644
--- a/public/pages/Indices/components/CloseIndexModal/CloseIndexModal.tsx
+++ b/public/pages/Indices/components/CloseIndexModal/CloseIndexModal.tsx
@@ -50,7 +50,7 @@ export default function CloseIndexModal(props: CloseIndexModalProps) {
       <EuiModalBody>
         <>
           <EuiCallOut color="warning" hidden={!showWarning}>
-            You are closing system like index, please be careful before you do any change to it.
+            This index may contain critical system data. Closing system indexes may break OpenSearch.
           </EuiCallOut>
           <EuiSpacer />
         </>
diff --git a/public/pages/Indices/components/CloseIndexModal/__snapshots__/CloseIndexModal.test.tsx.snap b/public/pages/Indices/components/CloseIndexModal/__snapshots__/CloseIndexModal.test.tsx.snap
index 77d591b0d..04fde70d1 100644
--- a/public/pages/Indices/components/CloseIndexModal/__snapshots__/CloseIndexModal.test.tsx.snap
+++ b/public/pages/Indices/components/CloseIndexModal/__snapshots__/CloseIndexModal.test.tsx.snap
@@ -58,7 +58,7 @@ HTMLCollection [
                   <div
                     class="euiTextColor euiTextColor--default"
                   >
-                    You are closing system like index, please be careful before you do any change to it.
+                    This index may contain critical system data. Closing system indexes may break OpenSearch.
                   </div>
                 </div>
               </div>
diff --git a/public/pages/Indices/components/DeleteIndexModal/DeleteIndexModal.test.tsx b/public/pages/Indices/components/DeleteIndexModal/DeleteIndexModal.test.tsx
index 1011b9a2d..96d587f4b 100644
--- a/public/pages/Indices/components/DeleteIndexModal/DeleteIndexModal.test.tsx
+++ b/public/pages/Indices/components/DeleteIndexModal/DeleteIndexModal.test.tsx
@@ -13,7 +13,9 @@ describe("<DeleteIndexModal /> spec", () => {
   it("renders the component", async () => {
     const { getByText } = render(<DeleteIndexModal selectedItems={[".kibana", "test"]} visible onConfirm={() => {}} onClose={() => {}} />);
 
-    await waitFor(() => expect(getByText("You are trying to delete system-like index, please be careful.")).toBeInTheDocument());
+    await waitFor(() =>
+      expect(getByText("These indexes may contain critical system data. Deleting system indexes may break OpenSearch.")).toBeInTheDocument()
+    );
     expect(document.body.children).toMatchSnapshot();
   });
 
diff --git a/public/pages/Indices/components/DeleteIndexModal/DeleteIndexModal.tsx b/public/pages/Indices/components/DeleteIndexModal/DeleteIndexModal.tsx
index 490f06209..cfa742a97 100644
--- a/public/pages/Indices/components/DeleteIndexModal/DeleteIndexModal.tsx
+++ b/public/pages/Indices/components/DeleteIndexModal/DeleteIndexModal.tsx
@@ -50,7 +50,9 @@ export default function DeleteIndexModal(props: DeleteIndexModalProps) {
       <EuiModalBody>
         {hasSystemIndex ? (
           <>
-            <EuiCallOut color="warning">You are trying to delete system-like index, please be careful.</EuiCallOut>
+            <EuiCallOut color="warning">
+              These indexes may contain critical system data. Deleting system indexes may break OpenSearch.
+            </EuiCallOut>
             <EuiSpacer />
           </>
         ) : null}
diff --git a/public/pages/Indices/components/DeleteIndexModal/__snapshots__/DeleteIndexModal.test.tsx.snap b/public/pages/Indices/components/DeleteIndexModal/__snapshots__/DeleteIndexModal.test.tsx.snap
index 6207dd6e6..19062a096 100644
--- a/public/pages/Indices/components/DeleteIndexModal/__snapshots__/DeleteIndexModal.test.tsx.snap
+++ b/public/pages/Indices/components/DeleteIndexModal/__snapshots__/DeleteIndexModal.test.tsx.snap
@@ -57,7 +57,7 @@ HTMLCollection [
                   <div
                     class="euiTextColor euiTextColor--default"
                   >
-                    You are trying to delete system-like index, please be careful.
+                    These indexes may contain critical system data. Deleting system indexes may break OpenSearch.
                   </div>
                 </div>
               </div>

From 90dc81b5ae513229ecdd04892ce2ab401d0665cb Mon Sep 17 00:00:00 2001
From: suzhou <suzhou@amazon.com>
Date: Tue, 10 Jan 2023 18:13:29 +0800
Subject: [PATCH 03/10] feat: update

Signed-off-by: suzhou <suzhou@amazon.com>
---
 public/lib/field/rules/index.ts                        |  4 ++++
 .../CreateIndex/components/IndexDetail/IndexDetail.tsx | 10 ++++++++--
 .../components/IndexSettings/IndexSettings.tsx         |  6 ++++++
 3 files changed, 18 insertions(+), 2 deletions(-)

diff --git a/public/lib/field/rules/index.ts b/public/lib/field/rules/index.ts
index 9ca515d6d..d26b0d365 100644
--- a/public/lib/field/rules/index.ts
+++ b/public/lib/field/rules/index.ts
@@ -45,6 +45,10 @@ const rules = {
     const isNum = typeof value === "number";
     const isStr = typeof value === "string";
 
+    if (value === "") {
+      return "";
+    }
+
     if (isNum) {
       key = "number";
     } else if (isStr) {
diff --git a/public/pages/CreateIndex/components/IndexDetail/IndexDetail.tsx b/public/pages/CreateIndex/components/IndexDetail/IndexDetail.tsx
index b0eb9b2fa..5b2fd18a4 100644
--- a/public/pages/CreateIndex/components/IndexDetail/IndexDetail.tsx
+++ b/public/pages/CreateIndex/components/IndexDetail/IndexDetail.tsx
@@ -261,6 +261,9 @@ const IndexDetail = (
             },
             {
               validator(rule, value, values) {
+                if (!value) {
+                  return Promise.reject("Number of primary shards is required.");
+                }
                 if (Number(value) !== parseInt(value)) {
                   return Promise.reject("Number of primary shards must be an integer.");
                 }
@@ -270,7 +273,7 @@ const IndexDetail = (
             },
           ],
           props: {
-            placeholder: "The number of primary shards in the index. Default is 1.",
+            placeholder: "Specify primary shard count.",
           },
         },
       },
@@ -290,6 +293,9 @@ const IndexDetail = (
             },
             {
               validator(rule, value, values) {
+                if (!value) {
+                  return Promise.reject("Number of replicas is required.");
+                }
                 if (Number(value) !== parseInt(value)) {
                   return Promise.reject("Number of replicas must be an integer.");
                 }
@@ -299,7 +305,7 @@ const IndexDetail = (
             },
           ],
           props: {
-            placeholder: "The number of replica shards each primary shard should have.",
+            placeholder: "Specify number of replicas.",
           },
         },
       },
diff --git a/public/pages/CreateIndexTemplate/components/IndexSettings/IndexSettings.tsx b/public/pages/CreateIndexTemplate/components/IndexSettings/IndexSettings.tsx
index 8d08b50f6..058fb6888 100644
--- a/public/pages/CreateIndexTemplate/components/IndexSettings/IndexSettings.tsx
+++ b/public/pages/CreateIndexTemplate/components/IndexSettings/IndexSettings.tsx
@@ -50,6 +50,9 @@ export default function IndexSettings(props: SubDetailProps) {
                   },
                   {
                     validator(rule, value) {
+                      if (!value) {
+                        return Promise.reject("Number of primary shards is required.");
+                      }
                       if (Number(value) !== parseInt(value)) {
                         return Promise.reject("Number of primary shards must be an integer.");
                       }
@@ -78,6 +81,9 @@ export default function IndexSettings(props: SubDetailProps) {
                   },
                   {
                     validator(rule, value) {
+                      if (!value) {
+                        return Promise.reject("Number of replicas is required.");
+                      }
                       if (Number(value) !== parseInt(value)) {
                         return Promise.reject("Number of replicas must be an integer");
                       }

From c1c4fa4c8ec8319a19ea0de8c17e127b5076e3a0 Mon Sep 17 00:00:00 2001
From: suzhou <suzhou@amazon.com>
Date: Tue, 10 Jan 2023 18:15:03 +0800
Subject: [PATCH 04/10] feat: update

Signed-off-by: suzhou <suzhou@amazon.com>
---
 .../pages/CreateIndex/components/IndexDetail/IndexDetail.tsx  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/public/pages/CreateIndex/components/IndexDetail/IndexDetail.tsx b/public/pages/CreateIndex/components/IndexDetail/IndexDetail.tsx
index 5b2fd18a4..f718fdcee 100644
--- a/public/pages/CreateIndex/components/IndexDetail/IndexDetail.tsx
+++ b/public/pages/CreateIndex/components/IndexDetail/IndexDetail.tsx
@@ -483,13 +483,13 @@ const IndexDetail = (
                     helpText: (
                       <>
                         <p>
-                          Specify a comma-delimited list of settings.
+                          Specify a comma-delimited list of settings.{" "}
                           <EuiLink external href={INDEX_SETTINGS_URL} target="_blank">
                             View index settings.
                           </EuiLink>
                         </p>
                         <p>
-                          All the settings will be handled in flat structure.
+                          All the settings will be handled in flat structure.{" "}
                           <EuiLink
                             href={`https://opensearch.org/docs/${docVersion}/api-reference/index-apis/get-index/#url-parameters`}
                             external

From dbb2a57baadc225c761b72ea386a743486818918 Mon Sep 17 00:00:00 2001
From: suzhou <suzhou@amazon.com>
Date: Tue, 10 Jan 2023 18:17:27 +0800
Subject: [PATCH 05/10] feat: add judgement

Signed-off-by: suzhou <suzhou@amazon.com>
---
 .../Aliases/containers/CreateAlias/index.tsx  | 20 ++++++++++++-------
 1 file changed, 13 insertions(+), 7 deletions(-)

diff --git a/public/pages/Aliases/containers/CreateAlias/index.tsx b/public/pages/Aliases/containers/CreateAlias/index.tsx
index 3978a5467..4cbb815ec 100644
--- a/public/pages/Aliases/containers/CreateAlias/index.tsx
+++ b/public/pages/Aliases/containers/CreateAlias/index.tsx
@@ -204,14 +204,20 @@ export default function CreateAlias(props: ICreateAliasProps) {
                   alias: values.alias,
                   index,
                 }));
-                result = await services?.commonService.apiCaller({
-                  endpoint: "indices.updateAliases",
-                  data: {
-                    body: {
-                      actions,
+                if (actions.length === 0) {
+                  result = {
+                    ok: true,
+                  };
+                } else {
+                  result = await services?.commonService.apiCaller({
+                    endpoint: "indices.updateAliases",
+                    data: {
+                      body: {
+                        actions,
+                      },
                     },
-                  },
-                });
+                  });
+                }
               } else {
                 result = await services?.commonService.apiCaller({
                   endpoint: "indices.putAlias",

From cfa4ccf2b6a59cd0e1ac19cb97bbef8981a6004d Mon Sep 17 00:00:00 2001
From: suzhou <suzhou@amazon.com>
Date: Tue, 10 Jan 2023 18:19:26 +0800
Subject: [PATCH 06/10] feat: fix template stuff

Signed-off-by: suzhou <suzhou@amazon.com>
---
 .../containers/TemplateDetail/TemplateDetail.tsx                 | 1 +
 1 file changed, 1 insertion(+)

diff --git a/public/pages/CreateIndexTemplate/containers/TemplateDetail/TemplateDetail.tsx b/public/pages/CreateIndexTemplate/containers/TemplateDetail/TemplateDetail.tsx
index 74d697b47..3940ce490 100644
--- a/public/pages/CreateIndexTemplate/containers/TemplateDetail/TemplateDetail.tsx
+++ b/public/pages/CreateIndexTemplate/containers/TemplateDetail/TemplateDetail.tsx
@@ -94,6 +94,7 @@ const TemplateDetail = (props: TemplateDetailProps, ref: Ref<FieldInstance>) =>
         })
         .catch(() => {
           // do nothing
+          props.history.replace(ROUTES.TEMPLATES);
         });
     }
     return () => {

From 5ee9e380bdad7baa79bf8fde846cd066158c7251 Mon Sep 17 00:00:00 2001
From: suzhou <suzhou@amazon.com>
Date: Tue, 10 Jan 2023 18:22:47 +0800
Subject: [PATCH 07/10] feat: update snapshot

Signed-off-by: suzhou <suzhou@amazon.com>
---
 .../__snapshots__/CreateIndexFlyout.test.tsx.snap           | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/public/pages/Reindex/components/CreateIndexFlyout/__snapshots__/CreateIndexFlyout.test.tsx.snap b/public/pages/Reindex/components/CreateIndexFlyout/__snapshots__/CreateIndexFlyout.test.tsx.snap
index 16b7d4cd9..514b4daeb 100644
--- a/public/pages/Reindex/components/CreateIndexFlyout/__snapshots__/CreateIndexFlyout.test.tsx.snap
+++ b/public/pages/Reindex/components/CreateIndexFlyout/__snapshots__/CreateIndexFlyout.test.tsx.snap
@@ -291,7 +291,7 @@ Object {
                               >
                                 <input
                                   class="euiFieldNumber"
-                                  placeholder="The number of primary shards in the index. Default is 1."
+                                  placeholder="Specify primary shard count."
                                   type="number"
                                   value="1"
                                 />
@@ -336,7 +336,7 @@ Object {
                               >
                                 <input
                                   class="euiFieldNumber"
-                                  placeholder="The number of replica shards each primary shard should have."
+                                  placeholder="Specify number of replicas."
                                   type="number"
                                   value="1"
                                 />
@@ -459,6 +459,7 @@ Object {
                                   >
                                     <p>
                                       Specify a comma-delimited list of settings.
+                                       
                                       <a
                                         class="euiLink euiLink--primary"
                                         href="https://opensearch.org/docs/latest/api-reference/index-apis/create-index#index-settings"
@@ -476,6 +477,7 @@ Object {
                                     </p>
                                     <p>
                                       All the settings will be handled in flat structure.
+                                       
                                       <a
                                         class="euiLink euiLink--primary"
                                         href="https://opensearch.org/docs/undefined/api-reference/index-apis/get-index/#url-parameters"

From ca83655c02a6bfe402e6032537b9b82ddd674c98 Mon Sep 17 00:00:00 2001
From: suzhou <suzhou@amazon.com>
Date: Wed, 11 Jan 2023 08:39:02 +0800
Subject: [PATCH 08/10] feat: update placeholder

Signed-off-by: suzhou <suzhou@amazon.com>
---
 cypress/integration/create_index.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cypress/integration/create_index.js b/cypress/integration/create_index.js
index f1a78f55d..1b6709568 100644
--- a/cypress/integration/create_index.js
+++ b/cypress/integration/create_index.js
@@ -181,7 +181,7 @@ describe("Create Index", () => {
         .type('{ "index.blocks.write": true, "index.number_of_shards": "3" }', { parseSpecialCharSequences: false, force: true })
         .end()
         .wait(1000)
-        .get('[placeholder="The number of replica shards each primary shard should have."]')
+        .get('[placeholder="Specify number of replicas."]')
         .clear()
         .type(2)
         .end();

From 6d9a07d46588cba31ed1f2a8ed2969dfa40a3d21 Mon Sep 17 00:00:00 2001
From: suzhou <suzhou@amazon.com>
Date: Wed, 11 Jan 2023 09:40:13 +0800
Subject: [PATCH 09/10] feat: use 2.5 version

Signed-off-by: suzhou <suzhou@amazon.com>
---
 .github/workflows/cypress-workflow.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/cypress-workflow.yml b/.github/workflows/cypress-workflow.yml
index 8acefb2cc..d73a908ca 100644
--- a/.github/workflows/cypress-workflow.yml
+++ b/.github/workflows/cypress-workflow.yml
@@ -7,7 +7,7 @@ on:
     branches:
       - "*"
 env:
-  OPENSEARCH_DASHBOARDS_VERSION: '2.x'
+  OPENSEARCH_DASHBOARDS_VERSION: '2.5'
   OPENSEARCH_VERSION: '2.5.0-SNAPSHOT'
 jobs:
   tests:

From 7e6fe14d65e64529ab3a3ee2ede0b7f1dd233679 Mon Sep 17 00:00:00 2001
From: suzhou <suzhou@amazon.com>
Date: Wed, 11 Jan 2023 10:12:45 +0800
Subject: [PATCH 10/10] feat: update

Signed-off-by: suzhou <suzhou@amazon.com>
---
 .github/workflows/cypress-workflow.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/cypress-workflow.yml b/.github/workflows/cypress-workflow.yml
index d73a908ca..8acefb2cc 100644
--- a/.github/workflows/cypress-workflow.yml
+++ b/.github/workflows/cypress-workflow.yml
@@ -7,7 +7,7 @@ on:
     branches:
       - "*"
 env:
-  OPENSEARCH_DASHBOARDS_VERSION: '2.5'
+  OPENSEARCH_DASHBOARDS_VERSION: '2.x'
   OPENSEARCH_VERSION: '2.5.0-SNAPSHOT'
 jobs:
   tests: