github/containerd
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 2 Wiki Activity

Change ListProcesses to ListPids

Browse Source 
These rpcs only return pids []uint32 so should be named that way in
order to have other rpcs that list Processes such as Exec'd processes.

Signed-off-by: Michael Crosby <crosbymichael@gmail.com>
This commit is contained in:
Michael Crosby
2017-06-28 16:09:29 -07:00
parent 55861c1a00
commit e2d5522435
11 changed files with 415 additions and 369 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
linux/task.go
View File

@@ -127,21 +127,14 @@ func (t *Task) Exec(ctx context.Context, opts runtime.ExecOpts) (runtime.Process
}, nil
}
func (t *Task) Processes(ctx context.Context) ([]uint32, error) {
resp, err := t.shim.ListProcesses(ctx, &shim.ListProcessesRequest{
func (t *Task) Pids(ctx context.Context) ([]uint32, error) {
resp, err := t.shim.ListPids(ctx, &shim.ListPidsRequest{
ID: t.containerID,
})
if err != nil {
return nil, errors.New(grpc.ErrorDesc(err))
}
pids := make([]uint32, 0, len(resp.Processes))
for _, ps := range resp.Processes {
pids = append(pids, ps.Pid)
}
return pids, nil
return resp.Pids, nil
}
func (t *Task) ResizePty(ctx context.Context, pid uint32, size runtime.ConsoleSize) error {