Problemas de casal que vejo.
Eu fiz um script de teste simples:
#!/bin/bash
startAndStop(){
echo "Start 1" &
echo "Sleep 2" && echo "Stop 3"
echo "Sleep 4" && exit
}
startAndStop
echo "Rest of Script"
Isso simula a função startAndStop do seu programa.
Aqui está a saída:
$ bash test.sh
Sleep 2
Stop 3
Sleep 4
Start 1
Dois pontos sobre essa função:
-
As coisas são executadas fora de ordem. O único
&
origina a tarefa inicial. Tem certeza de que queria fazer isso? Você não gostaria de esperar que o processo fosse concluído, começando a pará-lo? Eu sei que você colocasleep
s, mas esperar que saia é melhor, eu acho. As coisas podem ficar muito estranhas se o comando não terminar a tempo. -
Observe como não foi impressa
Rest of Script
. O comandoexit
não sai da função, ele sai do script inteiro.
Você tem /usr/bin/sudo su - user1 && $exec_start
. Eu não acho que é isso que você queria fazer. Você define exec_start="sudo systemctl start new-service"
. Então, o que você acaba fazendo é mudar para user1 e, em seguida, elevando para permissões de root para iniciar o serviço. Novo script, na minha máquina. $(pwd)
imprimirá o diretório atual.
#!/bin/bash
sudo su - user1 && /usr/bin/sudo echo $(pwd)
Vamos executá-lo:
[root]$ bash test.sh
[user1]$
Eu tenho uma concha! O &&
aguarda a conclusão do comando anterior - a saber, sudo su -
, que fornece um shell de root. O comando echo
não foi executado até eu sair do shell:
[user1]$ exit
logout
/root
[root]$
Mesmo assim, ele forneceu o diretório /root
, não /home/user1
. Isso definitivamente não é o que você queria. Que tal adicionar este script ao crontab para rodar como usuário ? Esta seria a linha crontab do root:
*/5 * * * * user1 /home/user1/new-service-watch.sh
Dessa forma, você não teria que mexer com o usuário do switch duplo. O script será iniciado por cron
as user1
, então você só precisa sudo
para poder executar systemctl
e manter todo o resto como usuário.
Aqui está uma ideia rápida. Por favor, não use como está !!!!
edit: alterando o script para iniciar o serviço como usuário1, por solicitação. removeu $sudo_cmd
das definições exec_start
e exec_stop
.
* edit2: chowning deve usar sudo
caso os arquivos sejam de propriedade de root. Além disso, parar o serviço deve usar sudo
apenas no caso de root ter iniciado o processo. '
#!/bin/bash
sudo_cmd="/usr/bin/sudo" # these are just my paths, change as needed
chown_cmd="$sudo_cmd /usr/bin/chown" # added sudo
systemctl_cmd="/bin/systemctl"
date_cmd="/bin/date"
# if define process before exec_start & exec_stop,
# you can use it in your exec_start & exec_stop definitions
process="new-service"
exec_start="$systemctl_cmd start $process" # don't sudo so it runs as user1
exec_stop="$sudo_cmd $systemctl_cmd stop $process" # added sudo
check1="/root/testing"
username="user1"
log_file="/var/log/new-service-watch.log" # why not log things? :)
# putting the entirety of the execution in
# braces lets you pipe it somewhere easily
{ # start_brace
changeOwnership()
{
echo "$($date_cmd) Changing ownership"
# there is no need to change directory
# because we will use '-R' to be recursive
$chown_cmd -R $username:$username $check1
}
startService()
{
echo "$($date_cmd) Starting $process"
$exec_start
sleep 5
}
stopService()
{
echo "$($date_cmd) Stopping $process"
$exec_stop
sleep 5
}
if ps ax | grep -v grep | grep $process > /dev/null
then
echo "$($date_cmd) Running...No action needed."
else
echo "$($date_cmd) Not running...Going to start the service."
startService
stopService
changeOwnership
startService
fi
} >> "$log_file" # end_brace