Skip to content

ref(k8s): build operator and push to gar#342

Merged
bmcquilkin-sentry merged 4 commits into
mainfrom
bmcquilkin/operator/build
Jul 16, 2026
Merged

ref(k8s): build operator and push to gar#342
bmcquilkin-sentry merged 4 commits into
mainfrom
bmcquilkin/operator/build

Conversation

@bmcquilkin-sentry

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

Copy link
Copy Markdown
Contributor

Build the operator image and push to the Sentry GAR. Modeled after sentry-kafka-management.

@bmcquilkin-sentry
bmcquilkin-sentry requested a review from a team as a code owner July 7, 2026 16:47
Comment thread .github/workflows/build-operator.yaml
Comment thread .github/workflows/build-operator.yaml Outdated
Comment thread sentry_streams_k8s/Dockerfile Outdated
Comment thread .github/workflows/build-operator.yaml
Comment thread sentry_streams_k8s/Dockerfile Outdated
@linear-code

linear-code Bot commented Jul 7, 2026

Copy link
Copy Markdown

STREAM-1253

Comment on lines +23 to +26
- registry: single-region
image: us-central1-docker.pkg.dev/sentryio/streaming-operator/image
- registry: multi-region
image: us-docker.pkg.dev/sentryio/streaming-operator-mr/image

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think we need both. Only publishing to multi-region is good enough. Some applications do both as we had to move from single -> multi region.

@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.

Please address the comment in line before merging

Comment thread sentry_streams_k8s/Dockerfile Outdated
Comment thread .github/workflows/build-operator.yaml
Comment thread .github/workflows/build-operator.yaml Outdated
Add a Kopf-based operator that reconciles StreamingConsumer CRs into
Deployments, using the shared consumer_builder logic.
Comment thread sentry_streams_k8s/tests/test_streaming_consumer.py Outdated
Comment thread sentry_streams_k8s/sentry_streams_k8s/operator/streaming_consumer.py Outdated
Comment thread sentry_streams_k8s/sentry_streams_k8s/operator/streaming_consumer.py Outdated
Comment thread sentry_streams_k8s/tests/test_streaming_consumer.py Outdated
@bmcquilkin-sentry
bmcquilkin-sentry force-pushed the bmcquilkin/operator/build branch from 62087ab to 4962ad2 Compare July 13, 2026 14:34

@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 4962ad2. Configure here.

Comment thread .github/workflows/build-operator.yaml
@bmcquilkin-sentry
bmcquilkin-sentry changed the base branch from bmcquilkin/operator/v1 to main July 16, 2026 14:09
@bmcquilkin-sentry
bmcquilkin-sentry merged commit 0569e7d into main Jul 16, 2026
19 checks passed
@bmcquilkin-sentry
bmcquilkin-sentry deleted the bmcquilkin/operator/build branch July 16, 2026 14: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.

4 participants