Skip to content

feat(operator): static membership - #346

Open
bmcquilkin-sentry wants to merge 1 commit into
bmcquilkin/runtime/control-serverfrom
bmcquilkin/operator/membership
Open

feat(operator): static membership#346
bmcquilkin-sentry wants to merge 1 commit into
bmcquilkin/runtime/control-serverfrom
bmcquilkin/operator/membership

Conversation

@bmcquilkin-sentry

@bmcquilkin-sentry bmcquilkin-sentry commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Adds static membership to consumers (and also adds some type fix mypy was complaining about).

@bmcquilkin-sentry
bmcquilkin-sentry requested a review from a team as a code owner July 22, 2026 00:01

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit cfec667. Configure here.

Comment thread sentry_streams/tests/adapters/arroyo/test_rust_arroyo.py Outdated
@linear-code

linear-code Bot commented Jul 22, 2026

Copy link
Copy Markdown

STREAM-1644

@fpacifici fpacifici left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Pelase address my comments on the usage of hte environment variable.

Comment thread sentry_streams/sentry_streams/adapters/arroyo/rust_arroyo.py Outdated
Comment thread sentry_streams/tests/adapters/arroyo/test_rust_arroyo.py Outdated
Comment thread sentry_streams/tests/adapters/arroyo/test_rust_arroyo.py Outdated
@fpacifici
fpacifici self-requested a review July 23, 2026 23:46

@fpacifici fpacifici left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Clicked the wrong button, please see my comments above

@bmcquilkin-sentry
bmcquilkin-sentry force-pushed the bmcquilkin/operator/membership branch 2 times, most recently from 5f6d450 to fdd90be Compare July 30, 2026 22:53
@bmcquilkin-sentry
bmcquilkin-sentry changed the base branch from bmcquilkin/operator/pods to bmcquilkin/runtime/control-server July 30, 2026 22:53
@bmcquilkin-sentry
bmcquilkin-sentry force-pushed the bmcquilkin/runtime/control-server branch from a7b4cda to 4937081 Compare July 31, 2026 20:12
@bmcquilkin-sentry
bmcquilkin-sentry force-pushed the bmcquilkin/operator/membership branch from fdd90be to 5d66f0e Compare July 31, 2026 20:12
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