process immediate exit after sigterm

[amphp]

process immediate exit after sigterm

[trowski]

I was not able to reproduce this on macOS, but I was able to reproduce this on Ubuntu. The sh process terminates in both, but on macOS join() does not resolve until the child PHP process exits, whereas on Ubuntu join() resolves as soon as the sh process ends.

[Ekstazi]

Thx for fix. But signals still broadcasted to child process. How i can prevent it ? I handled signals in my parent process and want to send it to child only from code.

[Ekstazi]

Also signal handling not worked after that fix. I changed test.php scrip:

<?php
require __DIR__.'/vendor/autoload.php';

\Amp\Loop::run(function(){
	$process = new \Amp\Process\Process("php test-process.php");
	$pid = yield $process->start();
	\Amp\ByteStream\pipe($process->getStdout(), \Amp\ByteStream\getStdout());
	\Amp\ByteStream\pipe($process->getStderr(), \Amp\ByteStream\getStderr());
	$onInterrupt = \Amp\Loop::onSignal(SIGINT, function ($reference) use ($process){
		echo "sigterm\n";
		yield new \Amp\Delayed(5000);
		echo "terminating child\n";
		$process->signal(SIGINT);
		\Amp\Loop::cancel($reference);
	});
	$code = yield $process->join();
	echo $code;
	if(file_exists("/proc/{$pid}")) {
		echo "Process still running\n";
	}
	yield new \Amp\Delayed(10000);
	\Amp\Loop::cancel($onInterrupt);
});

As you can see:

1. i started test-process.php
2. wait for sigint
3. if sigint received then wait 5s and send it child

As i can see sigint briadcasted on step 2 but i can't send it manually on step 3. May be need to use posix_kill instead of proc_terminate ?

[Ekstazi]

https://stackoverflow.com/questions/34926587/signal-being-forwarded-to-children-for-the-symfony-process-component

As i understand i cannot prevent SIGINT sent to process group. But problem with sending signal to children process from code still exist

[trowski]

Fix sending signals to the child process in d51bd9a using posix_kill instead of proc_terminate as you suggested, since the latter was sending the signal to the wrapper child instead of the target child. Obviously the Process::signal() method is not commonly used since this wasn't caught sooner. Generally child signal handlers catch signals from the console which are forwarded to all children. As you discovered, there is no way to change that behavior unless you change the group ID of the child process.

[Ekstazi]

Thx. I use Process::signal in my code.