feat: serverless database view #2836

astandrik · 2025-09-04T13:18:27Z

CI Results

Test Status: ⚠️ FLAKY

Total	Passed	Failed	Flaky	Skipped
378	373	0	3	2

Test Changes Summary ⏭️2

⏭️ Skipped Tests (2)

Scroll to row, get shareable link, navigate to URL and verify row is scrolled into view (tenant/diagnostics/tabs/queries.test.ts)
Copy result button copies to clipboard (tenant/queryEditor/queryEditor.test.ts)

Bundle Size: 🔺

Current: 85.41 MB | Main: 85.39 MB
Diff: +0.02 MB (0.02%)

⚠️ Bundle size increased. Please review.

ℹ️ CI Information

Test recordings for failed tests are available in the full report.
Bundle size is measured for the entire 'dist' directory.
📊 indicates links to detailed reports.
🔺 indicates increase, 🔽 decrease, and ✅ no change in bundle size.

Raubzeug · 2025-09-05T06:59:44Z

src/containers/Tenant/Diagnostics/TenantOverview/MetricsTabs/ServerlessPlaceholderTabs.scss

@@ -0,0 +1,9 @@
+.tenant-metrics-tabs_serverless {


bem notation doesn't allow modifiers as top level classes. Let's move this to MetricsTabs.scss

Raubzeug · 2025-09-05T07:19:41Z

src/containers/Tenant/Diagnostics/TenantOverview/TabCard/TabCard.tsx

+                    type="container"
+                    view={active ? 'outlined' : 'raised'}
+                >
+                    <div className={b('legend-wrapper')}>


It seems only card content varies depend on variant, while wrapper is same. Lets make renderContent function?

Raubzeug · 2025-09-05T07:20:16Z

src/containers/Tenant/Diagnostics/TenantOverview/TenantOverview.scss


+    &__serverless-tag {
+        margin-left: var(--g-spacing-2);
+        padding: 2px var(--g-spacing-2);


2px = var(--g-spacing-half)

now using Label component

Raubzeug · 2025-09-05T07:23:41Z

src/containers/Tenant/Diagnostics/TenantOverview/i18n/en.json

  "context_capacity-usage": "{{value}} of {{capacity}}",
  "network-stats-unavailable.title": "Network Statistics Unavailable",
-  "network-stats-unavailable.description": "Network ping and clock skew statistics require a newer backend version. Please upgrade your YDB installation to access these features."
+  "network-stats-unavailable.description": "Network ping and clock skew statistics require a newer backend version. Please upgrade your YDB installation to access these features.",


For keysets naming let's stick to https://nda.ya.ru/t/4A_EEYWW7JXQco

astandrik · 2025-09-05T11:06:52Z

bugbot run
/claude_review

cursor

✅ Bugbot reviewed your changes and found no bugs!

Copilot

Pull Request Overview

This PR implements a dedicated serverless database view that provides a simplified interface for serverless databases by hiding resource metrics that don't apply to auto-scaling environments and adding visual indicators for serverless mode.

Adds serverless-specific UI components and layouts for tenant overview pages
Filters available diagnostic tabs for serverless databases to remove inapplicable sections
Implements visual serverless tags and placeholder tabs for hidden metrics

Reviewed Changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
`src/utils/constants.ts`	Adds non-breaking space constant for UI formatting
`src/store/reducers/tenant/tenant.ts`	Extends tenant base info hook to include database type information
`src/containers/Tenant/Diagnostics/i18n/`	Creates new i18n module for diagnostics with "No data" translation
`src/containers/Tenant/Diagnostics/TenantOverview/i18n/en.json`	Adds serverless-specific translation keys and restructures network stats messages
`src/containers/Tenant/Diagnostics/TenantOverview/TenantStorage/TenantStorage.tsx`	Implements simplified storage view for serverless databases
`src/containers/Tenant/Diagnostics/TenantOverview/TenantOverview.tsx`	Adds serverless tag UI and adjusts active tab logic for serverless mode
`src/containers/Tenant/Diagnostics/TenantOverview/TenantOverview.scss`	Adds styling for serverless tag component
`src/containers/Tenant/Diagnostics/TenantOverview/TenantCpu/TenantCpu.tsx`	Conditionally hides resource charts for serverless databases
`src/containers/Tenant/Diagnostics/TenantOverview/TabCard/TabCard.tsx`	Refactors tab cards to support serverless display mode
`src/containers/Tenant/Diagnostics/TenantOverview/MetricsTabs/`	Creates modular tab components for different database types
`src/containers/Tenant/Diagnostics/DiagnosticsPages.ts`	Updates page filtering logic to handle serverless database constraints
`src/containers/Tenant/Diagnostics/Diagnostics.tsx`	Integrates serverless database type into diagnostics page rendering

src/containers/Tenant/Diagnostics/TenantOverview/MetricsTabs/CommonMetricsTabs.tsx

astandrik · 2025-09-05T13:15:23Z

fixed
Stand redeployed

feat: serverless database view

ca9986a

astandrik requested a review from Copilot September 4, 2025 13:18

This comment was marked as outdated.

Sign in to view

fix: some review fixes

5947cad

astandrik requested a review from Copilot September 4, 2025 14:02

This comment was marked as outdated.

Sign in to view

astandrik added 2 commits September 4, 2025 18:24

fix: review fixes

f0b8ec9

fix: code

a336c9d

ydb-platform deleted a comment from claude bot Sep 4, 2025

astandrik added 2 commits September 4, 2025 19:44

fix: better code

9c92d12

fix: better code

65574d0

astandrik requested a review from Copilot September 4, 2025 17:13

This comment was marked as outdated.

Sign in to view

astandrik marked this pull request as ready for review September 4, 2025 17:19

astandrik requested review from Raubzeug, adameat and artemmufazalov as code owners September 4, 2025 17:19

Raubzeug requested changes Sep 5, 2025

View reviewed changes

astandrik added 4 commits September 5, 2025 11:27

fix: label

e78f7ec

fix: nanofix

610df88

fix: better code

cfb0d28

fix: code

044b80e

astandrik requested a review from Copilot September 5, 2025 10:18

This comment was marked as outdated.

Sign in to view

fix: storage groups

d75bd92

astandrik requested review from Copilot September 5, 2025 11:06

This comment was marked as outdated.

Sign in to view

cursor bot reviewed Sep 5, 2025

View reviewed changes

ydb-platform deleted a comment from claude bot Sep 5, 2025

astandrik added 2 commits September 5, 2025 14:22

fix: title

bea42b4

fix: refactoring

fc7df32

astandrik requested a review from Copilot September 5, 2025 13:10

Copilot AI reviewed Sep 5, 2025

View reviewed changes

src/containers/Tenant/Diagnostics/TenantOverview/MetricsTabs/CommonMetricsTabs.tsx Show resolved Hide resolved

astandrik requested a review from Raubzeug September 5, 2025 13:15

Raubzeug approved these changes Sep 5, 2025

View reviewed changes

astandrik added this pull request to the merge queue Sep 5, 2025

Merged via the queue into main with commit 5654ed0 Sep 5, 2025
7 checks passed

astandrik deleted the astandrik.2753 branch September 5, 2025 15:00

ydb-platform-bot mentioned this pull request Sep 5, 2025

chore(main): release 12.0.0 #2822

Merged

feat: serverless database view #2836

feat: serverless database view #2836

Uh oh!

Conversation

astandrik commented Sep 4, 2025 • edited by github-actions bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

CI Results

Test Status: ⚠️ FLAKY

⏭️ Skipped Tests (2)

Bundle Size: 🔺

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

Raubzeug Sep 5, 2025

Choose a reason for hiding this comment

Uh oh!

Raubzeug Sep 5, 2025

Choose a reason for hiding this comment

Uh oh!

Raubzeug Sep 5, 2025

Choose a reason for hiding this comment

Uh oh!

astandrik Sep 5, 2025

Choose a reason for hiding this comment

Uh oh!

Raubzeug Sep 5, 2025

Choose a reason for hiding this comment

Uh oh!

astandrik Sep 5, 2025

Choose a reason for hiding this comment

Uh oh!

This comment was marked as outdated.

Uh oh!

astandrik commented Sep 5, 2025

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

cursor bot left a comment

Choose a reason for hiding this comment

✅ Bugbot reviewed your changes and found no bugs!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

astandrik commented Sep 5, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

astandrik commented Sep 4, 2025 •

edited by github-actions bot

Loading