docs: simplest sidecar attach recipe; correct the test counts#59
Merged
Conversation
Two fixes from a docs↔implementation audit: - The sidecar section showed only the k8s manifest — no minimal recipe. Add the simplest attach to both READMEs: build the proxy image, run it with the policy mounted (the image's default CMD), and set ONE env var on the agent (OPENAI_BASE_URL). Verified live: the image builds (161MB), boots, listens, and fail-closes (403) with the upstream down. - The metrics undercounted: "75 tests / 13 eBPF" missed the pasu-daemon (2) and pasu-cli (3) units and the new `pasu run` kernel E2E (2). Correct to 82 = 62 portable + 20 Linux (14 unit · 6 kernel E2E), and add a `pasu run` row to the claims↔evidence matrix in docs/metrics.svg (panel resized). Signed-off-by: Ho Geun Choi <ohyes9711@gmail.com>
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.
Two fixes from a docs↔implementation audit:
1. Simplest sidecar attach was missing
The sidecar section only showed the k8s manifest. Added the minimal recipe to both READMEs — build the proxy image,
docker runit with the policy mounted (default CMD), and set one env var on the agent (OPENAI_BASE_URL). Verified live: image builds (161MB), boots, listens, fail-closes (403) with the upstream down.2. Test counts were stale
"75 tests / 13 eBPF" missed pasu-daemon (2) + pasu-cli (3) units and the new
pasu runkernel E2E (2, PR #58). Corrected to 82 = 62 portable + 20 Linux (14 unit · 6 kernel E2E) in both READMEs +docs/metrics.svg, and added apasu runrow to the claims↔evidence matrix (panel resized; rendered and eyeballed in a browser).GFM render verified (3 tables intact, no stale "75"). Docs-only.