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

runtime/vc: runPrestartHooks should ignore GetHypervisorPid failure

Browse Source 
If we are running FC hypervisor, it is not started when prestart hooks
are executed. So we should just ignore such error and just go ahead and
run the hooks.

Signed-off-by: Peng Tao <bergwolf@hyper.sh>
This commit is contained in:
Peng Tao 2023-08-30 03:01:22 +00:00
parent 21204caf20
commit 2e4c874726
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/runtime/virtcontainers/sandbox.go
View File

@ -1322,13 +1322,12 @@ func (s *Sandbox) cleanSwap(ctx context.Context) {
} }
func (s *Sandbox) runPrestartHooks(ctx context.Context, prestartHookFunc func(context.Context) error) error { func (s *Sandbox) runPrestartHooks(ctx context.Context, prestartHookFunc func(context.Context) error) error {
hid, err := s.GetHypervisorPid() hid, _ := s.GetHypervisorPid()
if err != nil { // Ignore errors here as hypervisor might not have been started yet, likely in FC case.
s.Logger().Errorf("fail to get hypervisor pid for sandbox %s", s.id) if hid > 0 {
return err
}
s.Logger().Infof("sandbox %s hypervisor pid is %v", s.id, hid) s.Logger().Infof("sandbox %s hypervisor pid is %v", s.id, hid)
ctx = context.WithValue(ctx, HypervisorPidKey{}, hid) ctx = context.WithValue(ctx, HypervisorPidKey{}, hid)
}
if err := prestartHookFunc(ctx); err != nil { if err := prestartHookFunc(ctx); err != nil {
s.Logger().Errorf("fail to run prestartHook for sandbox %s: %s", s.id, err) s.Logger().Errorf("fail to run prestartHook for sandbox %s: %s", s.id, err)