Skip to content

add go e2b sdk - #21

Merged
furykerry merged 5 commits into
openkruise:masterfrom
ZhaoQing7892:go-e2b-api
May 26, 2026
Merged

add go e2b sdk#21
furykerry merged 5 commits into
openkruise:masterfrom
ZhaoQing7892:go-e2b-api

Conversation

@ZhaoQing7892

Copy link
Copy Markdown
Contributor

No description provided.

@kruise-bot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ZhaoQing7892
ZhaoQing7892 force-pushed the go-e2b-api branch 2 times, most recently from c4b119c to 893938d Compare May 9, 2026 08:49
Comment thread sdk/example/runtime_client/main.go
Comment thread sdk/runtime/client.go Outdated
Comment thread sdk/sandbox/config.go Outdated
Comment thread sdk/README.md Outdated
Comment thread sdk/runtime/config.go Outdated
Comment thread sdk/runtime/client.go Outdated
Comment thread sdk/README.md

---

## Quick Start: runtime (Runtime Client)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this example is not necessary, just keep the NewFromK8s example

Comment thread sdk/README.md Outdated
ctx := context.Background()

c, err := runtime.NewFromK8s(ctx, "default", "your-sandbox-name",
runtime.WithDomain("127.0.0.1:7788"),

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

plz comment about what the domain is. Maybe a service is enough for in-cluster access

@furykerry
furykerry merged commit 98f4bfd into openkruise:master May 26, 2026
1 check was pending
@ZhaoQing7892
ZhaoQing7892 deleted the go-e2b-api branch May 27, 2026 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants