You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficcontrol.apache.org by oren-shemesh <gi...@git.apache.org> on 2017/02/20 12:11:18 UTC
[GitHub] incubator-trafficcontrol pull request #295: [TC-158] daemonize() leaves pare...
GitHub user oren-shemesh opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/295
[TC-158] daemonize() leaves parent process with bad SIGCHLD handler
[TC-158] daemonize() leaves parent process with bad SIGCHLD handler
Renamed daemonize() to fork_and_daemonize() to make it clear that caller should
expect the function to return twice
Use double fork in fork_and_daemonize() so parent SIGCHLD handler is not affected
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/oren-shemesh/incubator-trafficcontrol no-ignore-sigchld
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-trafficcontrol/pull/295.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #295
----
commit 5378ae82610fc75863158136276fed8c4fae5b67
Author: Oren Shemesh <or...@qwilt.com>
Date: 2017-02-08T06:06:55Z
Fixed bad configuration in traffic_ops/app/conf/production/influxdb.conf,
preventing graphs from being shows in the portal, on production systems.
Now this file is consistent with it's sibling in test/ and development/.
commit 209f1a680a5246e56a18717814f50cd8bfac82e2
Author: Oren Shemesh <or...@qwilt.com>
Date: 2017-02-20T10:39:58Z
[TC-158] daemonize() leaves parent process with bad SIGCHLD handler
- Renamed daemonize() to fork_and_daemonize() to make it clear that caller should
expect the function to return twice
- Use double fork so parent SIGCHLD handler is not affected
commit 0af57f9b7d17cd8267a7a94f80f6cbb308776762
Author: Oren Shemesh <or...@qwilt.com>
Date: 2017-02-20T12:01:06Z
Merge remote-tracking branch 'official/master' into no-ignore-sigchld
commit 6897ceee7c07f4d75e4aac8a744e721255d2232d
Author: Oren Shemesh <or...@qwilt.com>
Date: 2017-02-20T12:09:18Z
As part of the fix to TC-158, remove previous fix, which is not needed any more
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-trafficcontrol issue #295: [TC-158] daemonize() leaves parent proc...
Posted by mitchell852 <gi...@git.apache.org>.
Github user mitchell852 commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/295
@elsloo or @dneuman64 - maybe one of you guys feel qualified to review / test this PR. I, unfortunately, do not.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-trafficcontrol pull request #295: [TC-158] daemonize() leaves pare...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/295
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---