diff options
| author | KubeEdge Bot <48982446+kubeedge-bot@users.noreply.github.com> | 2023-10-24 15:19:07 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-10-24 15:19:07 +0800 |
| commit | 7cff82b10d854375f95e055dc48424e79739880d (patch) | |
| tree | 7317a2206b8d9c65fb99df8e52ef19b06b39f641 | |
| parent | Merge pull request #5076 from Shelley-BaoYue/automated-cherry-pick-of-#5063-u... (diff) | |
| parent | fix keadm upgrade If edgecore is stopped, the keadm process will stop (diff) | |
| download | kubeedge-7cff82b10d854375f95e055dc48424e79739880d.tar.gz | |
Merge pull request #5110 from wlq1212/automated-cherry-pick-of-#5085-upstream-release-1.14v1.14.3
Automated cherry pick of #5085: fix keadm upgrade If edgecore is stopped, the keadm process
| -rw-r--r-- | edge/pkg/edgehub/upgrade/upgrade.go | 4 | ||||
| -rw-r--r-- | keadm/cmd/keadm/app/cmd/common/content.go | 1 |
2 files changed, 3 insertions, 2 deletions
diff --git a/edge/pkg/edgehub/upgrade/upgrade.go b/edge/pkg/edgehub/upgrade/upgrade.go index a8dd19d21..3a7ecfcd2 100644 --- a/edge/pkg/edgehub/upgrade/upgrade.go +++ b/edge/pkg/edgehub/upgrade/upgrade.go @@ -138,8 +138,8 @@ func (*keadmUpgrade) Upgrade(upgradeReq *commontypes.NodeUpgradeJobRequest) erro upgradeReq.UpgradeID, upgradeReq.HistoryID, version.Get(), upgradeReq.Version, opts.ConfigFile, image) // run upgrade cmd to upgrade edge node - // use setsid command and nohup command to start a separate progress - command := fmt.Sprintf("setsid nohup %s &", upgradeCmd) + // use nohup command to start a child progress + command := fmt.Sprintf("nohup %s &", upgradeCmd) cmd := exec.Command("bash", "-c", command) s, err := cmd.CombinedOutput() if err != nil { diff --git a/keadm/cmd/keadm/app/cmd/common/content.go b/keadm/cmd/keadm/app/cmd/common/content.go index 18db7682c..52a598872 100644 --- a/keadm/cmd/keadm/app/cmd/common/content.go +++ b/keadm/cmd/keadm/app/cmd/common/content.go @@ -34,6 +34,7 @@ ExecStart=%s Restart=always RestartSec=10 Environment=%s +KillMode=process [Install] WantedBy=multi-user.target |