Add CloseStdin to task
Signed-off-by: Michael Crosby <crosbymichael@gmail.com>
This commit is contained in:
parent
6aeeefe2b2
commit
9890bed1e1
@ -3,6 +3,9 @@ package containerd
|
|||||||
import (
|
import (
|
||||||
"bytes"
|
"bytes"
|
||||||
"context"
|
"context"
|
||||||
|
"fmt"
|
||||||
|
"io/ioutil"
|
||||||
|
"os"
|
||||||
"syscall"
|
"syscall"
|
||||||
"testing"
|
"testing"
|
||||||
)
|
)
|
||||||
@ -360,3 +363,85 @@ func TestContainerProcesses(t *testing.T) {
|
|||||||
}
|
}
|
||||||
<-statusC
|
<-statusC
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestContainerCloseStdin(t *testing.T) {
|
||||||
|
if testing.Short() {
|
||||||
|
t.Skip()
|
||||||
|
}
|
||||||
|
client, err := New(address)
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
defer client.Close()
|
||||||
|
|
||||||
|
var (
|
||||||
|
ctx = context.Background()
|
||||||
|
id = "ContainerCloseStdin"
|
||||||
|
)
|
||||||
|
image, err := client.GetImage(ctx, testImage)
|
||||||
|
if err != nil {
|
||||||
|
t.Error(err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
spec, err := GenerateSpec(WithImageConfig(ctx, image), WithProcessArgs("cat"))
|
||||||
|
if err != nil {
|
||||||
|
t.Error(err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
container, err := client.NewContainer(ctx, id, spec, WithImage(image), WithNewRootFS(id, image))
|
||||||
|
if err != nil {
|
||||||
|
t.Error(err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
defer container.Delete(ctx)
|
||||||
|
|
||||||
|
const expected = "hello\n"
|
||||||
|
stdout := bytes.NewBuffer(nil)
|
||||||
|
|
||||||
|
r, w, err := os.Pipe()
|
||||||
|
if err != nil {
|
||||||
|
t.Error(err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
task, err := container.NewTask(ctx, NewIO(r, stdout, ioutil.Discard))
|
||||||
|
if err != nil {
|
||||||
|
t.Error(err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
defer task.Delete(ctx)
|
||||||
|
|
||||||
|
statusC := make(chan uint32, 1)
|
||||||
|
go func() {
|
||||||
|
status, err := task.Wait(ctx)
|
||||||
|
if err != nil {
|
||||||
|
t.Error(err)
|
||||||
|
}
|
||||||
|
statusC <- status
|
||||||
|
}()
|
||||||
|
|
||||||
|
if err := task.Start(ctx); err != nil {
|
||||||
|
t.Error(err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
if _, err := fmt.Fprint(w, expected); err != nil {
|
||||||
|
t.Error(err)
|
||||||
|
}
|
||||||
|
w.Close()
|
||||||
|
if err := task.CloseStdin(ctx); err != nil {
|
||||||
|
t.Error(err)
|
||||||
|
}
|
||||||
|
|
||||||
|
<-statusC
|
||||||
|
|
||||||
|
if _, err := task.Delete(ctx); err != nil {
|
||||||
|
t.Error(err)
|
||||||
|
}
|
||||||
|
|
||||||
|
output := stdout.String()
|
||||||
|
|
||||||
|
if output != expected {
|
||||||
|
t.Errorf("expected output %q but received %q", expected, output)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
26
io.go
26
io.go
@ -54,6 +54,32 @@ func BufferedIO(stdin, stdout, stderr *bytes.Buffer) IOCreation {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func NewIO(stdin io.Reader, stdout, stderr io.Writer) IOCreation {
|
||||||
|
return func() (*IO, error) {
|
||||||
|
paths, err := fifoPaths()
|
||||||
|
if err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
|
i := &IO{
|
||||||
|
Terminal: false,
|
||||||
|
Stdout: paths.out,
|
||||||
|
Stderr: paths.err,
|
||||||
|
Stdin: paths.in,
|
||||||
|
}
|
||||||
|
set := &ioSet{
|
||||||
|
in: stdin,
|
||||||
|
out: stdout,
|
||||||
|
err: stderr,
|
||||||
|
}
|
||||||
|
closer, err := copyIO(paths, set, false)
|
||||||
|
if err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
|
i.closer = closer
|
||||||
|
return i, nil
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
// Stdio returns an IO implementation to be used for a task
|
// Stdio returns an IO implementation to be used for a task
|
||||||
// that outputs the container's IO as the current processes Stdio
|
// that outputs the container's IO as the current processes Stdio
|
||||||
func Stdio() (*IO, error) {
|
func Stdio() (*IO, error) {
|
||||||
|
9
task.go
9
task.go
@ -32,6 +32,7 @@ type Task interface {
|
|||||||
Wait(context.Context) (uint32, error)
|
Wait(context.Context) (uint32, error)
|
||||||
Exec(context.Context, *specs.Process, IOCreation) (Process, error)
|
Exec(context.Context, *specs.Process, IOCreation) (Process, error)
|
||||||
Processes(context.Context) ([]uint32, error)
|
Processes(context.Context) ([]uint32, error)
|
||||||
|
CloseStdin(context.Context) error
|
||||||
}
|
}
|
||||||
|
|
||||||
type Process interface {
|
type Process interface {
|
||||||
@ -158,3 +159,11 @@ func (t *task) Processes(ctx context.Context) ([]uint32, error) {
|
|||||||
}
|
}
|
||||||
return out, nil
|
return out, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func (t *task) CloseStdin(ctx context.Context) error {
|
||||||
|
_, err := t.client.TaskService().CloseStdin(ctx, &execution.CloseStdinRequest{
|
||||||
|
ContainerID: t.containerID,
|
||||||
|
Pid: t.pid,
|
||||||
|
})
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user