Repository navigation
Conversation
Signed-off-by: Srinivasa Consultants <78673153+dinakarjs@users.noreply.github.com>
|
🟡 Contributor Check: MEDIUM
Automated check by AgenTrust Contributor Check. |
github-actions
Bot
added
the
needs-review:MEDIUM
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
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.
Add a bounded experiment against the existing pinned official Collector 0.163.2 binary. A receiver barrier holds one export before commit while a second request occupies the remaining memory-queue capacity. Check overflow refusal, then separately SIGKILL at that boundary and measure recovery using a fresh request. SQLite retains attempted/committed identities; summaries distinguish submitted, HTTP-accepted and committed events.
Stacked follow-up to agentrust-io#100 at 2714186. This fork PR isolates the follow-up pending parent adoption. Coordinate the shared workflow/README changes with the fidelity companion when rebasing.
Validation: three accounting tests pass; Python compilation, focused Ruff and actionlint pass. The real pinned Collector saturation case returned 200/200/503 and committed the first two identities. A subsequent Collector startup failed with the known sandbox
failed to register process metrics: process does not existerror. The complete in-flight restart case has not passed locally and remains a required hosted experiment before merge; no full-run success is claimed.Direct synthetic OTLP requests exercise the Collector, not SDK retry behavior. The queue is in memory, the receiver deliberately discards the interrupted request, and accepted-but-uncommitted records establish only this experiment's observation. No persistent-queue, power-loss, production durability or exactly-once claim. AI-assisted with Codex; DCO signed.