Kill commands on SIGHUP

[leo-buneev]

Adds 'SIGHUP' to list of signals that trigger killing of child processes.

[coveralls]

Pull Request Test Coverage Report for Build 349

• 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 99.178%

---

Totals
Change from base Build 347:	0.0%
Covered Lines:	239
Relevant Lines:	239

---

💛 - Coveralls

[coveralls]

Pull Request Test Coverage Report for Build 348

• 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 99.178%

---

Totals
Change from base Build 347:	0.0%
Covered Lines:	239
Relevant Lines:	239

---

💛 - Coveralls

[leo-buneev]

During testing I've discovered that my particular issue was not caused by this.
However, I still think that adding SIGHUP to list of signlas is useful - up to you if you want to merge it.

[gustavohenke]

During testing I've discovered that my particular issue was not caused by this.

Fair enough.
I agree that this is useful nonetheless.

[gustavohenke]

Landed on v5.0.0!

[cben]

I added logging to tree-kill and it seems it doesn't know what's SIGHUP?

[2] yarn insights-proxy exited with code SIGINT
--> Sending SIGTERM to other processes..
[1] mockdata/mockserver.py exited with code 1
--> Sending SIGTERM to other processes..
tree-kill: sending SIGINT to process 19857
tree-kill: sending SIGINT to process 19856
tree-kill: sending SIGINT to process 19855
tree-kill: sending undefined to process 19856
~/u/uhc-portal [130] $ tree-kill: sending undefined to process 19855
tree-kill: sending undefined to process 19855