summaryrefslogtreecommitdiff
path: root/lib/mixlib/shellout
Commit message (Collapse)AuthorAgeFilesLines
* Block until child stream is emptyryan/issue_2062_12Ryan Cragun2014-10-311-1/+1
|
* Remove child process pipe from open pipes when the stream is closedRyan Cragun2014-10-311-0/+1
|
* Rescue ArgumentError on Marshal.load of buffered child statusRyan Cragun2014-10-311-1/+1
|
* Add buffering to the process status pipeRyan Cragun2014-10-311-7/+16
|
* Bump version number after 2.0.0 branching.Serdar Sutay2014-10-271-1/+1
|
* Merge pull request #57 from ClogenyTechnologies/kd/processgrp-fixlamont-granquist2014-09-111-1/+4
|\ | | | | aix getpgid fails when called from parent and parent and child have different session
| * fix getpgid issue on aixkaustubh-d2014-08-271-1/+4
| |
* | bumping version to 2.0.0.rc.0Lamont Granquist2014-09-051-1/+1
| |
* | remove LC_ALL default environment variableLamont Granquist2014-08-281-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-d2014-08-271-1/+1
| |
* | remove #clean_parent_file_descriptors which is not required now.kaustubh-d2014-08-251-27/+0
| |
* | clean_parent_file_descriptors can be achieve with close_others flag to exec ↵kaustubh-d2014-08-251-1/+1
|/ | | | method, aix removed method was slow.
* Bump version number for RC release.1.6.0.rc.0Serdar Sutay2014-08-081-1/+1
|
* Simplify live_stream vs live_stdout vs live_stderr based on PR commentsMax Lincoln2014-07-172-3/+3
|
* Support separate live stream for stderrMax Lincoln2014-07-172-2/+2
|
* AIX returns Errno::EPERM hereLamont Granquist2014-07-081-1/+1
|
* Correct spelling in spec filesTyler Cipriani2014-05-161-1/+1
|
* Correct spelling of "exceeded"Tyler Cipriani2014-05-151-2/+2
|
* Merge pull request #26 from akshaykarle/masterSerdar Sutay2014-05-132-1/+4
|\ | | | | [#MIXLIB-19] Copy stderr to live_stream along with stdout of the subprocess
| * Add stderr to live_stream.Akshay Karle2013-09-262-1/+4
| |
* | Bump version number post release.sersut2014-04-081-1/+1
| |
* | Bump the version number for release.1.4.01.4-stablesersut2014-04-081-1/+1
| |
* | Bump the version number for RC release.1.4.0.rc.1sersut2014-03-311-1/+1
| |
* | Handle ESRCH when getting pgid of a zombie on OS Xdanielsdeleo2014-03-311-1/+17
| |
* | Bump the version number for RC release.1.4.0.rc.0sersut2014-03-301-1/+1
| |
* | Handle leak fix for leaked token handle, avoid double closeadamedx2014-03-172-16/+25
| |
* | Only close handles that are set to a valid handle valueAdam Edwards2014-03-171-3/+3
| |
* | Retire the GC disable hack for all but ruby 1.8.7danielsdeleo2014-03-071-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 groupdanielsdeleo2013-12-091-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 cleanupdanielsdeleo2013-12-091-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.0sersut2013-12-031-1/+1
| |
* | Add termination info to error messagesdanielsdeleo2013-11-041-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 possibledanielsdeleo2013-11-041-0/+2
| | | | | | | | | | If a logger is configured, send `error` level messages about child process termination for timeout violations.
* | Cleanup control flow in main loopdanielsdeleo2013-11-011-13/+13
| | | | | | | | | | removes some complexity that was introduced with the fix to kill timed-out child processes
* | Force subprocess to exit after timeoutdanielsdeleo2013-11-011-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 ENOENTdanielsdeleo2013-09-111-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 errorsdanielsdeleo2013-09-091-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.0sersut2013-07-191-1/+1
|
* Bump the version number for RC release.1.2.0.rc.1sersut2013-06-241-1/+1
|
* close observed to throw EBADFLamont Granquist2013-06-141-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 candidateadamedx2013-05-021-1/+1
|
* More robust pipe checks and tests.sersut2013-04-171-5/+7
|
* Clean inherited file descriptors from the parent while forking subprocess.sersut2013-04-161-0/+22
|
* Set domain to . if no domain is passedMoser, Kevin2013-03-131-2/+3
|
* Update hash reference to symbolsMoser, Kevin2013-03-131-2/+2
|
* Add option validationMoser, Kevin2013-03-132-0/+14
|
* Update comments for why to use LogonUser and CreateProcessAsUser in some casesMoser, Kevin2013-02-221-1/+4
|
* Fix commentsMoser, Kevin2013-01-151-2/+3
|
* Revert "Bump version to 1.2.0 to package gem locally"Moser, Kevin2013-01-151-1/+1
| | | | This reverts commit 8d4efcd52d7a445769aa7e8b58285ac8fade64ed.
* Bump version to 1.2.0 to package gem locallyMoser, Kevin2013-01-151-1/+1
|