Skip to content

ref(taskworker): Mark raw-mode namespaces explicitly - #121297

Merged
untitaker merged 6 commits into
masterfrom
markusunterwaditzer/stream-1044-raw-mode-flag
Aug 7, 2026
Merged

ref(taskworker): Mark raw-mode namespaces explicitly#121297
untitaker merged 6 commits into
masterfrom
markusunterwaditzer/stream-1044-raw-mode-flag

Conversation

@untitaker

@untitaker untitaker commented Aug 5, 2026

Copy link
Copy Markdown
Member

blocked on a taskbroker-client release containing is_raw_mode (getsentry/taskbroker#777).

Sets is_raw_mode=True on the namespaces backing topics that taskbroker consumes in raw mode, and drops the task options taskbroker ignores for them.

ref STREAM-1044

🤖 Generated with Claude Code

@linear-code

linear-code Bot commented Aug 5, 2026

Copy link
Copy Markdown

STREAM-1044

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Aug 5, 2026
untitaker and others added 2 commits August 6, 2026 17:18
Set is_raw_mode on the namespaces backing topics that taskbroker consumes
in raw mode, and drop the task options that taskbroker ignores for them.

Raw-mode activations are built by taskbroker from the topic's `raw:` block
in its own config, not by a producer, so processing_deadline_duration and
compression_type on the decorator never reach the broker. They already
disagree with what is deployed: process_profile_from_kafka_raw declares
80s while every region runs the topic at 60s, and 60 is what applies.

Removing them is a no-op at runtime and makes the deployed config the only
place the values live. taskbroker-client now rejects them at import for
raw namespaces, so they cannot drift back in.

ingest.profiling.passthrough is left alone; infra has already migrated off
it and STREAM-1191 removes it.

Blocked on a taskbroker-client release containing is_raw_mode.

ref STREAM-1044

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@untitaker
untitaker force-pushed the markusunterwaditzer/stream-1044-raw-mode-flag branch from 243ffb7 to c160b82 Compare August 6, 2026 15:19
@untitaker
untitaker marked this pull request as ready for review August 6, 2026 15:19
@untitaker
untitaker requested review from a team as code owners August 6, 2026 15:19
@sentry

sentry Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Sentry Snapshot Testing

Name Added Removed Changed Renamed Unchanged Skipped Status
sentry-frontend
sentry-frontend
0 0 0 0 451 0 ✅ Unchanged

⚙️ sentry-frontend Snapshot Settings

@untitaker
untitaker enabled auto-merge (squash) August 7, 2026 18:06
@untitaker
untitaker merged commit e24a18a into master Aug 7, 2026
66 checks passed
@untitaker
untitaker deleted the markusunterwaditzer/stream-1044-raw-mode-flag branch August 7, 2026 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants