zhuangchong opened a new pull request #4217:
URL: https://github.com/apache/incubator-dolphinscheduler/pull/4217
## What is the purpose of the pull request
#4128
Dolphin executes the command using sudo to specify the -u parameter
sudo:
-u username /#uid without this parameter means that you want to execute the
instruction as root
At present, dolphin source code exists sudo kill $PID /sudo kill -9 $PID
situation
Executing the above statement will report insufficient permissions when
dolphin installs THE OS user's sudo permissions with only a partial user
account configuration.
## Brief change log
* sudo kill $PID changed to sudo -u username kill $PID
## Verify this pull request
This change added tests and can be verified as follows:
- *Manually verified the change by testing locally.*
---
## pull request的目的是什么
dolphin使用sudo 执行命令指定-u 参数
sudo：
-u username/#uid 不加此参数，代表要以 root 的身份执行指令
目前dolphin源码内存在sudo kill $pid /sudo kill -9 $pid 的情况
当dolphin 安装os用户的sudo权限只配置部分用户账号时，执行上面语句将会报出权限不足。
## 简短的更改日志
sudo kill $PID 更改为 sudo -u username kill $PID
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
