mirror of
https://github.com/kata-containers/kata-containers.git
synced 2025-06-29 16:57:18 +00:00
sandbox: return ErrNoSuchContainer when failing to find a container
So that caller can determine that it is ENOENT-alike error. Signed-off-by: Peng Tao <bergwolf@hyper.sh>
This commit is contained in:
parent
8215a3ce9a
commit
f5125421d0
@ -7,7 +7,6 @@ package virtcontainers
|
||||
|
||||
import (
|
||||
"context"
|
||||
"errors"
|
||||
"fmt"
|
||||
"io"
|
||||
"net"
|
||||
@ -18,7 +17,9 @@ import (
|
||||
"github.com/containernetworking/plugins/pkg/ns"
|
||||
specs "github.com/opencontainers/runtime-spec/specs-go"
|
||||
opentracing "github.com/opentracing/opentracing-go"
|
||||
"github.com/pkg/errors"
|
||||
"github.com/sirupsen/logrus"
|
||||
"github.com/vishvananda/netlink"
|
||||
|
||||
"github.com/kata-containers/agent/protocols/grpc"
|
||||
"github.com/kata-containers/runtime/virtcontainers/device/api"
|
||||
@ -31,7 +32,6 @@ import (
|
||||
"github.com/kata-containers/runtime/virtcontainers/store"
|
||||
"github.com/kata-containers/runtime/virtcontainers/types"
|
||||
"github.com/kata-containers/runtime/virtcontainers/utils"
|
||||
"github.com/vishvananda/netlink"
|
||||
)
|
||||
|
||||
const (
|
||||
@ -678,7 +678,7 @@ func (s *Sandbox) findContainer(containerID string) (*Container, error) {
|
||||
}
|
||||
}
|
||||
|
||||
return nil, fmt.Errorf("Could not find the container %q from the sandbox %q containers list",
|
||||
return nil, errors.Wrapf(vcTypes.ErrNoSuchContainer, "Could not find the container %q from the sandbox %q containers list",
|
||||
containerID, s.id)
|
||||
}
|
||||
|
||||
@ -694,7 +694,7 @@ func (s *Sandbox) removeContainer(containerID string) error {
|
||||
}
|
||||
|
||||
if _, ok := s.containers[containerID]; !ok {
|
||||
return fmt.Errorf("Could not remove the container %q from the sandbox %q containers list",
|
||||
return errors.Wrapf(vcTypes.ErrNoSuchContainer, "Could not remove the container %q from the sandbox %q containers list",
|
||||
containerID, s.id)
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user