cmueller/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Don't change to root dir in async_wrapper (#4779)

Browse source 
The daemonizing code here is taken from an ActiveState recipe, which
includes changing to / as a general best practice. While that is
normally true to allow for deleting the directory that the daemon
process started in, in this case it is not relevant as this is not
intended to be an actual long-running daemon.

Issue ansible/ansible#17466
This commit is contained in:
Monty Taylor 2016-09-12 11:31:40 -05:00 committed by Matt Clay
parent b9962d2389
commit e27b7af1e6
1 changed files with 0 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
lib/ansible/modules/utilities/logic/async_wrapper.py
View file

@ -52,7 +52,6 @@ def daemonize_self():
sys.exit("fork #1 failed: %d (%s)\n" % (e.errno, e.strerror)) sys.exit("fork #1 failed: %d (%s)\n" % (e.errno, e.strerror))
# decouple from parent environment # decouple from parent environment
os.chdir("/")
os.setsid() os.setsid()
os.umask(int('022', 8)) os.umask(int('022', 8))