diff --git a/src/runtime/virtcontainers/sandbox.go b/src/runtime/virtcontainers/sandbox.go index 363763a0c2..2a41a1b4b9 100644 --- a/src/runtime/virtcontainers/sandbox.go +++ b/src/runtime/virtcontainers/sandbox.go @@ -1012,6 +1012,12 @@ func (s *Sandbox) startVM(ctx context.Context) (err error) { s.cw = consoleWatcher } + defer func() { + if err != nil { + s.hypervisor.stopSandbox(ctx, false) + } + }() + if err := s.network.Run(ctx, s.networkNS.NetNsPath, func() error { if s.factory != nil { vm, err := s.factory.GetVM(ctx, VMConfig{ @@ -1031,12 +1037,6 @@ func (s *Sandbox) startVM(ctx context.Context) (err error) { return err } - defer func() { - if err != nil { - s.hypervisor.stopSandbox(ctx, false) - } - }() - // In case of vm factory, network interfaces are hotplugged // after vm is started. if s.factory != nil { @@ -1553,18 +1553,7 @@ func (s *Sandbox) Stop(ctx context.Context, force bool) error { return nil } -// list lists all sandbox running on the host. -func (s *Sandbox) list() ([]Sandbox, error) { - return nil, nil -} - -// enter runs an executable within a sandbox. -func (s *Sandbox) enter(args []string) error { - return nil -} - -// setSandboxState sets both the in-memory and on-disk state of the -// sandbox. +// setSandboxState sets the in-memory state of the sandbox. func (s *Sandbox) setSandboxState(state types.StateString) error { if state == "" { return vcTypes.ErrNeedState diff --git a/src/runtime/virtcontainers/sandbox_test.go b/src/runtime/virtcontainers/sandbox_test.go index e91650fdee..27df559d36 100644 --- a/src/runtime/virtcontainers/sandbox_test.go +++ b/src/runtime/virtcontainers/sandbox_test.go @@ -181,21 +181,6 @@ func TestCreateSandboxEmptyID(t *testing.T) { defer cleanUp() } -func TestSandboxListSuccessful(t *testing.T) { - sandbox := &Sandbox{} - - sandboxList, err := sandbox.list() - assert.NoError(t, err) - assert.Nil(t, sandboxList) -} - -func TestSandboxEnterSuccessful(t *testing.T) { - sandbox := &Sandbox{} - - err := sandbox.enter([]string{}) - assert.NoError(t, err) -} - func testCheckInitSandboxAndContainerStates(p *Sandbox, initialSandboxState types.SandboxState, c *Container, initialContainerState types.ContainerState) error { if p.state.State != initialSandboxState.State { return fmt.Errorf("Expected sandbox state %v, got %v", initialSandboxState.State, p.state.State)