Skip to content

fix: disable a few more new clickhouse tables#4269

Merged
aldy505 merged 1 commit intogetsentry:masterfrom
alkanna:fix/disable_some_clickhouse_log_tables
Apr 9, 2026
Merged

fix: disable a few more new clickhouse tables#4269
aldy505 merged 1 commit intogetsentry:masterfrom
alkanna:fix/disable_some_clickhouse_log_tables

Conversation

@alkanna
Copy link
Copy Markdown
Contributor

@alkanna alkanna commented Apr 8, 2026

Closes #4266

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

Copilot AI review requested due to automatic review settings April 8, 2026 07:06
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 8, 2026

Changelog Preview

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • Run SeaweedFS admin and worker instance by aldy505 in #4259
  • Support custom CA certificates for all containers by aldy505 in #4216
  • Remove 'vroom-cleanup' container by aldy505 in #4217

Bug Fixes 🐛

  • (install) Gracefully handle missing containers in minimize-downtime by shameemkpofficial-git in #4246
  • Disable a few more new clickhouse tables by alkanna in #4269
  • Use default buildx builder to resolve local images during build by maiqigh in #4250

Internal Changes 🔧

Deps

  • Bump j178/prek-action from 2.0.0 to 2.0.1 by dependabot in #4264
  • Bump BYK/docker-volume-cache-action from be89365902126f508dcae387a32ec3712df6b1cd to 0efa5cf5178c9906cb46ed8d1a357df8fd6b1a06 by dependabot in #4253
  • Bump astral-sh/setup-uv from 7.6.0 to 8.0.0 by dependabot in #4254
  • Bump getsentry/craft from 2.23.2 to 2.24.1 by dependabot in #4221
  • Bump astral-sh/setup-uv from 7.2.1 to 7.5.0 by dependabot in #4220

Other

  • Restore unpinned actions by aldy505 in #4243
  • Swap pre-commit with prek by aldy505 in #4235

Other

  • Bump postgres 14.22-bookworm by aminvakil in #4249

🤖 This preview updates automatically when you update the PR.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the self-hosted ClickHouse server configuration to explicitly disable additional newly introduced system log tables that can generate unnecessary CPU overhead in long-running installations.

Changes:

  • Disables query_views_log, processors_profile_log, error_log, and latency_log via remove="remove" directives in the ClickHouse config include.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@aldy505 aldy505 requested a review from aminvakil April 8, 2026 07:28
Copy link
Copy Markdown
Collaborator

@aldy505 aldy505 left a comment

Choose a reason for hiding this comment

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

why not

@aldy505 aldy505 merged commit 7f066c4 into getsentry:master Apr 9, 2026
16 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Clickhouse config seems to be missing some remove directives

3 participants