From 84d5de3073289686200dfb5acbf177dccff8377a Mon Sep 17 00:00:00 2001
From: Jincheng Wan <45655760+Kapian1234@users.noreply.github.com>
Date: Thu, 5 Sep 2024 11:20:34 +0800
Subject: [PATCH] [Workspace] Hide delete button for non OSD admin (#7987)

* Hide workspace delete button for non OSD admin

Signed-off-by: Kapian1234 <wanjinch@amazon.com>

* hide delete action instead of disabled

Signed-off-by: Kapian1234 <wanjinch@amazon.com>

* Changeset file for PR #7987 created/updated

---------

Signed-off-by: Kapian1234 <wanjinch@amazon.com>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
---
 changelogs/fragments/7987.yml                 |  2 ++
 .../workspace_detail/workspace_detail.tsx     | 35 ++++++++++---------
 .../components/workspace_list/index.tsx       |  1 +
 3 files changed, 22 insertions(+), 16 deletions(-)
 create mode 100644 changelogs/fragments/7987.yml

diff --git a/changelogs/fragments/7987.yml b/changelogs/fragments/7987.yml
new file mode 100644
index 000000000000..0b67c17de12d
--- /dev/null
+++ b/changelogs/fragments/7987.yml
@@ -0,0 +1,2 @@
+fix:
+- Hide delete button for non OSD admin ([#7987](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/7987))
\ No newline at end of file
diff --git a/src/plugins/workspace/public/components/workspace_detail/workspace_detail.tsx b/src/plugins/workspace/public/components/workspace_detail/workspace_detail.tsx
index 3543795f7e44..3d4b77c20a22 100644
--- a/src/plugins/workspace/public/components/workspace_detail/workspace_detail.tsx
+++ b/src/plugins/workspace/public/components/workspace_detail/workspace_detail.tsx
@@ -170,22 +170,25 @@ export const WorkspaceDetail = (props: WorkspaceDetailProps) => {
             setMountPoint={application.setAppDescriptionControls}
           />
         )}
-        <HeaderControl
-          controls={[
-            {
-              run: () => setDeletedWorkspace(currentWorkspace),
-              color: 'danger',
-              iconType: 'trash',
-              ariaLabel: i18n.translate('workspace.detail.delete.button', {
-                defaultMessage: 'Delete',
-              }),
-              testId: 'workspace-detail-delete-button',
-              controlType: 'icon',
-              display: 'base',
-            } as TopNavControlIconData,
-          ]}
-          setMountPoint={application.setAppRightControls}
-        />
+        {isDashboardAdmin && (
+          <HeaderControl
+            controls={[
+              {
+                run: () => setDeletedWorkspace(currentWorkspace),
+                color: 'danger',
+                iconType: 'trash',
+                ariaLabel: i18n.translate('workspace.detail.delete.button', {
+                  defaultMessage: 'Delete',
+                }),
+                testId: 'workspace-detail-delete-button',
+                controlType: 'icon',
+                display: 'base',
+              } as TopNavControlIconData,
+            ]}
+            setMountPoint={application.setAppRightControls}
+          />
+        )}
+
         <EuiPageContent>
           <WorkspaceDetailPanel
             useCaseUrl={useCaseUrl}
diff --git a/src/plugins/workspace/public/components/workspace_list/index.tsx b/src/plugins/workspace/public/components/workspace_list/index.tsx
index 23235345f75e..176f0dfdcc39 100644
--- a/src/plugins/workspace/public/components/workspace_list/index.tsx
+++ b/src/plugins/workspace/public/components/workspace_list/index.tsx
@@ -409,6 +409,7 @@ export const WorkspaceList = ({ registeredUseCases$ }: WorkspaceListProps) => {
           type: 'button',
           description: 'Delete workspace',
           'data-test-subj': 'workspace-list-delete-icon',
+          available: () => isDashboardAdmin,
           render: (item: WorkspaceAttribute) => {
             return (
               <EuiButtonEmpty