Mirroristas/etcd
2
0
Fork 0
mirror of https://github.com/etcd-io/etcd.git synced 2024-09-27 06:25:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

etcdmain: fix sd_notify for restricted environments

Browse Source 
Remove call to dumb IsRunningSystemd() as it doesn't check anything
This commit is contained in:
Sergey Shatunov 2019-03-02 23:41:15 +07:00
parent 2c69559819
commit fbf732d3dc
No known key found for this signature in database
GPG Key ID: C9228B4C880E14B6
1 changed files with 1 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
etcdmain/main.go
View File

@ -20,7 +20,6 @@ import (
"strings"
"github.com/coreos/go-systemd/daemon"
systemdutil "github.com/coreos/go-systemd/util"
"go.uber.org/zap"
)
@ -48,14 +47,7 @@ func Main() {
}
func notifySystemd(lg *zap.Logger) {
if !systemdutil.IsRunningSystemd() {
return
}
if lg != nil {
lg.Info("host was booted with systemd, sends READY=1 message to init daemon")
}
sent, err := daemon.SdNotify(false, "READY=1")
_, err := daemon.SdNotify(false, daemon.SdNotifyReady)
if err != nil {
if lg != nil {
lg.Error("failed to notify systemd for readiness", zap.Error(err))
@ -63,12 +55,4 @@ func notifySystemd(lg *zap.Logger) {
plog.Errorf("failed to notify systemd for readiness: %v", err)
}
}
if !sent {
if lg != nil {
lg.Warn("forgot to set Type=notify in systemd service file?")
} else {
plog.Errorf("forgot to set Type=notify in systemd service file?")
}
}
}