Skip to content

FEATURE: simple MVP for opensandbox egress #78

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
hittyt merged 9 commits into from
Jan 7, 2026
Merged

FEATURE: simple MVP for opensandbox egress #78

hittyt merged 9 commits into from
Jan 7, 2026

Conversation

@Pangjiping
Copy link
Collaborator

@Pangjiping Pangjiping commented Jan 7, 2026 •
edited
Loading

Summary

  • add minimal implementation of OpenSandbox egress component.
image

Testing

  • Not run (explain why)
  • Unit tests
  • Integration tests
  • e2e / manual verification

Breaking Changes

  • None
  • Yes (describe impact and migration path)

Checklist

  • Linked Issue or clearly described motivation
  • Added/updated docs (if needed)
  • Added/updated tests (if needed)
  • Security impact considered
  • Backward compatibility considered

@Pangjiping Pangjiping force-pushed the feat/components/egress-part1 branch from 0b67a64 to e90d41e Compare January 7, 2026 09:52
@Pangjiping Pangjiping force-pushed the feat/components/egress-part1 branch from d7e29e8 to 626f6a1 Compare January 7, 2026 10:36
@Pangjiping Pangjiping force-pushed the feat/components/egress-part1 branch from 626f6a1 to cd9c352 Compare January 7, 2026 10:40
hittyt
hittyt reviewed Jan 7, 2026
View reviewed changes
@Pangjiping
Copy link
Collaborator Author

Pangjiping commented Jan 7, 2026

for #56

@hittyt hittyt self-requested a review January 7, 2026 13:30
hittyt
hittyt approved these changes Jan 7, 2026
View reviewed changes
Copy link
Collaborator

@hittyt hittyt left a comment

Choose a reason for hiding this comment

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

LGTM

@hittyt hittyt merged commit 2fabb9d into alibaba:main Jan 7, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hittyt hittyt hittyt approved these changes

@jwx0925 jwx0925 Awaiting requested review from jwx0925 jwx0925 is a code owner

@ninan-nn ninan-nn Awaiting requested review from ninan-nn ninan-nn is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Pangjiping @hittyt