yuanbiao
/
emqx
1
0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(install_upgrade): remove `vm_args` backup in `install` 

The `install_upgrade.escript` depended on being able to retrieve the
`vm_args` argument from `init`, but that options has been removed
recently.  Since this backup was unused, we also remove it here.
This commit is contained in:
Thales Macedo Garitezi 2021-12-24 10:56:50 -03:00
parent 41694b7b34
commit dd36bef3bb
No known key found for this signature in database
GPG Key ID: DD279F8152A9B6DD
1 changed files with 0 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
bin/install_upgrade.escript
View File

@ -248,10 +248,6 @@ parse_version(V) when is_list(V) ->
hd(string:tokens(V,"/")). hd(string:tokens(V,"/")).
check_and_install(TargetNode, Vsn) -> check_and_install(TargetNode, Vsn) ->
%% Backup the vm.args. VM args should be unchanged during hot upgrade
%% but we still backup it here
{ok, [[CurrVmArgs]]} = rpc:call(TargetNode, init, get_argument, [vm_args], ?TIMEOUT),
{ok, _} = file:copy(CurrVmArgs, filename:join(["releases", Vsn, "vm.args"])),
%% Backup the sys.config, this will be used when we check and install release %% Backup the sys.config, this will be used when we check and install release
%% NOTE: We cannot backup the old sys.config directly, because the %% NOTE: We cannot backup the old sys.config directly, because the
%% configs for plugins are only in app-envs, not in the old sys.config %% configs for plugins are only in app-envs, not in the old sys.config