Skip to content

Make existing ClickPipes metrics and error diagnostics discoverable - #956

Merged
sdairs merged 2 commits into
codex/focused-922from
codex/focused-847
Sep 17, 2026
Merged

sdairs merged 2 commits into
codex/focused-922from
codex/focused-847

Conversation

@sdairs

@sdairs sdairs commented Sep 15, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Make existing ClickPipes diagnostics discoverable: filter the destination service's Prometheus samples by ClickPipe ID, or inspect recent streaming/object-storage errors through cloud service query and system.clickpipes_log.

README documents metric availability, CDC log limitations, record-error tables and authentication prerequisites. No commands, endpoints or runtime behavior are added.

Fixes #847.

Validation

Verified against official monitoring and error-reporting documentation. Per agent-query-safety, the SQL example includes a time filter, result limit, scan cap and timeout. Formatting and 3 whole-command-tree structural tests passed; no live query was run.

Stack

Appended to native stack 876.

Combined local validation

Combined stack head e20d63b3af4c8541a6f597d9494b530f2c56b666 passed all required local checks: formatting, CLI clippy/tests, telemetry-disabled check and all-target clippy, API/analyzer all-target clippy/tests, all-feature workspace build, Python classifier/drift tests, and diff checks.

Test totals: 1991 CLI, 672 API/analyzer, and 93 Python tests (2756 total). Ignored/live tests are excluded from these totals.

Snapshot API access was revalidated successfully on the exact heads of PRs #940 and #944 after the environment feature flag was enabled. Run links and coverage details are recorded in tracker #939.

Stack order, exact heads and explicit issue Development relationships were verified in the implementation tracker.

@sdairs
sdairs added this pull request to stack #876 September 15, 2026 15:37
@sdairs
sdairs marked this pull request as ready for review September 16, 2026 13:14
Carry existing lower-stack README corrections and ancestry forward. The merged tree matches the independently verified cumulative stack result.

@iskakaushik iskakaushik left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@sdairs
sdairs merged commit a65a9ca into main Sep 17, 2026
10 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.

Make existing ClickPipes metrics and error diagnostics discoverable

2 participants