[tarantool-patches] Re: [PATCHv2] Detect when instance is run or restarted by tarantoolctl.
sergepetrenko at tarantool.org
Thu Jul 5 09:58:47 MSK 2018
04.07.2018 21:04, Konstantin Osipov пишет:
> * Serge Petrenko <sergepetrenko at tarantool.org> [18/07/02 17:54]:
>> There are some hacks to know the instance was run by tarantoolctl,
>> none of them are too reliable, though. This patch introduces 2
>> environment variables set by tarantoolctl for the instance to
>> know when it's being run or restarted.
>> Closes: #3215
> Please add a petition to docbot to the commit message.
Hi, thank you for review. Petition added:
There are some hacks to know the instance was run by tarantoolctl,
none of them are too reliable, though. This patch introduces 2
environment variables set by tarantoolctl for the instance to
know when it's being run or restarted.
Title: tarantoolctl: document setting environment variables
tarantoolctl sets the `TARANTOOLCTL` environment variable when starting
an instance, and sets the `TARANTOOL_RESTARTED' environment variable
extra/dist/tarantoolctl.in | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/extra/dist/tarantoolctl.in b/extra/dist/tarantoolctl.in
index b6b8e09c9..ecab0a0f1 100755
@@ -514,6 +514,11 @@ local function start()
script_name = instance_path,
__defer_update = true
+ -- an env variable to tell when we the
+ -- instance is run under tarantoolctl
+ os.setenv('TARANTOOLCTL', 'true')
shift_argv(arg, 0, 2)
local success, data = pcall(dofile, instance_path)
-- if load fails - show last 10 lines of the log file and exit
@@ -593,6 +598,9 @@ local function restart()
+ -- an env variable to tell when
+ -- the instance was restarted
+ os.setenv('TARANTOOL_RESTARTED', 'true')
> Re TARANTOOL_RESTARTED, let's discuss this - I don't think
> this is going to address
> and gh-3215 is not asking for it.
> So for now I would remove it from the patch.
I believe you asked for it in the comments, if I understood correctly:
More information about the Tarantool-patches