ydb-embedded-ui 22.0.0 → 22.2.0

package/src/store/reducers/capabilities/hooks.ts

@@ -199,6 +199,15 @@ export const useAnalyzeOperationAvailable = () => {
     return operationListVersion >= 2 && Boolean(analyzeLongRunningOperationEnabled);
 };
 
+export const useTabletDevUiSecurePath = () => {
+    const database = useDatabaseFromQuery();
+
+    return useTypedSelector((state) => {
+        const {data, isError} = selectDatabaseCapabilities(state, database);
+        return !isError && data?.Settings?.Features?.EnableTabletDevUiSecurePath === true;
+    });
+};
+
 export const useGraphShardExists = () => {
     const database = useDatabaseFromQuery();
 

package/src/store/reducers/nodesList.ts

@@ -27,7 +27,6 @@ const createGetNodesListSelector = createSelector(
 );
 
 export const selectNodesMap = createSelector(
-    (state: RootState) => state,
-    (_state: RootState, database?: string) => createGetNodesListSelector(database),
-    (state, selectNodesList) => prepareNodesMap(selectNodesList(state).data),
+    (state: RootState, database?: string) => createGetNodesListSelector(database)(state).data,
+    prepareNodesMap,
 );

package/src/store/reducers/settings/constants.ts

@@ -45,6 +45,8 @@ export const SETTING_KEYS = {
     STORAGE_EXPERT_MODE: 'storageExpertMode',
     STORAGE_VDISKS_GROUP_BY: 'storageVDisksGroupBy',
     STORAGE_PDISKS_GROUP_BY: 'storagePDisksGroupBy',
+    STORAGE_NODES_VDISKS_GROUP_BY: 'storageNodesVDisksGroupBy',
+    STORAGE_NODES_PDISKS_GROUP_BY: 'storageNodesPDisksGroupBy',
 } as const;
 
 export type SettingKey = ValueOf<typeof SETTING_KEYS>;
@@ -87,6 +89,8 @@ export const DEFAULT_USER_SETTINGS = {
     [SETTING_KEYS.STORAGE_EXPERT_MODE]: false,
     [SETTING_KEYS.STORAGE_VDISKS_GROUP_BY]: 'State',
     [SETTING_KEYS.STORAGE_PDISKS_GROUP_BY]: 'State',
+    [SETTING_KEYS.STORAGE_NODES_VDISKS_GROUP_BY]: 'State',
+    [SETTING_KEYS.STORAGE_NODES_PDISKS_GROUP_BY]: 'State',
 } as const satisfies Record<SettingKey, unknown>;
 
 export const SETTINGS_OPTIONS: Record<string, SettingOptions | undefined> = {

package/src/store/reducers/storage/prepareGroupsDisks.ts

@@ -9,6 +9,7 @@ import type {PDiskType, PreparedVDisk} from '../../../utils/disks/types';
 import {parseOptionalNonNegativeNumber} from '../../../utils/utils';
 
 export function prepareGroupsVDisk(data: TStorageVDisk = {}): PreparedVDisk {
+    const hasWhiteboardData = data.HasWhiteboardData ?? Boolean(data.Whiteboard);
     const {Whiteboard: whiteboardVDisk = {}, PDisk, ...bscVDisk} = data;
 
     const mergedVDiskData = {
@@ -36,13 +37,14 @@ export function prepareGroupsVDisk(data: TStorageVDisk = {}): PreparedVDisk {
     const whiteboardSlotSize = parseOptionalNonNegativeNumber(
         PDisk?.Whiteboard?.EnforcedDynamicSlotSize,
     );
-    const whiteboardSizeFields = data.Whiteboard
-        ? prepareVDiskSizeFields({
-              AvailableSize: whiteboardAvailableSize,
-              AllocatedSize: whiteboardAllocatedSize,
-              SlotSize: whiteboardSlotSize,
-          })
-        : undefined;
+    const whiteboardSizeFields =
+        hasWhiteboardData && data.Whiteboard
+            ? prepareVDiskSizeFields({
+                  AvailableSize: whiteboardAvailableSize,
+                  AllocatedSize: whiteboardAllocatedSize,
+                  SlotSize: whiteboardSlotSize,
+              })
+            : undefined;
     const WhiteboardSize = whiteboardSizeFields
         ? {
               AllocatedSize:
@@ -72,6 +74,7 @@ export function prepareGroupsVDisk(data: TStorageVDisk = {}): PreparedVDisk {
     return {
         ...mergedVDiskData,
         ...vDiskSizeFields,
+        HasWhiteboardData: hasWhiteboardData,
         ...(WhiteboardSize ? {WhiteboardSize} : {}),
         PDisk: preparedPDisk,
         Donors: preparedDonors,
@@ -82,6 +85,7 @@ export function prepareGroupsVDisk(data: TStorageVDisk = {}): PreparedVDisk {
 }
 
 export function prepareGroupsPDisk(data: TStoragePDisk & {NodeId?: number} = {}) {
+    const hasWhiteboardData = data.HasWhiteboardData ?? Boolean(data.Whiteboard);
     const {Whiteboard: whiteboardPDisk, Status: DriveStatus, ...bscPDisk} = data;
 
     const mergedPDiskData = {
@@ -109,12 +113,12 @@ export function prepareGroupsPDisk(data: TStoragePDisk & {NodeId?: number} = {})
     const whiteboardAvailableSize = parseOptionalNonNegativeNumber(whiteboardPDisk?.AvailableSize);
     const whiteboardTotalSize = parseOptionalNonNegativeNumber(whiteboardPDisk?.TotalSize);
     const whiteboardSizeFields =
-        whiteboardPDisk === undefined
-            ? undefined
-            : preparePDiskSizeFields({
+        hasWhiteboardData && whiteboardPDisk
+            ? preparePDiskSizeFields({
                   AvailableSize: whiteboardAvailableSize,
                   TotalSize: whiteboardTotalSize,
-              });
+              })
+            : undefined;
     const WhiteboardSize = whiteboardSizeFields
         ? {
               AllocatedSize:
@@ -138,6 +142,7 @@ export function prepareGroupsPDisk(data: TStoragePDisk & {NodeId?: number} = {})
 
     return {
         ...mergedPDiskData,
+        HasWhiteboardData: hasWhiteboardData,
         StringifiedId,
         AllocatedPercent,
         AllocatedSize,

package/src/store/reducers/storage/utils.ts

@@ -76,14 +76,21 @@ const prepareStorageNodeData = (
         }).length || 0;
 
     const pDisks = PDisks?.map((pDisk) => {
-        return prepareWhiteboardPDiskData({
-            ...pDisk,
-            NodeId,
-        });
+        // Older viewers have no marker; identity and sizes can also come from BSC.
+        const hasWhiteboardData =
+            pDisk.HasWhiteboardData ??
+            (pDisk.State !== undefined ||
+                pDisk.StateFlag !== undefined ||
+                pDisk.Overall !== undefined ||
+                pDisk.CreateTime !== undefined ||
+                pDisk.ChangeTime !== undefined ||
+                pDisk.Device !== undefined ||
+                pDisk.Realtime !== undefined);
+        return prepareWhiteboardPDiskData({...pDisk, NodeId}, hasWhiteboardData ? pDisk : null);
     });
     const vDisks = VDisks?.map((vDisk) => {
         return {
-            ...prepareWhiteboardVDiskData(vDisk),
+            ...prepareWhiteboardVDiskData({...vDisk, VDiskId: vDisk.VDiskId}),
             NodeId,
         };
     });

package/src/store/reducers/table/table.ts

@@ -1,5 +1,6 @@
 import type {SchemaPathParam} from '../../../types/api/common';
 import type {TEvDescribeSchemeResult} from '../../../types/api/schema/schema';
+import {reachObjectCreationGoal} from '../../../utils/objectCreationMetrics';
 import {isQueryErrorResponse, parseQueryAPIResponse} from '../../../utils/query';
 import {api} from '../api';
 
@@ -42,6 +43,8 @@ export const tableApi = api.injectEndpoints({
                 database: string;
                 formValues: TableFormValues;
             }) => {
+                const objectType = formValues.type === 'row' ? 'row_table' : 'column_table';
+                reachObjectCreationGoal('createObject', objectType);
                 try {
                     const {
                         type,
@@ -79,12 +82,15 @@ export const tableApi = api.injectEndpoints({
                     });
 
                     if (isQueryErrorResponse(response)) {
+                        reachObjectCreationGoal('createObjectError', objectType, response);
                         return {error: response};
                     }
 
                     const data = parseQueryAPIResponse(response);
+                    reachObjectCreationGoal('createObjectSuccess', objectType);
                     return {data};
                 } catch (error) {
+                    reachObjectCreationGoal('createObjectError', objectType, error);
                     return {error};
                 }
             },

package/src/store/reducers/tableData.ts

@@ -16,7 +16,7 @@ interface PaginatedTableParams<T, F> {
     noBatching?: boolean;
 }
 
-function endpoints<T, F>(build: EndpointBuilder<BaseQueryFn, string, string>) {
+function endpoints<T, F>(build: EndpointBuilder<BaseQueryFn, string, typeof api.reducerPath>) {
     return {
         fetchTableChunk: build.query<PaginatedTableData<T>, PaginatedTableParams<T, F>>({
             queryFn: async (

package/src/store/reducers/tablet.ts

@@ -120,10 +120,13 @@ export const tabletApi = api.injectEndpoints({
             },
         }),
         getAdvancedTableInfo: build.query({
-            queryFn: async ({id, hiveId}: {id: string; hiveId: string}, {signal}) => {
+            queryFn: async (
+                {id, hiveId, useSecurePath}: {id: string; hiveId: string; useSecurePath?: boolean},
+                {signal},
+            ) => {
                 try {
                     const tabletResponseData = await window.api.tablets.getTabletFromHive(
-                        {id, hiveId},
+                        {id, hiveId, useSecurePath},
                         {signal},
                     );
 
@@ -153,9 +156,17 @@ export const tabletApi = api.injectEndpoints({
             },
         }),
         stopTablet: build.mutation({
-            queryFn: async ({id, hiveId}: {id: string; hiveId: string}) => {
+            queryFn: async ({
+                id,
+                hiveId,
+                useSecurePath,
+            }: {
+                id: string;
+                hiveId: string;
+                useSecurePath?: boolean;
+            }) => {
                 try {
-                    const data = await window.api.tablets.stopTablet(id, hiveId);
+                    const data = await window.api.tablets.stopTablet(id, hiveId, useSecurePath);
                     return {data};
                 } catch (error) {
                     return {error};
@@ -169,9 +180,17 @@ export const tabletApi = api.injectEndpoints({
             },
         }),
         resumeTablet: build.mutation({
-            queryFn: async ({id, hiveId}: {id: string; hiveId: string}) => {
+            queryFn: async ({
+                id,
+                hiveId,
+                useSecurePath,
+            }: {
+                id: string;
+                hiveId: string;
+                useSecurePath?: boolean;
+            }) => {
                 try {
-                    const data = await window.api.tablets.resumeTablet(id, hiveId);
+                    const data = await window.api.tablets.resumeTablet(id, hiveId, useSecurePath);
                     return {data};
                 } catch (error) {
                     return {error};

package/src/store/reducers/tenantOverview/topShards/tenantOverviewTopShards.ts

@@ -7,7 +7,7 @@ function createShardQuery(path: string, databaseFullPath: string) {
     return createPartitionStatsQuery({
         databaseFullPath,
         path,
-        selectFields: ['Path', 'TabletId', 'CPUCores'],
+        selectFields: ['Path', 'TabletId', 'CPUCores', 'FollowerId'],
         sortOrder: [{columnId: 'CPUCores', order: -1}],
         limit: TENANT_OVERVIEW_TABLES_LIMIT,
     });

package/src/store/reducers/topic/topic.ts

@@ -4,6 +4,7 @@ import {createSelector} from '@reduxjs/toolkit';
 import type {IProtobufTimeObject} from '../../../types/api/common';
 import type {TopicDataRequest} from '../../../types/api/topic';
 import {convertBytesObjectToSpeed} from '../../../utils/bytesParsers';
+import {reachObjectCreationGoal} from '../../../utils/objectCreationMetrics';
 import {isQueryErrorResponse, parseQueryAPIResponse} from '../../../utils/query';
 import {parseLag, parseTimestampToIdleTime} from '../../../utils/timeParsers';
 import type {RootState} from '../../defaultStore';
@@ -82,6 +83,7 @@ export const topicApi = api.injectEndpoints({
                 database: string;
                 formData: TopicFormValues;
             }) => {
+                reachObjectCreationGoal('createObject', 'topic');
                 try {
                     const query = buildCreateTopicQuery(formData);
 
@@ -92,11 +94,15 @@ export const topicApi = api.injectEndpoints({
                     });
 
                     if (isQueryErrorResponse(response)) {
+                        reachObjectCreationGoal('createObjectError', 'topic', response);
                         return {error: response};
                     }
 
-                    return {data: parseQueryAPIResponse(response)};
+                    const data = parseQueryAPIResponse(response);
+                    reachObjectCreationGoal('createObjectSuccess', 'topic');
+                    return {data};
                 } catch (error) {
+                    reachObjectCreationGoal('createObjectError', 'topic', error);
                     return {error};
                 }
             },

package/src/types/api/capabilities.ts

@@ -5,7 +5,7 @@ export interface CapabilitiesResponse {
     Capabilities: Record<Partial<Capability>, number>;
     Settings?: {
         Security?: Record<Partial<SecuritySetting>, boolean>;
-        Features?: Record<Partial<FeatureSetting>, boolean>;
+        Features?: Partial<Record<FeatureSetting, boolean>>;
         Database?: {
             GraphShardExists?: boolean;
         };
@@ -34,7 +34,7 @@ export type Capability =
 
 export type SecuritySetting = 'UseLoginProvider' | 'DomainLoginOnly';
 
-export type FeatureSetting = 'EnableAnalyzeLongRunningOperation';
+export type FeatureSetting = 'EnableAnalyzeLongRunningOperation' | 'EnableTabletDevUiSecurePath';
 
 export interface MetaCapabilitiesResponse {
     Capabilities: Record<Partial<MetaCapability>, number>;

package/src/types/api/pdisk.ts

@@ -17,6 +17,8 @@ export interface TEvPDiskStateResponse {
  * source: https://github.com/ydb-platform/ydb/blob/main/ydb/core/protos/node_whiteboard.proto
  */
 export interface TPDiskStateInfo {
+    /** Whether viewer received Whiteboard data for this disk. */
+    HasWhiteboardData?: boolean;
     PDiskId?: number;
     /** uint64 */
     CreateTime?: string;

package/src/types/api/schema/persQueueGroup.ts

@@ -8,7 +8,7 @@ export interface TPersQueueGroupDescription {
     PartitionsToDelete?: number[];
     NextPartitionId?: number;
     PartitionPerTablet?: number;
-    PQTabletConfig: TPQTabletConfig;
+    PQTabletConfig?: TPQTabletConfig;
     Partitions?: TPartition[];
     /** uint64 */
     AlterVersion?: string;
@@ -23,7 +23,7 @@ export interface TPersQueueGroupDescription {
 export interface TPQTabletConfig {
     /** uint64 */
     CacheSize?: string;
-    PartitionConfig: TPQPartitionConfig;
+    PartitionConfig?: TPQPartitionConfig;
     /** @deprecated use Partitions */
     PartitionIds?: number[];
     TopicName?: string;

package/src/types/api/schema/schema.ts

@@ -6,6 +6,7 @@ import type {TExternalDataSourceDescription} from './externalDataSource';
 import type {TExternalTableDescription} from './externalTable';
 import type {TPersQueueGroupDescription} from './persQueueGroup';
 import type {TReplicationDescription} from './replication';
+import type {TPathID} from './shared';
 import type {TSysViewDescription} from './sysView';
 import type {TTableDescription, TTableStats} from './table';
 import type {TIndexDescription} from './tableIndex';
@@ -90,6 +91,9 @@ export interface TPathDescription {
     SysViewDescription?: TSysViewDescription;
 
     ReplicationDescription?: TReplicationDescription;
+
+    // for resource pool
+    ResourcePoolDescription?: TResourcePoolDescription;
 }
 
 export interface TDirEntry {
@@ -349,3 +353,17 @@ interface TUserAttribute {
     Key?: string;
     Value?: string;
 }
+
+/**
+ * source: https://github.com/ydb-platform/ydb/blob/main/ydb/core/protos/flat_scheme_op.proto
+ *
+ * TResourcePoolDescription from flat_scheme_op.proto
+ */
+export interface TResourcePoolDescription {
+    Name?: string;
+    PathId?: TPathID;
+    Version?: string;
+    Properties?: {
+        Properties?: Record<string, string>;
+    };
+}

package/src/types/api/schema/tableIndex.ts

@@ -82,6 +82,7 @@ export interface TFulltextIndexAnalyzers {
     filter_length_min?: number;
     filter_length_max?: number;
     use_filter_snowball?: boolean;
+    use_filter_superlemmer?: boolean;
 }
 
 export interface TFulltextIndexColumnAnalyzers {

package/src/types/api/storage.ts

@@ -106,6 +106,8 @@ export interface TGroupsStorageGroupInfo {
  * VDisk data from storage/groups handler
  */
 export interface TStorageVDisk {
+    /** Whether viewer received Whiteboard data for this disk. */
+    HasWhiteboardData?: boolean;
     VDiskId?: string;
     NodeId?: number;
     /** uint64 */
@@ -123,6 +125,8 @@ export interface TStorageVDisk {
  * PDisk data from storage/groups handler
  */
 export interface TStoragePDisk {
+    /** Whether viewer received Whiteboard data for this disk. */
+    HasWhiteboardData?: boolean;
     PDiskId?: string;
     Path?: string;
     Type?: 'hdd' | 'ssd' | 'nvme';

package/src/types/api/vdisk.ts

@@ -6,6 +6,8 @@ import type {TPDiskStateInfo} from './pdisk';
  * source: https://github.com/ydb-platform/ydb/blob/main/ydb/core/protos/node_whiteboard.proto
  */
 export interface TVDiskStateInfo {
+    /** Whether viewer received Whiteboard data for this disk. */
+    HasWhiteboardData?: boolean;
     VDiskId?: TVDiskID;
     /** uint64 */
     CreateTime?: string;

package/src/uiFactory/types.ts

@@ -216,6 +216,12 @@ export type RenderHealthcheckAssistantAction = (
 ) => React.ReactNode;
 
 export interface UiMetricaGoals {
+    /** Creation attempt after form validation. Params: objectType (row_table, column_table, topic). */
+    createObject?: string;
+    /** Successful creation response. Params: objectType. */
+    createObjectSuccess?: string;
+    /** Failed creation. Params: objectType, errorType (ydb, http, network, timeout, cancelled, unknown), httpStatus (HTTP errors only). */
+    createObjectError?: string;
     runQuery?: string;
     explainAnalyzeQuery?: string;
     stopQuery?: string;

package/src/utils/developerUI/tabletDevUi.ts

@@ -0,0 +1,3 @@
+export function getTabletDevUiAppPage(useSecurePath = false) {
+    return useSecurePath ? 'app/secure' : 'app';
+}

package/src/utils/disks/displayState.ts

@@ -2,13 +2,14 @@ import type {IconData} from '@gravity-ui/uikit';
 
 import {DATA_SEVERITY, DISK_COLOR_STATE_TO_NUMERIC_SEVERITY} from './constants';
 import type {IconWithColor} from './iconCalculators';
-import type {DisplaySeverity, PreparedPDisk, PreparedVDisk} from './types';
+import type {DisplaySeverity, PDiskType, PreparedPDisk, PreparedVDisk} from './types';
 
 export type DiskDisplayMode =
     | 'state'
     | 'space'
     | 'frontQueues'
     | 'compaction'
+    | 'driveType'
     | 'all'
     | 'drive'
     | 'decommit'
@@ -42,11 +43,14 @@ export interface PDiskAllModeDisplayState {
 }
 
 export interface BaseDiskDisplayState {
+    driveType?: PDiskType;
     severity: DisplaySeverity;
     icon: DiskIndicatorValue | undefined;
     mode: DiskDisplayMode | undefined;
     isLegendInactive?: boolean;
+    borderless?: boolean;
     showNoDataPlaceholder?: boolean;
+    isNoData?: boolean;
 }
 
 export interface VDiskDisplayState extends BaseDiskDisplayState {

package/src/utils/disks/driveType.ts

@@ -0,0 +1,30 @@
+import {CircleQuestionFill} from '@gravity-ui/icons';
+
+import {NOT_AVAILABLE_SEVERITY} from './constants';
+import type {BaseDiskDisplayState} from './displayState';
+import {formatPDiskType} from './getPDiskType';
+import type {PDiskType, PreparedPDisk} from './types';
+
+export function getDriveTypeDisplayState(pDisk: PreparedPDisk = {}): BaseDiskDisplayState {
+    const hasWhiteboardData = pDisk.HasWhiteboardData ?? pDisk.WhiteboardSize !== undefined;
+    const isNoData = !hasWhiteboardData;
+    let driveType: PDiskType | undefined;
+    if (hasWhiteboardData) {
+        switch (pDisk.Type) {
+            case 'SSD':
+            case 'HDD':
+            case 'NVME':
+                driveType = pDisk.Type;
+        }
+    }
+
+    return {
+        mode: 'driveType',
+        driveType,
+        severity: NOT_AVAILABLE_SEVERITY,
+        icon: isNoData ? undefined : (formatPDiskType(driveType) ?? CircleQuestionFill),
+        isNoData,
+        borderless: true,
+        showNoDataPlaceholder: isNoData,
+    };
+}

package/src/utils/disks/getDiskBarTone.ts

@@ -1,15 +1,19 @@
 import {CircleQuestionFill} from '@gravity-ui/icons';
 
+import {EFlag} from '../../types/api/enums';
+
 import {DONOR_COLOR, NOT_AVAILABLE_SEVERITY} from './constants';
 import type {DiskIndicatorValue} from './displayState';
 import {getDisplaySeverityColor} from './helpers';
-import type {DiskBarTone} from './types';
+import type {DiskBarTone, PDiskType} from './types';
 
 interface GetDiskBarToneParams {
+    driveType?: PDiskType;
     severity?: number;
     isDonor?: boolean;
     showIndicator?: boolean;
     indicator?: DiskIndicatorValue;
+    isNoData?: boolean;
 }
 
 function hasMissingDataIndicator(indicator: DiskIndicatorValue | undefined) {
@@ -21,15 +25,25 @@ function hasMissingDataIndicator(indicator: DiskIndicatorValue | undefined) {
 }
 
 export function getDiskBarTone({
+    driveType,
     severity,
     isDonor,
     showIndicator,
     indicator,
+    isNoData,
 }: GetDiskBarToneParams): DiskBarTone {
+    if (driveType) {
+        return driveType;
+    }
+
     if (isDonor) {
         return DONOR_COLOR;
     }
 
+    if (isNoData) {
+        return EFlag.Grey;
+    }
+
     if (
         showIndicator &&
         severity === NOT_AVAILABLE_SEVERITY &&

package/src/utils/disks/getPDiskType.ts

@@ -32,6 +32,10 @@ export const PDISK_TYPES = {
     MVME: 'NVME',
 } as const;
 
+export function formatPDiskType(type?: string) {
+    return type === 'NVME' ? 'NVMe' : type;
+}
+
 // Bear with me.
 // Disk type is determined by the field Category.
 // Category is a bit field defined as follows:

package/src/utils/disks/groupBy.ts

@@ -3,6 +3,7 @@ export const VDisksGroupBy = {
     Space: 'Space',
     FrontQueues: 'FrontQueues',
     Compaction: 'Compaction',
+    DriveType: 'DriveType',
     All: 'All',
 } as const;
 

package/src/utils/disks/iconCalculators.ts

@@ -165,7 +165,7 @@ function getFlagIconWithColor(flag: EFlag | undefined): IconWithColor | undefine
     if (!flag || flag === EFlag.Grey) {
         return {
             icon: CircleQuestionFill,
-            color: 'rgba(162, 162, 162, 1)',
+            color: 'var(--g-color-text-hint)',
         };
     }
 
@@ -230,7 +230,7 @@ export function calculateFlagPairIcon(
  * - Yellow: TriangleExclamationFill (text-warning)
  * - Orange: CircleExclamationFill (text-danger)
  * - Red: CircleXmarkFill (text-primary)
- * - Missing: CircleQuestionFill (rgba(162, 162, 162, 1))
+ * - Missing: CircleQuestionFill (text-hint)
  */
 export function calculateCompactionIcon(
     vDisk: PreparedVDisk,

package/src/utils/disks/pdiskState.ts

@@ -1,7 +1,6 @@
 import {
     ArrowUpFromLine,
     Ban,
-    CircleExclamation,
     CircleExclamationFill,
     CircleQuestionFill,
     CircleStopFill,
@@ -11,6 +10,7 @@ import {
     ClockFill,
     HourglassStart,
     TrashBin,
+    TriangleExclamation,
     Wrench,
     Xmark,
 } from '@gravity-ui/icons';
@@ -128,7 +128,7 @@ const PDISK_DRIVE_DISPLAY_STATE: Record<EDriveStatus, PDiskStateDisplayState> =
     },
     FAULTY: {
         severity: DISK_COLOR_STATE_TO_NUMERIC_SEVERITY.Red,
-        icon: CircleExclamation,
+        icon: TriangleExclamation,
     },
     BROKEN: {
         severity: DISK_COLOR_STATE_TO_NUMERIC_SEVERITY.Red,