Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

SIGHUP kills process instead of reloading logging.json #7541

Closed
heifner opened this issue Jun 27, 2019 · 1 comment
Closed

SIGHUP kills process instead of reloading logging.json #7541

heifner opened this issue Jun 27, 2019 · 1 comment
Assignees
@heifner
Labels
bug

Comments

@heifner
Copy link
Contributor

heifner commented Jun 27, 2019

#6791 was supposed to add the ability to reload the logging.json on a SIGHUP. It appears that SIGHUP signal_set is never setup via start_sighup_handler since start_sighup_handler is never called so SIGHUP kills the process.
@heifner heifner added the bug label Jun 27, 2019
@heifner heifner self-assigned this Jun 27, 2019
This was referenced Jun 27, 2019
Merged
Merged
Merged
@heifner
Copy link
Contributor Author

heifner commented Jul 1, 2019

Released in 1.8

@heifner heifner closed this as completed Jul 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@heifner heifner

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@heifner