summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | Fix a broken specAkshay Karle2014-01-061-4/+3
| | |
| * | Add a spec for showing stderr to live_stream.Akshay Karle2014-01-061-0/+6
| | |
| * | Add stderr to live_stream.Akshay Karle2013-09-263-4/+7
| | |
* | | Merge pull request #45 from opscode/lcg/unix-onlylamont-granquist2014-04-081-2/+2
|\ \ \ | | | | | | | | marking some unix_only tests unix_only
| * | | marking some unix_only tests unix_onlyLamont Granquist2014-04-081-2/+2
|/ / /
* | | 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
| | |
* | | Merge pull request #43 from opscode/adamed/changelog-mixlib25Adam Edwards2014-04-011-0/+1
|\ \ \ | | | | | | | | MIXLIB-25 should be included in change log
| * | | CR feedback: Make changelog description closer to user identifiable symptomsadamedx2014-04-011-1/+1
| | | |
| * | | MIXLIB-25 should be included in change logadamedx2014-04-011-0/+1
|/ / /
* | | Merge pull request #44 from opscode/adamed/mixlib-6Adam Edwards2014-04-011-1/+3
|\ \ \ | | | | | | | | MIXLIB-6: Fedora test break due to symlink
| * | | [MIXLIB-6] Add comment describing why the test was changed, per Dan DeLeo.Julian C. Dunn2014-04-011-0/+2
| | | |
| * | | [MIXLIB-6] Use conditional to determine whether to use /tmp or /bin as a cwd ↵Julian C. Dunn2014-04-011-1/+1
| | | | | | | | | | | | | | | | for the test.
| * | | [MIXLIB-6] Fix test due to /bin being a symlink on Fedora. Use /tmp insteadJulian C. Dunn2014-04-011-1/+1
|/ / /
* | | Bump the version number for RC release.1.4.0.rc.1sersut2014-03-311-1/+1
| | |
* | | Exclude ESRCH test from windowsdanielsdeleo2014-03-311-1/+1
| | |
* | | Handle ESRCH when getting pgid of a zombie on OS Xdanielsdeleo2014-03-313-1/+37
| | |
* | | Bump the version number for RC release.1.4.0.rc.0sersut2014-03-301-1/+1
| | |
* | | Merge pull request #41 from opscode/adamed/mixlib-25Adam Edwards2014-03-272-18/+27
|\ \ \ | | | | | | | | MIXLIB-25: Only close handles that are set to a valid handle value
| * | | 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
|/ / /
* | | Update changelog for GC.disable removaldanielsdeleo2014-03-071-0/+1
| | |
* | | 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.
* | | Add debugging to volatile specsdanielsdeleo2014-03-071-7/+25
| | | | | | | | | | | | | | | | | | Attempting to reproduce this in isolation hasn't worked. Add debug information permanently so we may get a hint of why this test sometimes fails if we see it again.
* | | Merge pull request #38 from opscode/new-policy-filesSerdar Sutay2014-02-272-0/+18
|\ \ \ | | | | | | | | New policy files.
| * | | New policy files.sersut2014-02-272-0/+18
|/ / /
* | | Merge branch 'MIXLIB-18'danielsdeleo2014-01-212-8/+30
|\ \ \
| * | | Adds error? method to check whether the process exited successfully. This ↵Chris Armstrong2014-01-212-8/+30
|/ / / | | | | | | | | | allows the user to use custom error-handling logic. error! now uses error? internally.
* | | Merge branch 'travis-it-up'danielsdeleo2013-12-121-0/+10
|\ \ \ | | | | | | | | | | | | Add the travis.
| * | | Add travis yamldanielsdeleo2013-12-121-0/+10
|/ / /
* | | Merge branch 'track-child-by-pgid'danielsdeleo2013-12-093-12/+103
|\ \ \ | | | | | | | | | | | | | | | | * Closes https://tickets.opscode.com/browse/MIXLIB-24 * Expand timeout process cleanup to include grandchildren.
| * | | Always send KILL to process groupdanielsdeleo2013-12-092-34/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-092-7/+111
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | | Run timeout tests without a subshelldanielsdeleo2013-12-061-3/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | /bin/sh on Ubuntu is dash, which does not support bash's behavior of using exec to run a command (instead of fork) when possible. This means that, when given a single string command, instead of an array of command plus arguments, the process created by fork/exec is dash, and the ruby program is a child process of dash. When shellout signals the child to exit after a timeout, it will signal dash instead of the ruby program. To avoid this issue, the command needs to be given as an Array to avoid creating a subshell.
| * | | Remove -b from default rspec optionsdanielsdeleo2013-12-061-1/+1
|/ / / | | | | | | | | | | | | RSpec seems to be much better at filtering nowadays and the extra lines are a PITA in development.
* | | Merge pull request #32 from opscode/fix-windows-testsSeth Chisamore2013-12-052-6/+12
|\ \ \ | |_|/ |/| | Fix windows tests
| * | Silence `:rubygems` Bundler deprecation notice.Seth Chisamore2013-12-051-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | Message from Bundler before this change: The source :rubygems is deprecated because HTTP requests are insecure. Please change your source to 'https://rubygems.org'; if possible, or 'http://rubygems.org'; if not.
| * | Clean up `jenkins_run_tests.bat`Seth Chisamore2013-12-051-5/+10
|/ / | | | | | | This change ensures the bat file is in sync with what we have in opscode/chef.
* | Bump the version number for release.1.3.0sersut2013-12-031-1/+1
| |
* | Merge branch 'MIXLIB-16'danielsdeleo2013-11-0411-104/+124
|\ \
| * | Add termination info to error messagesdanielsdeleo2013-11-043-2/+6
| | | | | | | | | | | | | | | 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-042-0/+21
| | | | | | | | | | | | | | | If a logger is configured, send `error` level messages about child process termination for timeout violations.
| * | Cleanup control flow in main loopdanielsdeleo2013-11-012-19/+23
| | | | | | | | | | | | | | | removes some complexity that was introduced with the fix to kill timed-out child processes
| * | Force subprocess to exit after timeoutdanielsdeleo2013-11-012-19/+82
| | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | Update mock/stubs to avoid deprecationsdanielsdeleo2013-11-011-2/+2
| | |
| * | Remove cruftdanielsdeleo2013-11-018-74/+2
|/ /
* | Merge pull request #29 from opscode/lcg/MIXLIB-7lamont-granquist2013-10-313-4/+0
|\ \ | | | | | | [MIXLIB-7] remove awesome_print
| * | [MIXLIB-7] remove awesome_printLamont Granquist2013-10-313-4/+0
|/ /
* | Merge pull request #28 from opscode/lcg/ap-wtfbbqlamont-granquist2013-10-311-1/+1
|\ \ | | | | | | remove Associated Press Breaking API gem dep
| * | remove Associated Press Breaking API gem depLamont Granquist2013-10-231-1/+1
| | | | | | | | | | | | - my guess is awesome_print used to be called 'ap'?