Skip to content

Fix doc URLs in alerts and dashboards#1906

Merged
Slach merged 1 commit intoAltinity:0.26.0from
siggy:siggy/fix-urls
Jan 15, 2026
Merged

Fix doc URLs in alerts and dashboards#1906
Slach merged 1 commit intoAltinity:0.26.0from
siggy:siggy/fix-urls

Conversation

@siggy
Copy link
Copy Markdown
Contributor

@siggy siggy commented Jan 14, 2026

Thanks for taking the time to contribute to clickhouse-operator!

Please, read carefully instructions on how to make a Pull Request.

This will help a lot for maintainers to adopt your Pull Request.

Important items to consider before making a Pull Request

Please check items PR complies to:

  • All commits in the PR are squashed. More info
  • The PR is made into dedicated next-release branch, not into master branch1. More info
  • The PR is signed. More info

--

1 If you feel your PR does not affect any Go-code or any testable functionality (for example, PR contains docs only or supplementary materials), PR can be made into master branch, but it has to be confirmed by project's maintainer.

Copilot AI review requested due to automatic review settings January 14, 2026 22:08
Copy link
Copy Markdown

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

This PR updates ClickHouse documentation URLs in Grafana dashboards and Prometheus alert rules to match the current ClickHouse documentation site structure.

Changes:

  • Removed /en/ language prefix from all ClickHouse documentation URLs
  • Updated URL anchor formats from hyphenated to underscored style
  • Removed trailing slash before anchors in alert rules

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
grafana-dashboard/Altinity_ClickHouse_Operator_dashboard.json Updated three documentation URLs to remove language prefix and fix anchor formats
deploy/prometheus/prometheus-alert-rules-clickhouse.yaml Updated async_insert documentation URL to remove language prefix, trailing slash, and fix anchor format
deploy/helm/clickhouse-operator/files/Altinity_ClickHouse_Operator_dashboard.json Updated three documentation URLs (duplicate of main dashboard file)

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

@Slach
Copy link
Copy Markdown
Collaborator

Slach commented Jan 15, 2026

nice catches, yep, links outdated little bit, thanks for contribution

@Slach Slach merged commit c5aaf3e into Altinity:0.26.0 Jan 15, 2026
2 checks passed
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