Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Block until child stream is emptyryan/issue_2062_12 | Ryan Cragun | 2014-10-31 | 1 | -1/+1 |
| | |||||
* | Remove child process pipe from open pipes when the stream is closed | Ryan Cragun | 2014-10-31 | 1 | -0/+1 |
| | |||||
* | Rescue ArgumentError on Marshal.load of buffered child status | Ryan Cragun | 2014-10-31 | 1 | -1/+1 |
| | |||||
* | Add buffering to the process status pipe | Ryan Cragun | 2014-10-31 | 1 | -7/+16 |
| | |||||
* | Bump version number after 2.0.0 branching. | Serdar Sutay | 2014-10-27 | 1 | -1/+1 |
| | |||||
* | Merge pull request #57 from ClogenyTechnologies/kd/processgrp-fix | lamont-granquist | 2014-09-11 | 1 | -1/+4 |
|\ | | | | | aix getpgid fails when called from parent and parent and child have different session | ||||
| * | fix getpgid issue on aix | kaustubh-d | 2014-08-27 | 1 | -1/+4 |
| | | |||||
* | | bumping version to 2.0.0.rc.0 | Lamont Granquist | 2014-09-05 | 1 | -1/+1 |
| | | |||||
* | | remove LC_ALL default environment variable | Lamont Granquist | 2014-08-28 | 1 | -1/+0 |
| | | | | | | | | now that we no longer support ruby 1.8.7 its time to retire this | ||||
* | | aix does not allow resetting real uid once the euid is changed to non-root. | kaustubh-d | 2014-08-27 | 1 | -1/+1 |
| | | |||||
* | | remove #clean_parent_file_descriptors which is not required now. | kaustubh-d | 2014-08-25 | 1 | -27/+0 |
| | | |||||
* | | clean_parent_file_descriptors can be achieve with close_others flag to exec ↵ | kaustubh-d | 2014-08-25 | 1 | -1/+1 |
|/ | | | | method, aix removed method was slow. | ||||
* | Bump version number for RC release.1.6.0.rc.0 | Serdar Sutay | 2014-08-08 | 1 | -1/+1 |
| | |||||
* | Simplify live_stream vs live_stdout vs live_stderr based on PR comments | Max Lincoln | 2014-07-17 | 2 | -3/+3 |
| | |||||
* | Support separate live stream for stderr | Max Lincoln | 2014-07-17 | 2 | -2/+2 |
| | |||||
* | AIX returns Errno::EPERM here | Lamont Granquist | 2014-07-08 | 1 | -1/+1 |
| | |||||
* | Correct spelling in spec files | Tyler Cipriani | 2014-05-16 | 1 | -1/+1 |
| | |||||
* | Correct spelling of "exceeded" | Tyler Cipriani | 2014-05-15 | 1 | -2/+2 |
| | |||||
* | Merge pull request #26 from akshaykarle/master | Serdar Sutay | 2014-05-13 | 2 | -1/+4 |
|\ | | | | | [#MIXLIB-19] Copy stderr to live_stream along with stdout of the subprocess | ||||
| * | Add stderr to live_stream. | Akshay Karle | 2013-09-26 | 2 | -1/+4 |
| | | |||||
* | | Bump version number post release. | sersut | 2014-04-08 | 1 | -1/+1 |
| | | |||||
* | | Bump the version number for release.1.4.01.4-stable | sersut | 2014-04-08 | 1 | -1/+1 |
| | | |||||
* | | Bump the version number for RC release.1.4.0.rc.1 | sersut | 2014-03-31 | 1 | -1/+1 |
| | | |||||
* | | Handle ESRCH when getting pgid of a zombie on OS X | danielsdeleo | 2014-03-31 | 1 | -1/+17 |
| | | |||||
* | | Bump the version number for RC release.1.4.0.rc.0 | sersut | 2014-03-30 | 1 | -1/+1 |
| | | |||||
* | | Handle leak fix for leaked token handle, avoid double close | adamedx | 2014-03-17 | 2 | -16/+25 |
| | | |||||
* | | Only close handles that are set to a valid handle value | Adam Edwards | 2014-03-17 | 1 | -3/+3 |
| | | |||||
* | | Retire the GC disable hack for all but ruby 1.8.7 | danielsdeleo | 2014-03-07 | 1 | -4/+9 |
| | | | | | | | | | | | | | | | | Originally we needed this hack for 1.8.6 and 1.8.7, which were the most common versions in use at the time. 1.8.6 is dead so we don't need to account for it any more, which leaves 1.8.7 as the only case we need to care about for this hack. On Ruby 1.9 and greater this hack isn't needed and causes memory bloat for very long running shell commands. | ||||
* | | Always send KILL to process group | danielsdeleo | 2013-12-09 | 1 | -32/+10 |
| | | | | | | | | | | | | | | | | | | There is not a good way to reliably detect whether all grandchildren have exited after sending a TERM to the process group; if the child has correctly exited, grandchildren (which may be unresponsive) will have been adopted by init (ppid == 1) so the parent cannot wait() on them. To ensure no stuck grandchildren are left, send a KILL to the process group after allowing the processes time to clean up. | ||||
* | | Run commands with unique pgid to improve timeout cleanup | danielsdeleo | 2013-12-09 | 1 | -6/+59 |
| | | | | | | | | | | | | | | To ensure that all child processes are properly signaled to exit before forcibly killing them, use `setsid()` in the child process to give the child (and any grandchildren it might create) a new and unique process group. If the command times out, kill and reap the entire process group. | ||||
* | | Bump the version number for release.1.3.0 | sersut | 2013-12-03 | 1 | -1/+1 |
| | | |||||
* | | Add termination info to error messages | danielsdeleo | 2013-11-04 | 1 | -1/+3 |
| | | | | | | | | | | If no logger is configured, we still want to inform the user that the process was killed (and how). | ||||
* | | Add logging about timed-out cmd termination when possible | danielsdeleo | 2013-11-04 | 1 | -0/+2 |
| | | | | | | | | | | If a logger is configured, send `error` level messages about child process termination for timeout violations. | ||||
* | | Cleanup control flow in main loop | danielsdeleo | 2013-11-01 | 1 | -13/+13 |
| | | | | | | | | | | removes some complexity that was introduced with the fix to kill timed-out child processes | ||||
* | | Force subprocess to exit after timeout | danielsdeleo | 2013-11-01 | 1 | -17/+52 |
|/ | | | | | | | | Fixes MIXLIB-16. This issue is particularly prominent when yum/rpm commands go off the deep end repeatedly, but affects any case where a process takes longer than the timeout to complete. | ||||
* | Use blocking waitpid only on ENOENT | danielsdeleo | 2013-09-11 | 1 | -2/+10 |
| | | | | | | | | This is a compromise that prevents us from leaving zombie processes around after a failed exec raises ENOENT, but prevents us from blocking (possibly forever) on timed-out commands. The latter case can cause zombies, so this is not optimal for all cases. See MIXLIB-16 for a proposed enhancement to solve that case. | ||||
* | Wait for child process to die in case of critical errors | danielsdeleo | 2013-09-09 | 1 | -1/+1 |
| | | | | | | | | | | | | Ported the regression test from OHAI-455 to mixlib-shellout, which revealed a race condition in the way shellout reaps the child after a failed exec (most commonly caused by attempting to run a command that doesn't exist). The call to waitpid2 used WNOHANG to avoid hanging indefinitely if an error was caused but the child process was still alive, but this results in the child process not getting reaped if it exits after the call to waitpid2. In a single run of the stress test, this occurred 94/100 times, so it is very likely that mixlib-shellout will leak zombies for the ENOENT case without this change. | ||||
* | Release 1.2.0.1.2.0 | sersut | 2013-07-19 | 1 | -1/+1 |
| | |||||
* | Bump the version number for RC release.1.2.0.rc.1 | sersut | 2013-06-24 | 1 | -1/+1 |
| | |||||
* | close observed to throw EBADF | Lamont Granquist | 2013-06-14 | 1 | -2/+5 |
| | | | | | | - Unsure how File.for_fd() succeeds and then then #close on the File throws this, but it must be racing with something. I don't think we care. | ||||
* | Update version number for release candidate | adamedx | 2013-05-02 | 1 | -1/+1 |
| | |||||
* | More robust pipe checks and tests. | sersut | 2013-04-17 | 1 | -5/+7 |
| | |||||
* | Clean inherited file descriptors from the parent while forking subprocess. | sersut | 2013-04-16 | 1 | -0/+22 |
| | |||||
* | Set domain to . if no domain is passed | Moser, Kevin | 2013-03-13 | 1 | -2/+3 |
| | |||||
* | Update hash reference to symbols | Moser, Kevin | 2013-03-13 | 1 | -2/+2 |
| | |||||
* | Add option validation | Moser, Kevin | 2013-03-13 | 2 | -0/+14 |
| | |||||
* | Update comments for why to use LogonUser and CreateProcessAsUser in some cases | Moser, Kevin | 2013-02-22 | 1 | -1/+4 |
| | |||||
* | Fix comments | Moser, Kevin | 2013-01-15 | 1 | -2/+3 |
| | |||||
* | Revert "Bump version to 1.2.0 to package gem locally" | Moser, Kevin | 2013-01-15 | 1 | -1/+1 |
| | | | | This reverts commit 8d4efcd52d7a445769aa7e8b58285ac8fade64ed. | ||||
* | Bump version to 1.2.0 to package gem locally | Moser, Kevin | 2013-01-15 | 1 | -1/+1 |
| |