Dev observability stack: SigNoz integration tests, harness fix & repo hygiene#31
Merged
Merged
Conversation
added 9 commits
June 13, 2026 13:38
Replace the stringly-typed signalPath argument with dedicated ConfigureTraceExporter/ConfigureMetricExporter/ConfigureLogExporter methods. Each names its signal; the per-signal path lives in one place and the gRPC/HttpProtobuf endpoint logic stays in the shared core.
Drop the env-var config overrides, network Reachability probe and the IntegrationFact/SqlIntegrationFact skip attributes; tests now run against fixed local endpoints.
Add SigNoz helm values and a valid NodePort service (otel-collector selector, in-range nodePorts 30317/30318), align the other helm install scripts, and update editorconfig, solution layout and sample projects.
Export traces, metrics and logs over OTLP HTTP (port 30318) and verify ingestion via SigNoz's query_range API. A helm bootstrap Job registers the dev admin on first boot so the tests work without manual signup. Also fix OtelTestHost to instantiate the Tracer/Meter providers up front, so the ActivityListener and metric readers are active before spans/measurements are created (previously only logs were exported).
…CI/doc fixes - Move the prepare-release skill from .claude/ to .agents/ and retarget all pointers (AGENTS.md, CONTRIBUTING.md, .slnx); gitignore the local settings - Add a repo-tailored /evaluate-repository command and SECURITY.md - Trim CLAUDE.md to a pure pointer (its notes already live in AGENTS.md) - Fix stale AGENTS.md test docs (no IntegrationFact/auto-skip; hardcoded creds) - Fix ci.yml trailing-backtick continuation and make tool-manifest idempotent
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds a SigNoz path to the local dev observability stack with full round-trip integration tests, fixes a latent test-harness bug, and does a round of repo hygiene.
Telemetry / tests
SigNozIntegrationTests): export traces, metrics and logs over OTLP HTTP (:30318) and verify ingestion via SigNoz'squery_rangeAPI (:30111). NewSigNozClienthandles the (undocumented 0.128) auth flow + queries.signoz.bootstrap-admin.yamlJob registersadmin@web.deon first boot (idempotent), wired intohelm-install-signoz.cmd. Credentials are local dev-only defaults.OtelTestHost): instantiate the Tracer/Meter providers up front so theActivityListenerand metric readers are active before spans/measurements are created — previously only logs were exported. This also fixes the OpenObserve traces/metrics tests.v1/{traces,metrics,logs}) and a simplified integration harness (drops the unusedIntegrationFact/Reachabilityauto-skip).Sample app
Repo hygiene
prepare-releaseskill from.claude/to.agents/and retarget all pointers (AGENTS.md, CONTRIBUTING.md, SKILL.md,.slnx); gitignore the local settings.SECURITY.md(root) and a repo-tailored/evaluate-repositorycommand.CLAUDE.mdto a pure pointer; fix stale AGENTS.md test docs.ci.ymltrailing-backtick continuation and make the tool-manifest step idempotent.Notes
Category=Integration→ excluded from CI; they need the local helm stack up.🤖 Generated with Claude Code