feat(migrations): add webhook deliveries table to ClickHouse - #2741
Conversation
There was a problem hiding this comment.
Code Review
The migration is missing the ON CLUSTER 'cluster' clause for DDL statements, which is necessary for schema synchronization across shards. The sandbox_id column should use the UUID type instead of String for improved storage efficiency and performance.
❌ 7 Tests Failed:
View the full list of 7 ❄️ flaky test(s)
To view more test analytics, go to the Test Analytics Dashboard |
sitole
left a comment
There was a problem hiding this comment.
just nit with with field name, lgtm
…response_http_status_code Updated the schema for the webhook_deliveries_local table by reordering the request_url and request_headers columns, and added response_http_status_code to enhance data tracking capabilities.
🤖 I have created a release *beep* *boop* --- ## 1.0.0 (2026-07-07) ### Features * Adding client-proxy and clickhouse to e2b-artifacts ([#3210](#3210)) ([5686d88](5686d88)) * **api:** LD-gated ClickHouse read switcher ([#3061](#3061)) ([29e74ca](29e74ca)) * **clickhouse:** implement multi-cluster fan-out for events and stats ([#2925](#2925)) ([39594c6](39594c6)) * **migrations:** add webhook deliveries table to ClickHouse ([#2741](#2741)) ([f55a5bd](f55a5bd)) * **orchestrator:** LD-gated ClickHouse write fan-out feature flag ([#3152](#3152)) ([f046fcf](f046fcf)) * **orch:** re-enable memory.peak per-FD reset (cgroups v2) ([#2430](#2430)) ([5d9ac5f](5d9ac5f)) * per-team events TTL limit (tier + addons) ([#3181](#3181)) ([f76b2cb](f76b2cb)) ### Bug Fixes * push clickhouse-migrator image to both latest and commit SHA tags ([#2954](#2954)) ([3b780d5](3b780d5)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: e2b-release-please[bot] <298072688+e2b-release-please[bot]@users.noreply.github.com>
🤖 I have created a release *beep* *boop* --- ## 0.0.1 (2026-07-10) ### Features * Adding client-proxy and clickhouse to e2b-artifacts ([#3210](#3210)) ([5686d88](5686d88)) * **api:** LD-gated ClickHouse read switcher ([#3061](#3061)) ([29e74ca](29e74ca)) * **clickhouse:** implement multi-cluster fan-out for events and stats ([#2925](#2925)) ([39594c6](39594c6)) * **migrations:** add webhook deliveries table to ClickHouse ([#2741](#2741)) ([f55a5bd](f55a5bd)) * **orchestrator:** LD-gated ClickHouse write fan-out feature flag ([#3152](#3152)) ([f046fcf](f046fcf)) * per-team events TTL limit (tier + addons) ([#3181](#3181)) ([f76b2cb](f76b2cb)) ### Bug Fixes * correct 3 CVES ([#3218](#3218)) ([076823b](076823b)) * push clickhouse-migrator image to both latest and commit SHA tags ([#2954](#2954)) ([3b780d5](3b780d5)) * reset artifacts ([#3259](#3259)) ([93f7eb5](93f7eb5)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: e2b-release-please[bot] <298072688+e2b-release-please[bot]@users.noreply.github.com> Co-authored-by: Charlie Wyse <charlie.wyse@e2b.dev>
Add ClickHouse storage for webhook delivery attempts so delivery status, timing, request/response metadata, and errors can be queried for debugging and observability.
Uses the existing local + distributed table pattern with team-based sharding and 7-day TTL retention.