Skip to content

docs: update egress control docs - #166

Merged
yingdi-shan merged 1 commit into
kvcache-ai:mainfrom
LSX-s-Software:docs/update-networking-doc
Aug 13, 2026
Merged

docs: update egress control docs#166
yingdi-shan merged 1 commit into
kvcache-ai:mainfrom
LSX-s-Software:docs/update-networking-doc

Conversation

@LSX-s-Software

Copy link
Copy Markdown
Member

What

Clarify the egress control document and update stale examples.

Related issue

Closes #156

Scope and non-goals

Documentation

Design and behavior changes

None

Compatibility and operations

  • Public API or generated protocol: N/A
  • Configuration or defaults: N/A
  • Snapshot manifest, artifact layout, or storage format: N/A
  • Upgrade and rollback: N/A
  • Host requirements, permissions, ports, or dependencies: N/A

Validation

  • make fmt
  • make clippy
  • make test-unit
  • Relevant Rust integration tests
  • make -C services test (required when services/ changes)
  • Generated clients/server regenerated with the documented make target
  • Documentation updated
  • Benchmarks or performance comparison completed

Skipped checks and reasons:

Doc changes only

Risks and reviewer notes

None

Checklist

  • The PR contains one coherent change and no unrelated formatting or refactoring.
  • New behavior is covered by tests, or I explained why testing is impractical.
  • Logs and examples contain no credentials, tokens, or private registry information.
  • I did not manually edit generated code without updating its source and regenerating it.

@LSX-s-Software
LSX-s-Software force-pushed the docs/update-networking-doc branch from ae1e24c to fed3460 Compare August 13, 2026 02:54
@github-actions

Copy link
Copy Markdown
Contributor

OpenCodeReview: Review skipped: no items were selected.

@yingdi-shan
yingdi-shan merged commit f8f725e into kvcache-ai:main Aug 13, 2026
1 check 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.

NewSandbox silently drops allowInternetAccess: sandbox created with open egress when isolation was requested

2 participants