Operator UI Adjustments (#1805)

Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>

portal-ui/src/screens/Console/Common/FormComponents/FormSwitchWrapper/FormSwitchWrapper.tsx

@@ -44,9 +44,6 @@ interface IFormSwitch {
 
 const styles = (theme: Theme) =>
   createStyles({
-    divContainer: {
-      marginBottom: 20,
-    },
     indicatorLabelOn: {
       fontWeight: "bold",
       color: "#081C42 !important",
@@ -172,7 +169,7 @@ const FormSwitchWrapper = ({
   }
 
   return (
-    <div className={classes.divContainer}>
+    <div>
       <Grid container alignItems={"center"}>
         <Grid item xs={12} sm={8} md={8}>
           {label !== "" && (
@@ -193,8 +190,8 @@ const FormSwitchWrapper = ({
         <Grid
           item
           xs={12}
-          sm={4}
-          md={4}
+          sm={label !== "" ? 4 : 12}
+          md={label !== "" ? 4 : 12}
           textAlign={"right"}
           justifyContent={"end"}
           className={classes.switchContainer}

portal-ui/src/screens/Console/Tenants/AddTenant/AddTenant.tsx

@@ -803,7 +803,15 @@ const AddTenant = ({
           entity="Tenant"
         />
       )}
-      <PageHeader label={"Create New Tenant"} />
+      <PageHeader
+        label={
+          <BackLink
+            to={"/tenants"}
+            label={"Tenants"}
+            executeOnClick={resetAddTenantForm}
+          />
+        }
+      />
 
       <PageLayout>
         {addSending && (
@@ -811,14 +819,6 @@ const AddTenant = ({
             <LinearProgress />
           </Grid>
         )}
-
-        <Grid item xs={12}>
-          <BackLink
-            to={"/tenants"}
-            label={"Tenant List"}
-            executeOnClick={resetAddTenantForm}
-          />
-        </Grid>
         <Grid item xs={12} className={classes.pageBox}>
           <GenericWizard wizardSteps={filteredWizardSteps} />
         </Grid>

portal-ui/src/screens/Console/Tenants/ListTenants/InformationItem.tsx

@@ -20,13 +20,25 @@ interface IInformationItemProps {
   label: string;
   value: string;
   unit?: string;
+  variant?: "normal" | "faded";
 }
 
-const InformationItem = ({ label, value, unit }: IInformationItemProps) => {
+const InformationItem = ({
+  label,
+  value,
+  unit,
+  variant = "normal",
+}: IInformationItemProps) => {
   return (
     <div style={{ margin: "0px 20px" }}>
       <div style={{ textAlign: "center" }}>
-        <span style={{ fontSize: 18, color: "#000", fontWeight: 400 }}>
+        <span
+          style={{
+            fontSize: 18,
+            color: variant === "normal" ? "#000" : "#999",
+            fontWeight: 400,
+          }}
+        >
           {value}
         </span>
         {unit && (
@@ -43,7 +55,7 @@ const InformationItem = ({ label, value, unit }: IInformationItemProps) => {
       <div
         style={{
           textAlign: "center",
-          color: "#767676",
+          color: variant === "normal" ? "#767676" : "#bababa",
           fontSize: 12,
           whiteSpace: "nowrap",
         }}

portal-ui/src/screens/Console/Tenants/ListTenants/ListTenants.tsx

@@ -95,7 +95,6 @@ const styles = (theme: Theme) =>
       marginBottom: 8,
     },
     tenantsList: {
-      marginTop: 25,
       height: "calc(100vh - 195px)",
     },
   });

portal-ui/src/screens/Console/Tenants/ListTenants/TenantListItem.tsx

@@ -24,10 +24,9 @@ import { Theme } from "@mui/material/styles";
 import createStyles from "@mui/styles/createStyles";
 import withStyles from "@mui/styles/withStyles";
 import { niceBytes, niceBytesInt } from "../../../../common/utils";
-import { tenantIsOnline } from "./utils";
-import { Button } from "@mui/material";
 import InformationItem from "./InformationItem";
 import TenantCapacity from "./TenantCapacity";
+import { DrivesIcon } from "../../../../icons";
 
 const styles = (theme: Theme) =>
   createStyles({
@@ -76,9 +75,8 @@ const styles = (theme: Theme) =>
       height: 10,
     },
     tenantItem: {
-      border: "1px solid #EAEDEE",
-      borderRadius: 3,
-      marginBottom: 20,
+      border: "1px solid #EAEAEA",
+      marginBottom: 16,
       padding: "15px 30px",
       "&:hover": {
         backgroundColor: "#FAFAFA",
@@ -154,6 +152,8 @@ const TenantListItem = ({ tenant, classes }: ITenantListItem) => {
   let raw: ValueUnit = { value: "n/a", unit: "" };
   let capacity: ValueUnit = { value: "n/a", unit: "" };
   let used: ValueUnit = { value: "n/a", unit: "" };
+  let localUse: ValueUnit = { value: "n/a", unit: "" };
+  let tieredUse: ValueUnit = { value: "n/a", unit: "" };
 
   if (tenant.capacity_raw) {
     const b = niceBytes(`${tenant.capacity_raw}`, true);
@@ -175,7 +175,6 @@ const TenantListItem = ({ tenant, classes }: ITenantListItem) => {
   }
 
   let spaceVariants: CapacityValues[] = [];
-
   if (!tenant.tiers || tenant.tiers.length === 0) {
     spaceVariants = [
       { value: tenant.capacity_usage || 0, variant: "STANDARD" },
@@ -184,6 +183,26 @@ const TenantListItem = ({ tenant, classes }: ITenantListItem) => {
     spaceVariants = tenant.tiers.map((itemTenant) => {
       return { value: itemTenant.size, variant: itemTenant.name };
     });
+    let internalUsage = tenant.tiers
+      .filter((itemTenant) => {
+        return itemTenant.type === "internal";
+      })
+      .reduce((sum, itemTenant) => sum + itemTenant.size, 0);
+    let tieredUsage = tenant.tiers
+      .filter((itemTenant) => {
+        return itemTenant.type !== "internal";
+      })
+      .reduce((sum, itemTenant) => sum + itemTenant.size, 0);
+
+    const t = niceBytesInt(tieredUsage, true);
+    const parts = t.split(" ");
+    tieredUse.value = parts[0];
+    tieredUse.unit = parts[1];
+
+    const is = niceBytesInt(internalUsage, true);
+    const partsInternal = is.split(" ");
+    localUse.value = partsInternal[0];
+    localUse.unit = partsInternal[1];
   }
 
   const openTenantDetails = () => {
@@ -238,14 +257,10 @@ const TenantListItem = ({ tenant, classes }: ITenantListItem) => {
                     value={capacity.value}
                     unit={capacity.unit}
                   />
-                  <InformationItem
-                    label={"Usage"}
-                    value={used.value}
-                    unit={used.unit}
-                  />
                   <InformationItem
                     label={"Pools"}
                     value={tenant.pool_count.toString()}
+                    variant={"faded"}
                   />
                 </Grid>
                 <Grid
@@ -258,40 +273,86 @@ const TenantListItem = ({ tenant, classes }: ITenantListItem) => {
                   </span>
                 </Grid>
               </Grid>
-              <Grid
-                item
-                xs={2}
-                sx={{
-                  display: "flex",
-                  alignItems: "center",
-                  justifyContent: "flex-end",
-                }}
-              >
-                <Button
-                  id={"manage-tenant-" + tenant.name}
-                  disabled={!tenantIsOnline(tenant)}
-                  onClick={(e) => {
-                    e.preventDefault();
-                    e.stopPropagation();
+              <Grid item xs={3}>
+                <Fragment>
+                  <Grid container>
+                    <Grid
+                      item
+                      xs={2}
+                      textAlign={"center"}
+                      justifyContent={"center"}
+                      justifyItems={"center"}
+                    >
+                      <DrivesIcon
+                        style={{ width: 25, color: "rgb(91,91,91)" }}
+                      />
+                      <div
+                        style={{
+                          color: "rgb(118, 118, 118)",
+                          fontSize: 12,
+                          fontWeight: "400",
+                        }}
+                      >
+                        Usage
+                      </div>
+                    </Grid>
+                    <Grid item xs={1} />
+                    <Grid item style={{ paddingTop: 8 }}>
+                      {(!tenant.tiers || tenant.tiers.length === 0) && (
+                        <div
+                          style={{
+                            fontSize: 14,
+                            fontWeight: 400,
+                          }}
+                        >
+                          <span
+                            style={{
+                              color: "rgb(62,62,62)",
+                            }}
+                          >
+                            Internal:{" "}
+                          </span>{" "}
+                          {`${used.value} ${used.unit}`}
+                        </div>
+                      )}
 
-                    history.push(
-                      `/namespaces/${tenant.namespace}/tenants/${tenant.name}/hop`
-                    );
-                  }}
-                  disableTouchRipple
-                  disableRipple
-                  focusRipple={false}
-                  sx={{
-                    color: "#5E5E5E",
-                    border: "#5E5E5E 1px solid",
-                    whiteSpace: "nowrap",
-                    paddingLeft: 4.5,
-                    paddingRight: 4.5,
-                  }}
-                  variant={"outlined"}
-                >
-                  Manage
-                </Button>
+                      {tenant.tiers && tenant.tiers.length > 0 && (
+                        <Fragment>
+                          <div
+                            style={{
+                              fontSize: 14,
+                              fontWeight: 400,
+                            }}
+                          >
+                            <span
+                              style={{
+                                color: "rgb(62,62,62)",
+                              }}
+                            >
+                              Internal:{" "}
+                            </span>{" "}
+                            {`${localUse.value} ${localUse.unit}`}
+                          </div>
+                          <div
+                            style={{
+                              fontSize: 14,
+                              fontWeight: 400,
+                            }}
+                          >
+                            <span
+                              style={{
+                                color: "rgb(62,62,62)",
+                              }}
+                            >
+                              Tiered:{" "}
+                            </span>{" "}
+                            {`${tieredUse.value} ${tieredUse.unit}`}
+                          </div>
+                        </Fragment>
+                      )}
+                    </Grid>
+                  </Grid>
+                </Fragment>
               </Grid>
             </Grid>
           </Grid>

portal-ui/src/screens/Console/Tenants/TenantDetails/Pools/Details/PoolsListing.tsx

@@ -128,10 +128,6 @@ const PoolsListing = ({
           variant={"contained"}
         />
       </Grid>
-      <Grid item xs={12}>
-        <br />
-      </Grid>
-
       <Grid item xs={12} className={classes.tableBlock}>
         <TableWrapper
           itemActions={listActions}

portal-ui/src/screens/Console/Tenants/TenantDetails/TenantDetails.tsx

@@ -39,7 +39,7 @@ import { AppState } from "../../../../store";
 import { ErrorResponseHandler } from "../../../../common/types";
 import api from "../../../../common/api";
 import PageHeader from "../../Common/PageHeader/PageHeader";
-import { CircleIcon, TrashIcon } from "../../../../icons";
+import { CircleIcon, MinIOTierIconXs, TrashIcon } from "../../../../icons";
 import { niceBytes } from "../../../../common/utils";
 import ScreenTitle from "../../Common/ScreenTitle/ScreenTitle";
 import EditIcon from "../../../../icons/EditIcon";
@@ -51,6 +51,7 @@ import VerticalTabs from "../../Common/VerticalTabs/VerticalTabs";
 import BoxIconButton from "../../Common/BoxIconButton/BoxIconButton";
 import withSuspense from "../../Common/Components/withSuspense";
 import { IAM_PAGES } from "../../../../common/SecureComponent/permissions";
+import { tenantIsOnline } from "../ListTenants/utils";
 
 const TenantYAML = withSuspense(React.lazy(() => import("./TenantYAML")));
 const TenantSummary = withSuspense(React.lazy(() => import("./TenantSummary")));
@@ -377,7 +378,7 @@ const TenantDetails = ({
                   }}
                   size="large"
                 >
-                  <span>Delete Tenant</span> <TrashIcon />
+                  <span>Delete</span> <TrashIcon />
                 </BoxIconButton>
                 <BoxIconButton
                   classes={{
@@ -392,9 +393,26 @@ const TenantDetails = ({
                   }}
                   size="large"
                 >
-                  <span>Edit Tenant</span>
+                  <span>YAML</span>
                   <EditIcon />
                 </BoxIconButton>
+                <BoxIconButton
+                  classes={{
+                    root: classes.tenantActionButton,
+                  }}
+                  tooltip={"Management Console"}
+                  onClick={() => {
+                    history.push(
+                      `/namespaces/${tenantNamespace}/tenants/${tenantName}/hop`
+                    );
+                  }}
+                  disabled={!tenantInfo || !tenantIsOnline(tenantInfo)}
+                  variant={"outlined"}
+                  color="primary"
+                >
+                  <span>Console</span>{" "}
+                  <MinIOTierIconXs style={{ height: 16 }} />
+                </BoxIconButton>
                 <BoxIconButton
                   classes={{
                     root: classes.tenantActionButton,
@@ -407,7 +425,7 @@ const TenantDetails = ({
                     setTenantDetailsLoad(true);
                   }}
                 >
-                  <span>Reload</span> <RefreshIcon />
+                  <span>Refresh</span> <RefreshIcon />
                 </BoxIconButton>
               </div>
             }

portal-ui/src/screens/Console/Tenants/TenantDetails/TenantLogging.tsx

@@ -207,43 +207,52 @@ const TenantLogging = ({
           dbMemRequest={logInfo.logDBMemRequest}
         />
       )}
-      <h1 className={classes.sectionTitle}>Logging</h1>
-      <div className={classes.actionsTray}>
-        <FormSwitchWrapper
-          value="enableLogging"
-          id="enableLogging"
-          name="enableLogging"
-          checked={!preDisabled}
-          onChange={(e) => {
-            const targetD = e.target;
-            const checked = targetD.checked;
-            if (checked) {
-              setEnableDialogOpen(true);
-            } else {
-              setDisableDialogOpen(true);
-            }
-          }}
-          label={"Logging"}
-          indicatorLabels={["Enabled", "Disabled"]}
-        />
-        {!disabled && !loadingTenantLogs && (
-          <RBIconButton
-            tooltip={"Edit Logging configuration"}
-            text={"Edit"}
-            onClick={() => {
-              setEdit(true);
+      <Grid container alignItems={"center"}>
+        <Grid item xs>
+          <h1 className={classes.sectionTitle}>Logging</h1>
+        </Grid>
+        <Grid item xs={4}>
+          <FormSwitchWrapper
+            value="enableLogging"
+            id="enableLogging"
+            name="enableLogging"
+            checked={!preDisabled}
+            onChange={(e) => {
+              const targetD = e.target;
+              const checked = targetD.checked;
+              if (checked) {
+                setEnableDialogOpen(true);
+              } else {
+                setDisableDialogOpen(true);
+              }
             }}
-            icon={<EditIcon />}
-            color="primary"
-            variant={"contained"}
+            indicatorLabels={["Enabled", "Disabled"]}
           />
-        )}
-      </div>
+        </Grid>
+      </Grid>
       {!disabled && !loadingTenantLogs && (
         <Paper className={classes.paperContainer}>
           <Grid container>
             <Grid item xs={12}>
-              <h2>Logging API Service Details</h2>
+              <Grid container alignItems={"center"}>
+                <Grid xs={8}>
+                  <h3>Configuration</h3>
+                </Grid>
+                <Grid xs={4} justifyContent={"end"} textAlign={"right"}>
+                  <RBIconButton
+                    tooltip={"Edit Logging configuration"}
+                    text={"Edit"}
+                    onClick={() => {
+                      setEdit(true);
+                    }}
+                    icon={<EditIcon />}
+                    color="primary"
+                    variant={"contained"}
+                  />
+                </Grid>
+              </Grid>
+            </Grid>
+            <Grid item xs={12}>
               <hr className={classes.hrClass} />
               <table width={"100%"}>
                 <tbody>

portal-ui/src/screens/Console/Tenants/TenantDetails/TenantMonitoring.tsx

@@ -188,37 +188,48 @@ const TenantMonitoring = ({
         />
       )}
 
-      <h1 className={classes.sectionTitle}>Monitoring</h1>
-      <div className={classes.actionsTray}>
-        <FormSwitchWrapper
-          label={"Prometheus Monitoring"}
-          indicatorLabels={["Enabled", "Disabled"]}
-          checked={prometheusMonitoringEnabled}
-          value={"monitoring_status"}
-          id="monitoring-status"
-          name="monitoring-status"
-          onChange={(e) => {
-            setConfirmOpen(true);
-          }}
-          description=""
-        />
-        {prometheusMonitoringEnabled && (
-          <RBIconButton
-            tooltip={"Edit Monitoring configuration"}
-            text={"Edit"}
-            onClick={() => {
-              setEdit(true);
+      <Grid container alignItems={"center"}>
+        <Grid item xs>
+          <h1 className={classes.sectionTitle}>Monitoring</h1>
+        </Grid>
+        <Grid item xs={4}>
+          <FormSwitchWrapper
+            indicatorLabels={["Enabled", "Disabled"]}
+            checked={prometheusMonitoringEnabled}
+            value={"monitoring_status"}
+            id="monitoring-status"
+            name="monitoring-status"
+            onChange={(e) => {
+              setConfirmOpen(true);
             }}
-            icon={<EditIcon />}
-            color="primary"
-            variant={"contained"}
+            description=""
           />
-        )}
-      </div>
+        </Grid>
+      </Grid>
       {prometheusMonitoringEnabled && monitoringInfo !== undefined && (
         <Paper className={classes.paperContainer}>
           <Grid container>
             <Grid item xs={12}>
+              <Grid container alignItems={"center"}>
+                <Grid xs={8}>
+                  <h3>Configuration</h3>
+                </Grid>
+                <Grid xs={4} justifyContent={"end"} textAlign={"right"}>
+                  <RBIconButton
+                    tooltip={"Edit Monitoring configuration"}
+                    text={"Edit"}
+                    onClick={() => {
+                      setEdit(true);
+                    }}
+                    icon={<EditIcon />}
+                    color="primary"
+                    variant={"contained"}
+                  />
+                </Grid>
+              </Grid>
+            </Grid>
+            <Grid item xs={12}>
+              <hr className={classes.hrClass} />
               <table width={"100%"}>
                 <tbody>
                   {loadingTenant ? (

portal-ui/src/screens/Console/Tenants/TenantDetails/TenantSummary.tsx

@@ -29,16 +29,12 @@ import Paper from "@mui/material/Paper";
 import { ITenant } from "../ListTenants/types";
 import UpdateTenantModal from "./UpdateTenantModal";
 import { AppState } from "../../../../store";
-import history from "./../../../../history";
-import { tenantIsOnline } from "../ListTenants/utils";
 import AButton from "../../Common/AButton/AButton";
 import { styled } from "@mui/styles";
 import SummaryUsageBar from "../../Common/UsageBarWrapper/SummaryUsageBar";
 import LabelValuePair from "../../Common/UsageBarWrapper/LabelValuePair";
 import FormSwitchWrapper from "../../Common/FormComponents/FormSwitchWrapper/FormSwitchWrapper";
 import StackRow from "../../Common/UsageBarWrapper/StackRow";
-import { SettingsIcon } from "../../../../icons";
-import RBIconButton from "../../Buckets/BucketDetails/SummaryItems/RBIconButton";
 
 interface ITenantsSummary {
   classes: any;
@@ -237,21 +233,7 @@ const TenantSummary = ({
         <StackItem>
           <h3>Details</h3>
         </StackItem>
-        <StackItem>
-          <RBIconButton
-            tooltip={"Manage Tenant"}
-            text={"Manage Tenant"}
-            onClick={() => {
-              history.push(
-                `/namespaces/${tenantNamespace}/tenants/${tenantName}/hop`
-              );
-            }}
-            disabled={!tenant || !tenantIsOnline(tenant)}
-            icon={<SettingsIcon />}
-            color="primary"
-            variant={"contained"}
-          />
-        </StackItem>
+        <StackItem></StackItem>
       </Stack>
 
       <StorageSummary tenant={tenant} classes={classes} />