feat(storage) add volume detail page and adjust rename header

canonical · Oct 4, 2023 · 8940efa · 8940efa
1 parent 6c111c3
commit 8940efa
Show file tree

Hide file tree

Showing 32 changed files with 724 additions and 79 deletions.
diff --git a/src/App.tsx b/src/App.tsx
@@ -31,6 +31,9 @@ const CertificateGenerate = lazy(
 const Login = lazy(() => import("pages/login/Login"));
 const ProtectedRoute = lazy(() => import("components/ProtectedRoute"));
 const StorageDetail = lazy(() => import("pages/storage/StorageDetail"));
+const StorageVolumeDetail = lazy(
+  () => import("pages/storage/StorageVolumeDetail")
+);
 const NetworkMap = lazy(() => import("pages/networks/NetworkMap"));
 const CreateInstanceForm = lazy(
   () => import("pages/instances/CreateInstanceForm")
@@ -252,6 +255,14 @@ const App: FC = () => {
           path="/ui/project/:project/storage/detail/:name/:activeTab"
           element={<ProtectedRoute outlet={<StorageDetail />} />}
         />
+        <Route
+          path="/ui/project/:project/storage/detail/:pool/:type/:volume"
+          element={<ProtectedRoute outlet={<StorageVolumeDetail />} />}
+        />
+        <Route
+          path="/ui/project/:project/storage/detail/:pool/:type/:volume/:activeTab"
+          element={<ProtectedRoute outlet={<StorageVolumeDetail />} />}
+        />
         <Route
           path="/ui/cluster"
           element={<ProtectedRoute outlet={<ClusterList />} />}

diff --git a/src/api/storage-pools.tsx b/src/api/storage-pools.tsx
@@ -1,4 +1,4 @@
-import { handleResponse } from "util/helpers";
+import { handleEtagResponse, handleResponse } from "util/helpers";
 import {
   LxdStoragePool,
   LxdStoragePoolResources,
@@ -101,6 +101,44 @@ export const fetchStorageVolumes = (
   });
 };
 
+export const fetchStorageVolume = (
+  pool: string,
+  project: string,
+  type: string,
+  volume: string
+): Promise<LxdStorageVolume> => {
+  return new Promise((resolve, reject) => {
+    fetch(
+      `/1.0/storage-pools/${pool}/volumes/${type}/${volume}?project=${project}&recursion=1`
+    )
+      .then(handleEtagResponse)
+      .then((data) => resolve(data as LxdStorageVolume))
+      .catch(reject);
+  });
+};
+
+export const renameStorageVolume = (
+  pool: string,
+  project: string,
+  volume: LxdStorageVolume,
+  newName: string
+) => {
+  return new Promise((resolve, reject) => {
+    fetch(
+      `/1.0/storage-pools/${pool}/volumes/${volume.type}/${volume.name}?project=${project}`,
+      {
+        method: "POST",
+        body: JSON.stringify({
+          name: newName,
+        }),
+      }
+    )
+      .then(handleResponse)
+      .then(resolve)
+      .catch(reject);
+  });
+};
+
 export const createIsoStorageVolume = (
   pool: string,
   isoFile: File,
@@ -154,6 +192,30 @@ export const createStorageVolume = (
   });
 };
 
+export const updateStorageVolume = (
+  pool: string,
+  project: string,
+  volume: Partial<LxdStorageVolume>
+) => {
+  return new Promise((resolve, reject) => {
+    fetch(
+      `/1.0/storage-pools/${pool}/volumes/${volume.type ?? ""}/${
+        volume.name ?? ""
+      }?project=${project}`,
+      {
+        method: "PUT",
+        body: JSON.stringify(volume),
+        headers: {
+          "If-Match": volume.etag ?? "invalid-etag",
+        },
+      }
+    )
+      .then(handleResponse)
+      .then((data) => resolve(data))
+      .catch(reject);
+  });
+};
+
 export const deleteStorageVolume = (
   volume: string,
   pool: string,

diff --git a/src/components/RenameHeader.tsx b/src/components/RenameHeader.tsx
@@ -12,7 +12,7 @@ export interface RenameHeaderValues {
 interface Props {
   name: string;
   titleClassName?: string;
-  parentItem: ReactNode;
+  parentItems: ReactNode[];
   centerControls?: ReactNode;
   controls?: ReactNode;
   isLoaded: boolean;
@@ -23,7 +23,7 @@ interface Props {
 const RenameHeader: FC<Props> = ({
   name,
   titleClassName,
-  parentItem,
+  parentItems,
   centerControls,
   controls,
   isLoaded,
@@ -52,7 +52,11 @@ const RenameHeader: FC<Props> = ({
             aria-label="Breadcrumbs"
           >
             <ol className="p-breadcrumbs__items">
-              <li className="p-breadcrumbs__item">{parentItem}</li>
+              {parentItems.map((item, key) => (
+                <li className="p-breadcrumbs__item" key={key}>
+                  {item}
+                </li>
+              ))}
               {formik.values.isRenaming ? (
                 <li className="p-breadcrumbs__item rename">
                   <Input

diff --git a/src/pages/instances/InstanceDetailHeader.tsx b/src/pages/instances/InstanceDetailHeader.tsx
@@ -70,9 +70,11 @@ const InstanceDetailHeader: FC<Props> = ({ name, instance, project }) => {
     <RenameHeader
       name={name}
       titleClassName="instance-detail-title"
-      parentItem={
-        <Link to={`/ui/project/${project}/instances`}>Instances</Link>
-      }
+      parentItems={[
+        <Link to={`/ui/project/${project}/instances`} key={1}>
+          Instances
+        </Link>,
+      ]}
       renameDisabledReason={
         instance?.status !== "Stopped"
           ? "Stop the instance to rename"

diff --git a/src/pages/networks/NetworkDetailHeader.tsx b/src/pages/networks/NetworkDetailHeader.tsx
@@ -65,7 +65,11 @@ const NetworkDetailHeader: FC<Props> = ({ name, network, project }) => {
   return (
     <RenameHeader
       name={name}
-      parentItem={<Link to={`/ui/project/${project}/networks`}>Networks</Link>}
+      parentItems={[
+        <Link to={`/ui/project/${project}/networks`} key={1}>
+          Networks
+        </Link>,
+      ]}
       renameDisabledReason={
         !isManaged
           ? "Can not rename, network is not managed"

diff --git a/src/pages/profiles/ProfileDetailHeader.tsx b/src/pages/profiles/ProfileDetailHeader.tsx
@@ -68,7 +68,11 @@ const ProfileDetailHeader: FC<Props> = ({
   return (
     <RenameHeader
       name={name}
-      parentItem={<Link to={`/ui/project/${project}/profiles`}>Profiles</Link>}
+      parentItems={[
+        <Link to={`/ui/project/${project}/profiles`} key={1}>
+          Profiles
+        </Link>,
+      ]}
       renameDisabledReason={
         profile && profile.name === "default"
           ? "Cannot rename the default profile"

diff --git a/src/pages/projects/ProjectConfigurationHeader.tsx b/src/pages/projects/ProjectConfigurationHeader.tsx
@@ -60,7 +60,7 @@ const ProjectConfigurationHeader: FC<Props> = ({ project }) => {
   return (
     <RenameHeader
       name={project.name}
-      parentItem="Project configuration"
+      parentItems={["Project configuration"]}
       renameDisabledReason={
         project.name === "default"
           ? "Cannot rename the default project"

diff --git a/src/pages/projects/forms/DiskSizeSelector.tsx b/src/pages/projects/forms/DiskSizeSelector.tsx
@@ -6,9 +6,10 @@ import { parseMemoryLimit } from "util/limits";
 interface Props {
   value?: string;
   setMemoryLimit: (val?: string) => void;
+  disabled?: boolean;
 }
 
-const DiskSizeSelector: FC<Props> = ({ value, setMemoryLimit }) => {
+const DiskSizeSelector: FC<Props> = ({ value, setMemoryLimit, disabled }) => {
   const limit = parseMemoryLimit(value) ?? {
     value: 1,
     unit: BYTES_UNITS.GIB,
@@ -32,7 +33,8 @@ const DiskSizeSelector: FC<Props> = ({ value, setMemoryLimit }) => {
           step="Any"
           placeholder="Enter value"
           onChange={(e) => setMemoryLimit(e.target.value + limit.unit)}
-          value={limit.value}
+          value={value?.match(/^\d/) ? limit.value : ""}
+          disabled={disabled}
         />
         <Select
           id="memUnitSelect"
@@ -42,6 +44,7 @@ const DiskSizeSelector: FC<Props> = ({ value, setMemoryLimit }) => {
             setMemoryLimit(`${limit.value ?? 0}${e.target.value}`)
           }
           value={limit.unit}
+          disabled={disabled}
         />
       </div>
     </div>

diff --git a/src/pages/storage/CustomIsoList.tsx b/src/pages/storage/CustomIsoList.tsx
@@ -5,6 +5,7 @@ import {
   List,
   MainTable,
   SearchBox,
+  useNotify,
 } from "@canonical/react-components";
 import { humanFileSize, isoTimeToString } from "util/helpers";
 import { useQuery } from "@tanstack/react-query";
@@ -20,6 +21,7 @@ interface Props {
 }
 
 const CustomIsoList: FC<Props> = ({ project }) => {
+  const notify = useNotify();
   const [query, setQuery] = useState<string>("");
 
   const { data: images = [], isLoading } = useQuery({
@@ -57,6 +59,9 @@ const CustomIsoList: FC<Props> = ({ project }) => {
               pool={image.pool ?? ""}
               volume={image.volume}
               project={project}
+              onFinish={() =>
+                notify.success(`Custom iso ${image.aliases} deleted.`)
+              }
             />,
           ]}
         />

diff --git a/src/pages/storage/StorageDetailHeader.tsx b/src/pages/storage/StorageDetailHeader.tsx
@@ -6,7 +6,7 @@ import * as Yup from "yup";
 import { LxdStoragePool } from "types/storage";
 import { renameStoragePool } from "api/storage-pools";
 import DeleteStorageBtn from "pages/storage/actions/DeleteStorageBtn";
-import { testDuplicateName } from "util/storagePool";
+import { testDuplicateStoragePoolName } from "util/storagePool";
 import { useNotify } from "@canonical/react-components";
 
 interface Props {
@@ -22,7 +22,7 @@ const StorageDetailHeader: FC<Props> = ({ name, storagePool, project }) => {
 
   const RenameSchema = Yup.object().shape({
     name: Yup.string()
-      .test(...testDuplicateName(project, controllerState))
+      .test(...testDuplicateStoragePoolName(project, controllerState))
       .required("This field is required"),
   });
 
@@ -56,9 +56,11 @@ const StorageDetailHeader: FC<Props> = ({ name, storagePool, project }) => {
   return (
     <RenameHeader
       name={name}
-      parentItem={
-        <Link to={`/ui/project/${project}/storage`}>Storage pools</Link>
-      }
+      parentItems={[
+        <Link to={`/ui/project/${project}/storage`} key={1}>
+          Storage pools
+        </Link>,
+      ]}
       controls={
         <DeleteStorageBtn
           key="delete"

diff --git a/src/pages/storage/StoragePoolForm.tsx b/src/pages/storage/StoragePoolForm.tsx
@@ -26,7 +26,7 @@ import {
   zfsDriver,
 } from "util/storageOptions";
 import ItemName from "components/ItemName";
-import { testDuplicateName } from "util/storagePool";
+import { testDuplicateStoragePoolName } from "util/storagePool";
 import NotificationRow from "components/NotificationRow";
 
 const StoragePoolForm: FC = () => {
@@ -37,7 +37,9 @@ const StoragePoolForm: FC = () => {
 
   const StorageSchema = Yup.object().shape({
     name: Yup.string()
-      .test(...testDuplicateName(panelParams.project, controllerState))
+      .test(
+        ...testDuplicateStoragePoolName(panelParams.project, controllerState)
+      )
       .required("This field is required"),
   });
 

diff --git a/src/pages/storage/StorageUsedBy.tsx b/src/pages/storage/StorageUsedBy.tsx
@@ -1,13 +1,13 @@
 import React, { FC } from "react";
 import { Link } from "react-router-dom";
 import ImageName from "pages/images/ImageName";
-import { LxdStoragePool } from "types/storage";
+import { LxdStoragePool, LxdStorageVolume } from "types/storage";
 import { filterUsedByType, LxdUsedBy } from "util/usedBy";
 import InstanceLink from "pages/instances/InstanceLink";
 import ExpandableList from "components/ExpandableList";
 
 interface Props {
-  storage: LxdStoragePool;
+  storage: LxdStoragePool | LxdStorageVolume;
   project: string;
 }
 
@@ -36,7 +36,7 @@ const StorageUsedBy: FC<Props> = ({ storage, project }) => {
           <td>
             <ExpandableList
               items={data[INSTANCES].map((item) => (
-                <div key={item.name}>
+                <div key={`${item.name}-${item.project}`}>
                   <InstanceLink instance={item} />
                   {item.project !== project && ` (project ${item.project})`}
                 </div>
@@ -51,7 +51,7 @@ const StorageUsedBy: FC<Props> = ({ storage, project }) => {
           <td>
             <ExpandableList
               items={data[PROFILES].map((item) => (
-                <div key={item.name}>
+                <div key={`${item.name}-${item.project}`}>
                   <Link
                     to={`/ui/project/${item.project}/profiles/detail/${item.name}`}
                   >
@@ -69,7 +69,7 @@ const StorageUsedBy: FC<Props> = ({ storage, project }) => {
             <ExpandableList
               items={data[IMAGES].map((item) => (
                 <ImageName
-                  key={item.name}
+                  key={`${item.name}-${item.project}`}
                   id={item.name}
                   project={item.project}
                 />
@@ -84,7 +84,7 @@ const StorageUsedBy: FC<Props> = ({ storage, project }) => {
           <td>
             <ExpandableList
               items={data[SNAPSHOTS].map((item) => (
-                <div key={item.name}>
+                <div key={`${item.instance}-${item.name}-${item.project}`}>
                   <Link
                     to={`/ui/project/${item.project}/instances/detail/${item.instance}/snapshots`}
                   >
@@ -101,7 +101,7 @@ const StorageUsedBy: FC<Props> = ({ storage, project }) => {
           <td>
             <ExpandableList
               items={data[CUSTOM].map((item) => (
-                <div key={item.name}>
+                <div key={`${item.name}-${item.project}`}>
                   {item.name}
                   {item.project !== project && ` (project ${item.project})`}
                 </div>