Skip to content

fix: CHC properly detected in Status page#9144

Merged
royendo merged 2 commits intomainfrom
royendo/chc-cloud-label-detection
Apr 6, 2026
Merged

fix: CHC properly detected in Status page#9144
royendo merged 2 commits intomainfrom
royendo/chc-cloud-label-detection

Conversation

@royendo
Copy link
Copy Markdown
Contributor

@royendo royendo commented Mar 30, 2026

Status page today doesnt detect CHC properly shows up at
Clickhouse (self-managed)

Screenshot 2026-03-30 at 9 32 32

Based on inputs of DSN or config properties

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

@royendo royendo requested review from begelundmuller and k-anshul and removed request for k-anshul March 30, 2026 13:33
@royendo royendo requested a review from AdityaHegde April 3, 2026 16:04
@royendo
Copy link
Copy Markdown
Contributor Author

royendo commented Apr 6, 2026

Remove "self-managed logic" since we're not doing special case for CHC, its all Live Connectors as prod-slots

@royendo royendo dismissed begelundmuller’s stale review April 6, 2026 16:04

taking out go logic :)

@royendo royendo merged commit 8006151 into main Apr 6, 2026
12 of 13 checks passed
@royendo royendo deleted the royendo/chc-cloud-label-detection branch April 6, 2026 16:04
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