Delete connection ref from notebooks when connection is deleted

opendatahub-io · Nov 4, 2024 · 66922ff · 66922ff
1 parent 217787a
commit 66922ff
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 10 deletions.
diff --git a/frontend/src/pages/projects/screens/detail/connections/ConnectedResources.tsx b/frontend/src/pages/projects/screens/detail/connections/ConnectedResources.tsx
@@ -25,7 +25,7 @@ const ConnectedResources: React.FC<Props> = ({ connection }) => {
     return <Spinner size="sm" />;
   }
 
-  if (!connectedNotebooks.length) {
+  if (!connectedNotebooks.length && !connectedModels.length) {
     return '-';
   }
 

diff --git a/frontend/src/pages/projects/screens/detail/connections/ConnectionsDeleteModal.tsx b/frontend/src/pages/projects/screens/detail/connections/ConnectionsDeleteModal.tsx
@@ -1,5 +1,4 @@
 import React from 'react';
-import { K8sStatus } from '@openshift/dynamic-plugin-sdk-utils';
 import {
   Badge,
   Bullseye,
@@ -20,19 +19,18 @@ import {
 import { useNotebooksStates } from '~/pages/projects/notebook/useNotebooksStates';
 import { NotebookKind } from '~/k8sTypes';
 import { useInferenceServicesForConnection } from '~/pages/projects/useInferenceServicesForConnection';
+import { deleteSecret, removeNotebookSecret } from '~/api';
 
 type Props = {
   namespace: string;
   deleteConnection: Connection;
   onClose: (deleted?: boolean) => void;
-  onDelete: () => Promise<K8sStatus>;
 };
 
 export const ConnectionsDeleteModal: React.FC<Props> = ({
   namespace,
   deleteConnection,
   onClose,
-  onDelete,
 }) => {
   const [isDeleting, setIsDeleting] = React.useState(false);
   const [error, setError] = React.useState<Error>();
@@ -61,8 +59,18 @@ export const ConnectionsDeleteModal: React.FC<Props> = ({
       onDelete={() => {
         setIsDeleting(true);
         setError(undefined);
-
-        onDelete()
+        Promise.all(
+          connectedNotebooks.map((notebook) =>
+            removeNotebookSecret(
+              notebook.metadata.name,
+              notebook.metadata.namespace,
+              deleteConnection.metadata.name,
+            ),
+          ),
+        )
+          .then(() =>
+            deleteSecret(deleteConnection.metadata.namespace, deleteConnection.metadata.name),
+          )
           .then(() => {
             onClose(true);
           })

diff --git a/frontend/src/pages/projects/screens/detail/connections/ConnectionsTable.tsx b/frontend/src/pages/projects/screens/detail/connections/ConnectionsTable.tsx
@@ -1,6 +1,5 @@
 import * as React from 'react';
 import { Connection, ConnectionTypeConfigMapObj } from '~/concepts/connectionTypes/types';
-import { deleteSecret } from '~/api';
 import { Table } from '~/components/table';
 import ConnectionsTableRow from './ConnectionsTableRow';
 import { getColumns } from './connectionsTableColumns';
@@ -65,9 +64,6 @@ const ConnectionsTable: React.FC<ConnectionsTableProps> = ({
               refreshConnections();
             }
           }}
-          onDelete={() =>
-            deleteSecret(deleteConnection.metadata.namespace, deleteConnection.metadata.name)
-          }
         />
       )}
     </>