burble/nomad-driver-containerd
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Error out on invalid signal.

Browse Source
This commit is contained in:
Shishir Mahajan 2020-08-26 10:05:36 -07:00
parent 418cd4a855
commit ac74e4f226
No known key found for this signature in database
GPG Key ID: D41782E7688DEC4A
1 changed files with 4 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
containerd/driver.go
View File

@ -20,7 +20,6 @@ package containerd
import ( import (
"context" "context"
"fmt" "fmt"
"os"
"syscall" "syscall"
"time" "time"
@ -583,13 +582,11 @@ func (d *Driver) SignalTask(taskID string, signal string) error {
// The given signal will be forwarded to the target taskID. // The given signal will be forwarded to the target taskID.
// Please checkout https://github.com/hashicorp/consul-template/blob/master/signals/signals_unix.go // Please checkout https://github.com/hashicorp/consul-template/blob/master/signals/signals_unix.go
// for a list of supported signals. // for a list of supported signals.
sig := os.Interrupt sig, ok := signals.SignalLookup[signal]
if s, ok := signals.SignalLookup[signal]; ok { if !ok {
sig = s return fmt.Errorf("Invalid signal: %s", signal)
} else {
d.logger.Warn("unknown signal to send to task, using SIGINT instead", "signal", signal, "task_id", handle.taskConfig.ID)
} }
return handle.signal(d.ctxContainerd, sig) return handle.signal(d.ctxContainerd, sig)
} }