Skip to content

Add OOM canary settings page to the sidebar#6390

Open
fastio wants to merge 1 commit into
ClickHouse:mainfrom
fastio:oom-canary-sidebar
Open

Add OOM canary settings page to the sidebar#6390
fastio wants to merge 1 commit into
ClickHouse:mainfrom
fastio:oom-canary-sidebar

Conversation

@fastio

@fastio fastio commented Jun 15, 2026

Copy link
Copy Markdown

The new OOM canary documentation page is added to ClickHouse/ClickHouse in
ClickHouse/ClickHouse#107465. This adds the matching
sidebar entry so the page is not flagged as a "floating" page by the docs build.

Merge order: the docs build copies the English docs from ClickHouse/ClickHouse
master, where oom-canary.md does not exist yet, so CI here will fail with a
dangling sidebar reference until that PR is merged into master. Please merge this
after ClickHouse/ClickHouse#107465.

Related: ClickHouse/ClickHouse#107465


Note

Low Risk
Single sidebar entry with no runtime or application logic; CI may fail temporarily until the synced doc file exists upstream.

Overview
Adds operations/settings/oom-canary to the Settings section of the serverAdmin sidebar in sidebars.js, placed after memory overcommit so the new OOM canary doc is linked in navigation instead of being treated as a floating page.

This is a navigation-only change; the markdown page is introduced in ClickHouse/ClickHouse and is expected to land via the docs sync. Until that upstream doc exists on master, the docs build may fail on a dangling sidebar reference—merge after the companion ClickHouse PR.

Reviewed by Cursor Bugbot for commit e8d584f. Bugbot is set up for automated code reviews on this repo. Configure here.

@fastio fastio requested a review from a team as a code owner June 15, 2026 06:42
@vercel

vercel Bot commented Jun 15, 2026

Copy link
Copy Markdown

@fastio is attempting to deploy a commit to the ClickHouse Team on Vercel.

A member of the Team first needs to authorize it.

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.

1 participant