RepoMirrors
/
haproxy
mirror of http://git.haproxy.org/git/haproxy.git/
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

MINOR: startup: reintroduce *env keywords support 

setenv/resetenv/presetenv/unsetenv keywords in the configuration modify the
process environment. In case of master-worker and programs we need to restore
the initial process environment before reload, as the configuration could
change in between and newly forked workers and programs should be launched
in the environment corresponded to this new configuration.

To achieve this we backup the initial process environment before the first
configuration read, when 'global' and 'program' sections are read. And then we
clean up master process environment and restore the initial one from the backup
in mworker_reexec().
This commit is contained in:
Valentine Krasnobaeva 2024-10-12 14:44:03 +02:00 committed by Willy Tarreau
parent d5ad92c7aa
commit dc53c37234
1 changed files with 37 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
src/haproxy.c
View File

@ -155,7 +155,7 @@ char *build_features = "";
/* list of config files */
static struct list cfg_cfgfiles = LIST_HEAD_INIT(cfg_cfgfiles);
int pid; /* current process id */
char **init_env;
char **init_env; /* to keep current process env variables backup */
int pidfd = -1; /* FD to keep PID */
static unsigned long stopping_tgroup_mask; /* Thread groups acknowledging stopping */
@ -727,6 +727,24 @@ static void mworker_reexec(int hardreload)
int x_off = 0; /* disable -x by putting -x /dev/null */
mworker_block_signals();
/* restore initial environment (before parsing the config) and do re-exec.
* The initial process environment should be restored here, preceded by
* clean_env(), which do the same job as clearenv().
* Otherwise, after the re-exec we will start the new worker in the
* environment modified by '*env' keywords from the previous configuration,
* i.e. existed before the reload.
*/
if (clean_env() != 0) {
ha_alert("Master encountered a non-recoverable error, exiting.\n");
exit(EXIT_FAILURE);
}
if (restore_env() != 0) {
ha_alert("Master encountered a non-recoverable error, exiting.\n");
exit(EXIT_FAILURE);
}
setenv("HAPROXY_MWORKER_REEXEC", "1", 1);
mworker_cleanup_proc();
@ -3729,6 +3747,12 @@ int main(int argc, char **argv)
}
}
/* backup initial process env, because parse_cfg() could modify it with
* setenv/unsetenv/presetenv/resetenv keywords.
*/
if (backup_env() != 0)
exit(EXIT_FAILURE);
/* parse conf in disovery mode and set modes from config */
read_cfg_in_discovery_mode(argc, argv);
@ -3775,6 +3799,18 @@ int main(int argc, char **argv)
* reset non_global_section_parsed counter for the second
* configuration reading
*/
if (global.mode & MODE_MWORKER) {
if (clean_env() != 0) {
ha_alert("Worker failed to clean its env, exiting.\n");
exit(EXIT_FAILURE);
}
if (restore_env() != 0) {
ha_alert("Worker failed to restore its env, exiting.\n");
exit(EXIT_FAILURE);
}
setenv("HAPROXY_MWORKER", "1", 1);
}
non_global_section_parsed = 0;
if (read_cfg(progname) < 0) {
list_for_each_entry_safe(cfg, cfg_tmp, &cfg_cfgfiles, list) {