-
Notifications
You must be signed in to change notification settings - Fork 159
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.
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
feasibility-research(runtime): Support kubernetes as the backend vendor #179
Comments
Actually, I am not very familiar with Kubernetes CRI, I need some time to learn it. |
Line 232 in 8e1e498
We mount the build context into the docker container now, it is impossible if we use kubernetes. Maybe we need a new rule in starlark to deal with source code |
Exec(ctx context.Context, cname string, cmd []string) error
Destroy(ctx context.Context, name string) (string, error)
List(ctx context.Context) ([]types.Container, error)
// GPUEnabled returns true if nvidia container runtime exists in docker daemon.
GPUEnabled(ctx context.Context) (bool, error)
// Start creates the container for the given tag and container name.
StartEnvd(ctx context.Context, tag, name, buildContext string,
gpuEnabled bool, g ir.Graph, timeout time.Duration, mountOptionsStr []string) (string, string, error) These funcs in the docker interface should be abstracted to a new interface |
I think we could not mount local dir to container in k8s cluster directly, how about we spawn a rsyncd inside the container, and sync working dir by using rsync?
This module https://pkg.go.dev/k8s.io/client-go/tools/portforward might be helpful. |
Yep we may need https://syncthing.net/ |
#dibs |
Maybe this is relevant to the discussion. @gaocegege It also utilises syncthing https://github.com/ksync/ksync/blob/master/docs/architecture.md |
Thanks for the info! it seems like a clean wrapper of sycthing. |
The text was updated successfully, but these errors were encountered: