Skip to content

Add total Worker count#3641

Draft
laurakwhit wants to merge 2 commits into
mainfrom
DT-3912-worker-count
Draft

Add total Worker count#3641
laurakwhit wants to merge 2 commits into
mainfrom
DT-3912-worker-count

Conversation

@laurakwhit

@laurakwhit laurakwhit commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Description & motivation 💭

Polls new CountWorkers API (GET /namespaces/{namespace}/worker-count) to display worker count next to refresh button on /workers and /workers/deployments pages, as well as get the total count to show in the workers table.

Screenshots (if applicable) 📸

Design Considerations 🎨

Testing 🧪

How was this tested 👻

  • Manual testing
  • E2E tests added
  • Unit tests added

Steps for others to test: 🚶🏽‍♂️🚶🏽‍♀️

Checklists

Draft Checklist

  • Verify server version with support and update value in workerCountEnabled
  • Verify server version is deployed in Cloud

Merge Checklist

Issue(s) closed

DT-3912

Docs

Any docs updates needed?

@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
holocene Ready Ready Preview, Comment Jul 8, 2026 12:32am

Request Review

Comment thread src/lib/stores/workers.ts
export const workerCountEnabled = derived(
[isCloud, temporalVersion],
([$isCloud, $temporalVersion]) =>
$isCloud || minimumVersionRequired('1.31.2', $temporalVersion),

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note: Verify which version includes temporalio/temporal#9476.

request = fetch,
): Promise<{ count: number }> => {
let count = 0;
try {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • ⚠️ No overload matches this call.

@temporal-cicd

temporal-cicd Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor
Warnings
⚠️

📊 Strict Mode: 5 errors in 2 files (0.6% of 859 total)

src/lib/services/worker-service.ts (3)
  • L24:30: No overload matches this call.
  • L52:30: No overload matches this call.
  • L71:28: No overload matches this call.
src/lib/holocene/table/paginated-table/api-paginated.svelte (2)
  • L101:20: Element implicitly has an 'any' type because expression of type 'string' can't be used to index type '{ items: T[]; nextPageToken: string; }'.
  • L117:20: Element implicitly has an 'any' type because expression of type 'string' can't be used to index type '{ items: T[]; nextPageToken: string; }'.

Generated by 🚫 dangerJS against 61f2433

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant