Merge pull request #6910 from wllenyj/debug

shim: fix debug flag not working
This commit is contained in:
Phil Estes 2022-05-14 19:53:19 +01:00 committed by GitHub
commit e217c83e8c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 2 deletions

View File

@ -69,7 +69,7 @@ type manager struct {
name string name string
} }
func newCommand(ctx context.Context, id, containerdBinary, containerdAddress, containerdTTRPCAddress string) (*exec.Cmd, error) { func newCommand(ctx context.Context, id, containerdBinary, containerdAddress, containerdTTRPCAddress string, debug bool) (*exec.Cmd, error) {
ns, err := namespaces.NamespaceRequired(ctx) ns, err := namespaces.NamespaceRequired(ctx)
if err != nil { if err != nil {
return nil, err return nil, err
@ -87,6 +87,9 @@ func newCommand(ctx context.Context, id, containerdBinary, containerdAddress, co
"-id", id, "-id", id,
"-address", containerdAddress, "-address", containerdAddress,
} }
if debug {
args = append(args, "-debug")
}
cmd := exec.Command(self, args...) cmd := exec.Command(self, args...)
cmd.Dir = cwd cmd.Dir = cwd
cmd.Env = append(os.Environ(), "GOMAXPROCS=4") cmd.Env = append(os.Environ(), "GOMAXPROCS=4")
@ -114,7 +117,7 @@ func (m manager) Name() string {
} }
func (manager) Start(ctx context.Context, id string, opts shim.StartOpts) (_ string, retErr error) { func (manager) Start(ctx context.Context, id string, opts shim.StartOpts) (_ string, retErr error) {
cmd, err := newCommand(ctx, id, opts.ContainerdBinary, opts.Address, opts.TTRPCAddress) cmd, err := newCommand(ctx, id, opts.ContainerdBinary, opts.Address, opts.TTRPCAddress, opts.Debug)
if err != nil { if err != nil {
return "", err return "", err
} }

View File

@ -54,6 +54,7 @@ type StartOpts struct {
ContainerdBinary string ContainerdBinary string
Address string Address string
TTRPCAddress string TTRPCAddress string
Debug bool
} }
type StopStatus struct { type StopStatus struct {
@ -329,6 +330,7 @@ func run(ctx context.Context, manager Manager, initFunc Init, name string, confi
ContainerdBinary: containerdBinaryFlag, ContainerdBinary: containerdBinaryFlag,
Address: addressFlag, Address: addressFlag,
TTRPCAddress: ttrpcAddress, TTRPCAddress: ttrpcAddress,
Debug: debugFlag,
} }
address, err := manager.Start(ctx, id, opts) address, err := manager.Start(ctx, id, opts)