Fix process management

b6cf01a4 refactor: make path to std::process::exit shorter

This patch reduces the variable passing trace length. Instead of returning it from function to function, call exit() in place.

The trace of exit code before the patch:

std::process::exit() <- main <- main_run <- fork(child) <- tarantool_main

And now:

std::process::exit() <- tarantool_main

The type safety is ensured at compilation time by using ! type for fn main(). See https://doc.rust-lang.org/reference/types/never.html

5b5d25f3 Handle supervisor termination in child

Handle supervisor termination in child

We don't want a child process to live without the supervisor.

Usually, supervisor waits for child forever and retransmits termination signals. But if the parent is killed with a SIGKILL there's no way to pass anything.

This patch supplies a child process with a supervisor_fuse fiber. It tries to read from a pipe (that supervisor never writes to), and if the writing end is closed, it means the supervisor has terminated. In this case, child process terminates too.

Part of #56 (closed)

c3af1724 Forward signals from supervisor to child

That's it.

Close #56 (closed)

changed milestone to %Cluster membership changes

added in-progress label

added 1 commit

• 28168a16 - Forward signals from supervisor to child

Compare with previous version

changed the description

mentioned in merge request !71 (merged)

requested review from @sv

added 10 commits

• 28168a16...2873f401 - 7 commits from branch raft-join-05-tarantool_main-macro
• b6cf01a4 - refactor: make path to std::process::exit shorter
• 5b5d25f3 - Handle supervisor termination in child
• c3af1724 - Forward signals from supervisor to child

Compare with previous version

changed the description

deleted the raft-join-05-tarantool_main-macro branch. This merge request now targets the master branch

В логах уровня Info и менее подробных не должно быть упоминаний supervisor. Убирать все равно придется, сейчас или потом. Лучше сейчас.

approved this merge request

marked this merge request as ready

resolved all threads

merged