Skip to content

Add option to exclude system workers in ListWorkers API#777

Merged
rkannan82 merged 4 commits intomasterfrom
kannan/list-workers-exclude-system
Apr 30, 2026
Merged

Add option to exclude system workers in ListWorkers API#777
rkannan82 merged 4 commits intomasterfrom
kannan/list-workers-exclude-system

Conversation

@rkannan82
Copy link
Copy Markdown
Contributor

@rkannan82 rkannan82 commented Apr 30, 2026

What

Add bool include_system_workers field to ListWorkersRequest proto. When true, the response includes system workers that are not created by the user. By default, system workers are excluded.

Why

System workers (per-namespace workers for scheduler, batcher, etc.) show up in ListWorkers results, which is confusing for users who only care about their own workers.

How did you test it?

Build

🤖 Generated with Claude Code

rkannan82 and others added 2 commits April 29, 2026 15:53
Allows callers to opt in to seeing system workers (on temporal-sys-* task
queues) in ListWorkers responses. By default they are excluded.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rkannan82 rkannan82 changed the title Add include_system_workers to ListWorkersRequest Add option to exclude system workers in ListWorkers API Apr 30, 2026
@rkannan82 rkannan82 marked this pull request as ready for review April 30, 2026 05:13
@rkannan82 rkannan82 requested review from a team April 30, 2026 05:13
@rkannan82 rkannan82 enabled auto-merge (squash) April 30, 2026 17:18
@rkannan82 rkannan82 disabled auto-merge April 30, 2026 17:21
@rkannan82 rkannan82 requested a review from ShahabT April 30, 2026 20:05
@rkannan82 rkannan82 merged commit 8e0453c into master Apr 30, 2026
4 checks passed
@rkannan82 rkannan82 deleted the kannan/list-workers-exclude-system branch April 30, 2026 20:31
rkannan82 added a commit to temporalio/temporal that referenced this pull request May 2, 2026
## What
Exclude internal system workers from ListWorkers API results by default.
Users can opt in to seeing them via `include_system_workers` on the
request.

Depends on API change: temporalio/api#777

## Why
System workers (per-namespace workers for scheduler, batcher, worker
deployment, etc.) appear in ListWorkers results, which is confusing for
users who only care about their own workers.

## How did you test it?
Unit tests.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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.

3 participants