github/kata-containers
1
0
Fork 1
mirror of https://github.com/kata-containers/kata-containers.git synced 2025-06-30 09:13:29 +00:00
Code Issues Projects Releases Wiki Activity

config: fix panic in type assertion

Browse Source 
when use shim v2 interface to run container, no need to use kata-proxy
and kata-shim, remove kata-proxy and kata-shim in config file will cause
panic since type assertion. add check to avoid panic

Fixes: #1440

Signed-off-by: Ace-Tang <aceapril@126.com>
This commit is contained in:
Ace-Tang 2019-03-28 19:07:15 +08:00
parent c89eb81dec
commit 1b6affe498
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
pkg/katautils/config.go
View File

@ -935,7 +935,7 @@ func checkNetNsConfig(config oci.RuntimeConfig) error {
if config.InterNetworkModel != vc.NetXConnectNoneModel { if config.InterNetworkModel != vc.NetXConnectNoneModel {
return fmt.Errorf("config disable_new_netns only works with 'none' internetworking_model") return fmt.Errorf("config disable_new_netns only works with 'none' internetworking_model")
} }
} else if config.ShimConfig.(vc.ShimConfig).Trace { } else if shim, ok := config.ShimConfig.(vc.ShimConfig); ok && shim.Trace {
// Normally, the shim runs in a separate network namespace. // Normally, the shim runs in a separate network namespace.
// But when tracing, the shim process needs to be able to talk // But when tracing, the shim process needs to be able to talk
// to the Jaeger agent running in the host network namespace. // to the Jaeger agent running in the host network namespace.