feat(FR-2858): add revision number display and expand filter/sort in revision history tab

Author: agatha197

data/schema.graphql

@@ -202,10 +202,12 @@ const DeploymentConfigurationSection: React.FC<
         }
         currentRevision @since(version: "26.4.3") {
           id
+          revisionNumber
           ...DeploymentRevisionDetail_revision
         }
         deployingRevision @since(version: "26.4.3") {
           id
+          revisionNumber
           ...DeploymentRevisionDetail_revision
         }
         ...DeploymentRevisionHistoryTab_deployment
@@ -376,7 +378,10 @@ const DeploymentConfigurationSection: React.FC<
                 icon={<LoadingOutlined spin />}
                 showIcon
                 title={t('deployment.DeployingRevisionApplying', {
-                  name: toLocalId(deployingRevision.id) ?? '',
+                  revisionNumber:
+                    deployingRevision.revisionNumber != null
+                      ? `#${deployingRevision.revisionNumber}`
+                      : (toLocalId(deployingRevision.id) ?? ''),
                 })}
                 action={
                   <Button

react/src/components/DeploymentConfigurationSection.tsx

@@ -14,6 +14,7 @@ import {
   BAIFlex,
   BAIResourceNumberWithIcon,
   BAITag,
+  BAIText,
   filterOutEmpty,
   filterOutNullAndUndefined,
   toLocalId,
@@ -40,6 +41,7 @@ const DeploymentRevisionDetail: React.FC<{
     graphql`
       fragment DeploymentRevisionDetail_revision on ModelRevision {
         id
+        revisionNumber
         createdAt
         clusterConfig {
           mode
@@ -129,11 +131,21 @@ const DeploymentRevisionDetail: React.FC<{
 
   const baseItems: DescriptionsItemType[] = filterOutEmpty([
     {
-      key: 'revision-name',
+      key: 'revision-number',
+      label: t('deployment.RevisionNumber'),
+      children:
+        revision.revisionNumber != null ? (
+          <BAIText>{`#${revision.revisionNumber}`}</BAIText>
+        ) : (
+          renderFallback()
+        ),
+    },
+    {
+      key: 'revision-id',
       label: t('modelService.RevisionID'),
       children: revision.id ? (
         <BAIFlex gap="xs" align="center">
-          <Typography.Text>{toLocalId(revision.id)}</Typography.Text>
+          <BAIText copyable>{toLocalId(revision.id)}</BAIText>
           {status === 'current' && (
             <BAITag color="success">{t('deployment.Current')}</BAITag>
           )}

react/src/components/DeploymentRevisionDetail.tsx

@@ -24,12 +24,15 @@ import {
   BAITable,
   BAITag,
   BAIUnmountAfterClose,
+  BAIId,
   INITIAL_FETCH_KEY,
   type GraphQLFilter,
   filterOutNullAndUndefined,
+  isValidUUID,
   toLocalId,
   useBAILogger,
   useFetchKey,
+  BAIText,
 } from 'backend.ai-ui';
 import dayjs from 'dayjs';
 import * as _ from 'lodash-es';
@@ -51,12 +54,20 @@ import {
 type RevisionNode = NonNullable<
   NonNullable<
     NonNullable<
-      DeploymentRevisionHistoryTabListQuery['response']['deployment']
-    >['revisionHistory']['edges'][number]
+      NonNullable<
+        DeploymentRevisionHistoryTabListQuery['response']['deployment']
+      >['revisionHistory']
+    >['edges'][number]
   >['node']
 >;
 
-const availableRevisionSorterKeys = ['createdAt'] as const;
+const availableRevisionSorterKeys = [
+  'revisionNumber',
+  'createdAt',
+  'resourceGroup',
+  'clusterMode',
+  'runtimeVariantName',
+] as const;
 const availableRevisionSorterValues = [
   ...availableRevisionSorterKeys,
   ...availableRevisionSorterKeys.map((key) => `-${key}` as const),
@@ -195,12 +206,16 @@ const DeploymentRevisionHistoryTab: React.FC<
               edges {
                 node {
                   id
+                  revisionNumber
                   createdAt
 
                   clusterConfig {
                     mode
                     size
                   }
+                  resourceConfig {
+                    resourceGroupName
+                  }
                   modelRuntimeConfig {
                     runtimeVariant {
                       name
@@ -220,6 +235,13 @@ const DeploymentRevisionHistoryTab: React.FC<
                       canonicalName
                     }
                   }
+                  modelMountConfig {
+                    vfolderId
+                    vfolder {
+                      id
+                      name
+                    }
+                  }
                   ...DeploymentRevisionDetail_revision
                 }
               }
@@ -277,12 +299,11 @@ const DeploymentRevisionHistoryTab: React.FC<
   };
 
   const handleRollback = (revision: RevisionNode): Promise<boolean> => {
-    const revisionLabel = toLocalId(revision.id);
     return new Promise<boolean>((resolveOuter) => {
       modal.confirm({
         title: t('deployment.Deploy'),
         content: t('deployment.DeployConfirm', {
-          revisionNumber: revisionLabel,
+          revisionNumber: revision.revisionNumber,
         }),
         okText: t('deployment.Deploy'),
         okButtonProps: {
@@ -315,7 +336,7 @@ const DeploymentRevisionHistoryTab: React.FC<
                 upsertNotification({
                   open: true,
                   message: t('deployment.DeploySuccess', {
-                    revisionNumber: revisionLabel,
+                    revisionNumber: revision.revisionNumber,
                   }),
                 });
                 handleRefresh();
@@ -342,10 +363,11 @@ const DeploymentRevisionHistoryTab: React.FC<
 
   const columns: BAIColumnType<RevisionNode>[] = [
     {
-      title: t('deployment.RevisionNumber'),
+      title: t('deployment.RevisionNumberWithID'),
       dataIndex: 'revisionNumber',
       key: 'revisionNumber',
       fixed: 'left',
+      sorter: true,
       render: (_value, record) => {
         // `currentRevisionId` and `deployingRevisionId` come back as raw
         // UUIDs from the scalar fields, while `record.id` is the
@@ -378,8 +400,15 @@ const DeploymentRevisionHistoryTab: React.FC<
                     })
                   }
                 >
-                  {recordLocalId ?? '-'}
+                  {record.revisionNumber != null
+                    ? `#${record.revisionNumber}`
+                    : '-'}
                 </Typography.Link>
+                <BAIFlex gap={0} align="center">
+                  {'('}
+                  <BAIId globalId={record.id} />
+                  {')'}
+                </BAIFlex>
                 {isCurrent ? (
                   <BAITag color="success">{t('deployment.Current')}</BAITag>
                 ) : null}
@@ -423,6 +452,7 @@ const DeploymentRevisionHistoryTab: React.FC<
     {
       title: t('deployment.ModelVersion'),
       key: 'modelVersion',
+      defaultHidden: true,
       render: (_value, record) => {
         const model = record.modelDefinition?.models?.[0];
         if (!model) return '-';
@@ -441,40 +471,96 @@ const DeploymentRevisionHistoryTab: React.FC<
     },
     {
       title: t('deployment.RuntimeVariant'),
-      key: 'runtimeVariant',
-      dataIndex: 'runtimeVariant',
+      key: 'runtimeVariantName',
+      dataIndex: 'runtimeVariantName',
+      sorter: true,
       render: (_value, record) =>
         record.modelRuntimeConfig?.runtimeVariant?.name ?? '-',
     },
     {
       title: t('deployment.Image'),
       key: 'image',
+      defaultHidden: true,
       render: (_value, record) => {
         const canonicalName = record.imageV2?.identity?.canonicalName;
-        if (!canonicalName) return '-';
+        const imageGlobalId = record.imageV2?.id;
+        if (!canonicalName && !imageGlobalId) return '-';
+        return (
+          <BAIFlex gap="xs" align="center" wrap="wrap">
+            {canonicalName ? (
+              <BAIText
+                copyable
+                ellipsis={{ tooltip: canonicalName }}
+                style={{ maxWidth: 180 }}
+              >
+                {canonicalName}
+              </BAIText>
+            ) : null}
+            {imageGlobalId ? (
+              <BAIFlex gap={0} align="center">
+                {'('}
+                <BAIId globalId={imageGlobalId} />
+                {')'}
+              </BAIFlex>
+            ) : null}
+          </BAIFlex>
+        );
+      },
+    },
+    {
+      title: t('deployment.ModelFolder'),
+      key: 'modelFolder',
+      defaultHidden: true,
+      render: (_value, record) => {
+        const vfolder = record.modelMountConfig?.vfolder;
+        const vfolderId = record.modelMountConfig?.vfolderId;
+        if (!vfolder?.name && !vfolderId) return '-';
         return (
-          <Typography.Text
-            copyable={{ text: canonicalName }}
-            ellipsis={{ tooltip: canonicalName }}
-            style={{ maxWidth: 240 }}
-          >
-            {canonicalName}
-          </Typography.Text>
+          <BAIFlex gap="xs" align="center" wrap="wrap">
+            {vfolder?.name ? (
+              <Typography.Text>{vfolder.name}</Typography.Text>
+            ) : null}
+            {vfolder?.id ? (
+              <BAIFlex gap={0} align="center">
+                {'('}
+                <BAIId globalId={vfolder.id} />
+                {')'}
+              </BAIFlex>
+            ) : vfolderId ? (
+              <Typography.Text type="secondary">{vfolderId}</Typography.Text>
+            ) : null}
+          </BAIFlex>
         );
       },
     },
     {
       title: t('deployment.ClusterSize'),
-      key: 'clusterSize',
+      key: 'clusterMode',
+      dataIndex: 'clusterMode',
+      sorter: true,
       render: (_value, record) => {
         const mode = record.clusterConfig?.mode;
         const size = record.clusterConfig?.size;
         if (size == null) return '-';
         return mode ? `${size} (${mode})` : `${size}`;
       },
     },
+    {
+      title: t('deployment.ResourceGroup'),
+      key: 'resourceGroup',
+      dataIndex: 'resourceGroup',
+      sorter: true,
+      defaultHidden: true,
+      render: (_value, record) =>
+        record.resourceConfig?.resourceGroupName ?? '-',
+    },
   ];
 
+  const uuidRule = {
+    message: t('general.InvalidUUID'),
+    validate: isValidUUID,
+  };
+
   const filterProperties = [
     {
       key: 'revisionNumber',
@@ -488,6 +574,30 @@ const DeploymentRevisionHistoryTab: React.FC<
       operators: ['after' as const, 'before' as const],
       defaultOperator: 'after' as const,
     },
+    {
+      key: 'resourceGroup',
+      propertyLabel: t('deployment.ResourceGroup'),
+      type: 'string' as const,
+    },
+    {
+      key: 'clusterMode',
+      propertyLabel: t('deployment.ClusterMode'),
+      type: 'string' as const,
+    },
+    {
+      key: 'imageId',
+      propertyLabel: t('deployment.Image'),
+      type: 'uuid' as const,
+      fixedOperator: 'equals' as const,
+      rule: uuidRule,
+    },
+    {
+      key: 'modelVfolderId',
+      propertyLabel: t('deployment.ModelFolder'),
+      type: 'uuid' as const,
+      fixedOperator: 'equals' as const,
+      rule: uuidRule,
+    },
   ];
 
   const filterValue: GraphQLFilter | undefined = queryParams.rvFilter

react/src/components/DeploymentRevisionHistoryTab.tsx

@@ -918,7 +918,7 @@
     "DeployDisabled": "Es kann keine Revision ausgewählt werden, die gerade bereitgestellt wird oder bereits bereitgestellt ist.",
     "DeploySuccess": "Die Bereitstellung von Revision #{{revisionNumber}} wurde angefordert.",
     "Deploying": "Wird bereitgestellt",
-    "DeployingRevisionApplying": "Revision {{name}} wird bereitgestellt.",
+    "DeployingRevisionApplying": "Revision {{revisionNumber}} wird bereitgestellt.",
     "DeployingRevisionDetail": "Details zur laufenden Bereitstellung",
     "Deployment": "Bereitstellung",
     "DeploymentCreated": "Bereitstellung wurde erstellt.",
@@ -987,6 +987,7 @@
     "RevisionId": "Revisions-ID",
     "RevisionIdPlaceholder": "Revisions-ID eingeben (optional)",
     "RevisionNumber": "Revisionsnummer",
+    "RevisionNumberWithID": "Revision (ID)",
     "Revisions": "Revisionen",
     "Running": "Läuft",
     "RuntimeVariant": "Laufzeit",
@@ -1469,6 +1470,7 @@
     "InProgress": "In Arbeit",
     "Inactive": "Inaktiv",
     "InvalidJSONFormat": "Ungültiges JSON-Format.",
+    "InvalidUUID": "Bitte geben Sie eine gültige UUID ein",
     "LastUpdated": "Zuletzt aktualisiert",
     "Loading": "Wird geladen...",
     "Manual": "Handbuch",

resources/i18n/de.json

@@ -918,7 +918,7 @@
     "DeployDisabled": "Δεν είναι δυνατή η επιλογή Revision που βρίσκεται σε ανάπτυξη ή είναι ήδη αναπτυγμένη.",
     "DeploySuccess": "Το αίτημα ανάπτυξης για τη Revision #{{revisionNumber}} υποβλήθηκε.",
     "Deploying": "Σε ανάπτυξη",
-    "DeployingRevisionApplying": "Γίνεται ανάπτυξη της Revision {{name}}.",
+    "DeployingRevisionApplying": "Γίνεται ανάπτυξη της Revision {{revisionNumber}}.",
     "DeployingRevisionDetail": "Λεπτομέρειες Εκτελούμενης Ανάπτυξης",
     "Deployment": "Ανάπτυξη",
     "DeploymentCreated": "Η ανάπτυξη δημιουργήθηκε.",
@@ -987,6 +987,7 @@
     "RevisionId": "ID Revision",
     "RevisionIdPlaceholder": "Εισάγετε ID Revision (προαιρετικό)",
     "RevisionNumber": "Αριθμός Αναθεώρησης",
+    "RevisionNumberWithID": "Αναθεώρηση (ID)",
     "Revisions": "Revision",
     "Running": "Σε εκτέλεση",
     "RuntimeVariant": "Χρόνος εκτέλεσης",
@@ -1467,6 +1468,7 @@
     "InProgress": "Σε εξέλιξη",
     "Inactive": "Αδρανής",
     "InvalidJSONFormat": "Μη έγκυρη μορφή JSON.",
+    "InvalidUUID": "Παρακαλώ εισάγετε έγκυρο UUID",
     "LastUpdated": "Τελευταία ενημέρωση",
     "Loading": "Φόρτωση...",
     "Manual": "Χειροκίνητο",