summaryrefslogtreecommitdiff
path: root/spec/unit
Commit message (Collapse)AuthorAgeFilesLines
* fix spacingsmurawski/dsc_rebootsSteven Murawski2015-11-172-10/+7
|
* Add reboot notification to dsc_resourceSteven Murawski2015-11-172-0/+26
|
* Merge pull request #4040 from chef/tm/live_execute_streamingThom May2015-11-122-36/+94
|\ | | | | Implement live streaming for execute resources
| * respond to review commentsThom May2015-11-091-64/+70
| | | | | | | | | | | | | | | | | | | | the logic is now: if the resource is not sensitive, and if it's explicitly requested to be streamed or if the log level is info or debug, then we'll consider streaming it. If we're configured to send the output to the events stream, we'll do so. Otherwise, if we're not daemonized and have a TTY, we'll go to STDOUT
| * Implement code review comments and fix failing testsThom May2015-11-091-2/+2
| |
| * Implement live streaming for execute resourcesAdam Jacob2015-11-092-26/+78
| | | | | | | | | | | | | | | | | | | | This brings live streaming of execute resource output to the output formatters. It also adds a mechanism for checking to see if an output formatter is in use through the event dispatch system. It adds a new configuration option, "always_stream_execute", which does what it says on the tin.
* | Merge pull request #3991 from chef/jdm/remote-file-sha2Thom May2015-11-121-36/+62
|\ \ | | | | | | Modify remote_file cache_control_data to use sha256 for its name
| * | Update cache_control_data_ specs to use sha256 for path namejdm/remote-file-sha2Jay Mundrawala2015-09-281-36/+62
| | |
* | | Merge pull request #4014 from nsdavidson/issue-3923Thom May2015-11-121-0/+40
|\ \ \ | | | | | | | | Adding ksh resource for #3923
| * | | updating copyright statment in ksh_spec.rbNolan Davidson2015-10-131-1/+1
| | | |
| * | | Adding ksh resource and specsNolan Davidson2015-10-061-0/+40
| | | |
* | | | Merge pull request #4142 from tas50/masterThom May2015-11-121-0/+6
|\ \ \ \ | | | | | | | | | | Use the proper python interpretor for yum-dump.py on Fedora 21+
| * | | | Use the proper python interpretor for yum-dump.py on Fedora 21+Tim Smith2015-11-051-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fedora 21+ use dnf as the primary package manager. Lamont added code in 12.5 to allow for a yum compat mode. This doesn't entirely work though as we need yum-dump.py to correctly run. We were parsing the shabang in the yum binary to find the path to python. On a dnf system the yum binary is a bash script though so we were trying to run yum-dump.py using bash which obviously fails. I added a fallback method to use python if the shebang parsing returns bash. With this in place AND the yum package installed you can use the package resource on Fedora.
* | | | | Merge pull request #4064 from josb/masterThom May2015-11-123-9/+13
|\ \ \ \ \ | | | | | | | | | | | | Use the initializer to avoid NoMethodError on nil.include?
| * | | | | Route all tags access through Chef::Node#{tags,tag}Jos Backus2015-10-202-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This should avoid node tags not being initialized properly to an empty Array.
| * | | | | Stop initializing node tags hereJos Backus2015-10-202-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | This should be done in one place (Chef::Node#tags).
| * | | | | Add a test for correct tags initializationJos Backus2015-10-201-0/+4
| | | | | |
* | | | | | fix log location resolution in windows serviceMatt Wrock2015-11-091-3/+4
| |_|_|_|/ |/| | | |
* | | | | Merge pull request #4137 from chef/btm/knife-backtraceBryan McLellan2015-11-061-1/+14
|\ \ \ \ \ | | | | | | | | | | | | don't squash Chef::Config[:verbosity] on subsequent instances of Chef::Knife::Bootstrap
| * | | | | Avoid squashing/humanizing the exception when config[:verbosity] == 2btm/knife-backtraceBryan McLellan2015-11-061-1/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We create an instance of Chef::Knife::Bootstrap in knife-ec2, but we've already processed ARGV so we substitute and process an empty argv. config[:verbosity] comes from Chef::Application::Knife. Normally we merge those options with the ones from the knife subcommand in Chef::Knife.run. Since we don't in this case, we don't set it to the default of 0 that is specified there, leaving it nil. We set Chef::Config[:verbosity] to config[:verbosity] in Chef::Knife#initialize which now becomes nil. This change makes it so we do not update Chef::Config[:verbosity] if config[:verbosity] is nil, so that later when we rescue we don't call humanize_exception if Chef::Config[:verbosity] is 2.
* | | | | | Handle nil run list option in knife bootstrapnil-run-list-bootstrapdanielsdeleo2015-11-051-0/+15
|/ / / / /
* | | | | Fixed knife_spec unit testJay Mundrawala2015-11-051-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | This is failing on both my and btm's machine. No idea how it passes in other places.
* | | | | Merge pull request #3836 from nathwill/svc-helpers-altThom May2015-11-051-30/+6
|\ \ \ \ \ | |_|/ / / |/| | | | simplify service helpers
| * | | | fix specsNathan Williams2015-10-231-1/+1
| | | | |
| * | | | i have no idea what is happeningNathan Williams2015-10-231-30/+6
| | | | |
* | | | | windows service uses log fileMatt Wrock2015-11-051-46/+82
| | | | | | | | | | | | | | | | | | | | location from config if none is given on commandline
* | | | | Add tests for Chef::HTTP retry behaviordanielsdeleo2015-11-041-1/+103
| | | | |
* | | | | Improve detection of ChefFS-based commands in `knife rehash`ssd/rehash-cheffsSteven Danna2015-10-291-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ChefFS-based commands have a superclass (Chef::ChefFS::Knife) which defines its own inherited method that calls super. This breaks our detection of where the subcommand is defined since the file with the definition is no longer at the top of the call stack. This commit special-cases subclasses with a superclass of Chef::ChefFS::Knife to account for this. Fixes #4089
* | | | | Make Property.derive create derived properties of the same typejk/custom_propertiesJohn Keiser2015-10-281-0/+48
| | | | |
* | | | | Updated --no-color unit tests based on feedback from btm and KartikTyler Fitch2015-10-281-11/+11
| | | | |
* | | | | Update knife bootstrap command to honor --no-color flag in chef-client run ↵Tyler Fitch2015-10-281-4/+11
| | | | | | | | | | | | | | | | | | | | that is part of the bootstrap process.
* | | | | add YARD docsLamont Granquist2015-10-261-1/+1
| | | | |
* | | | | stub #validate! methodLamont Granquist2015-10-261-0/+2
| | | | |
* | | | | fix some busted specsLamont Granquist2015-10-261-1/+1
| | | | |
* | | | | improve error message, add more testsLamont Granquist2015-10-261-1/+51
| | | | |
* | | | | extend metadata to include chef_version and ohai_versionLamont Granquist2015-10-261-1/+71
| | | | |
* | | | | Use SHA256 instead for registry_key when data is not displayableJay Mundrawala2015-10-241-2/+2
| | | | |
* | | | | fix tests for my changeAubrey Holland2015-10-241-1/+1
| | | | |
* | | | | We shouldn't be subclassing Struct.new - if the file is loaded twice we get ↵tyler-ball2015-10-241-45/+44
| | | | | | | | | | | | | | | | | | | | a superclass mismatch error
* | | | | write spec for --json-attribute-file and move configration under render_templatesawanoboly2015-10-241-6/+33
| | | | |
* | | | | Consider only knife configuration for chef environment in bootstrap contextEvan Gilman2015-10-241-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Setting these values in Chef::Config is being deprecated in favor of using the knife config directly. https://github.com/chef/chef/issues/3946
* | | | | Remove duplicate 'Accept' header in specJulian C. Dunn2015-10-241-2/+2
| | | | |
* | | | | add spec for stdout/stderr suppresion in error of sensitive execute resourceRanjib Dey2015-10-241-0/+9
| | | | |
* | | | | Adjust tests per BTMJeff Blaine2015-10-241-8/+8
| | | | |
* | | | | Changes --hide-healthy to --hide-by-mins MINSJeff Blaine2015-10-241-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #3679 Reasoning: The definition of "healthy" is overloaded in the old form (--hide-healthy) to be "a host that has run chef". The code makes no check to determine if the chef run was successful, so it has the capability to provide false positives for "health". Just because a node object was saved with ohai_time set doesn't mean the Chef run was successful. There are exception handlers like lastrun that intentionally save the node object on exception. The previous 1 hour hardcoded time was totally arbitrary. Perhaps "healthy" to others means "nodes that have run chef in the last 4 hours" (or 30 minutes, or 3 days...).
* | | | | Define == for node objectsDavid Bresson2015-10-241-0/+8
| | | | |
* | | | | provider/user/dscl: Set "comment" default valueMikhail Zholobov2015-10-241-0/+7
|/ / / / | | | | | | | | | | | | | | | | "comment" attribute should be defined to create user account properly on OS X. Otherwise, the RealName will be empty.
* | | | provider/user/dscl: Set default gid to 20Mikhail Zholobov2015-10-231-0/+7
| | | | | | | | | | | | | | | | | | | | gid should always be defined to create user account properly on OS X. "staff" (gid 20) is a default group for regular user accounts.
* | | | overhaul solaris SMF service providerlcg/3187Dave Eddy2015-10-231-39/+110
| | | |
* | | | Merge pull request #4079 from chef/service_loggerMatt Wrock2015-10-231-2/+10
|\ \ \ \ | | | | | | | | | | add logger to windows service shellout