summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Correcting re-registration to pass testsad/reregistration-keysapiandy-dufour2015-11-181-1/+1
|
* Spacing style fixandy-dufour2015-11-181-2/+2
|
* Chef-client's registration update function will now talk to the keys API if ↵andy-dufour2015-11-181-0/+9
| | | | necessary
* Merge pull request #4158 from donaldguy/patch-1Thom May2015-11-171-2/+1
|\ | | | | Allow named_run_list to be loaded from config
| * Allow named_run_list to be loaded from configDonald Guy2015-11-111-2/+1
| | | | | | | | | | nil default in application/client.rb shadowed any value from config when executing chef-client (but not chef-shell -z) default in config is set to nil at https://github.com/chef/chef/blob/b0dbe243d469cc36477ba8102b74a8456b6f276d/chef-config/lib/chef-config/config.rb#L363, so otherwise this change is a no-op
* | Merge pull request #4165 from chef/lcg/multipackage-api-improvementLamont Granquist2015-11-177-413/+558
|\ \ | | | | | | Multipackage internal API improvements
| * | add CHANGELOGlcg/multipackage-api-improvementLamont Granquist2015-11-171-0/+1
| | |
| * | base package provider fixesLamont Granquist2015-11-176-413/+557
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * removes the mutation of the new_resource.version * adds package_class_supports_arrays for multipackage providers * cleans up the package resource initializer and sets the package_name correctly through the accessor method. By mutating new_resource.version we were destroying the original intent of what the user was requesting. This is a bug that must be fixed. Generally, subclasses will be able to get the correct information they need through either the version argument they were passed in install_package(name, version) instead, or through their `#target_version_array` method. If this breaks anything then those providers need bugfixes as well (since this is a change to an internal API accessible only to subclassing, and there are backwards compatible and correct ways to get the information, this is *not* a SemVer violating change). See the fix I made to the OpenBSD provider to preserve the same semantics, avoid using new_resource.version where it was inappropriate, and make the code more symmetrical where before substantially different looking code in install_package and remove_package had exactly the same ultimate effect. The package_class_supports_arrays 'DSL' for writing multipackage providers coerces the arguments to the virtual methods (e.g. install_package) into arrays, even if the user is only requesting a single package install. This removes most of the `is_a?(Array)` checks from the implementation subclasses. The cleanup of the initializer is similarly necessary so that we can use Chef 12.5 coercions to make the package_name and version arguments accept string, but always be Arrays. This should eliminate the rest of the need to write `is_a?(Array)` code all over the package provider implementations.
* | Restore rspec 3.4 by setting project_source_dirsjk/rspec-3.4John Keiser2015-11-173-1/+6
| | | | | | | | to be the cookbooks (see http://rspec.info/blog/2015/11/rspec-3-4-has-been-released/, Better Failure Source Detection, for more info)
* | Raise an error when a command failsJohn Keiser2015-11-172-14/+20
| |
* | Remove bundler cache so external tests will runJohn Keiser2015-11-171-0/+9
| | | | | | | | (the --path bundle/vendor screws up our ability to run bundle install)
* | Merge pull request #4177 from chef/lcg/pin-rspecLamont Granquist2015-11-161-1/+1
|\ \ | | | | | | rspec 3.4.0 broke master
| * | rspec 3.4.0 broke masterlcg/pin-rspecLamont Granquist2015-11-161-1/+1
|/ / | | | | | | pinning to 3.3.x as a temp fix
* | remove repeated first line in 12.5.1 changelogJulian C. Dunn2015-11-141-5/+0
| |
* | Merge pull request #4040 from chef/tm/live_execute_streamingThom May2015-11-1211-38/+153
|\ \ | | | | | | Implement live streaming for execute resources
| * | More code reviewtm/live_execute_streamingThom May2015-11-112-10/+3
| | |
| * | respond to review commentsThom May2015-11-096-71/+94
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-094-9/+24
| | |
| * | Implement live streaming for execute resourcesAdam Jacob2015-11-098-27/+111
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-122-40/+99
|\ \ \ | | | | | | | | 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
| | | |
| * | | allow generate_checksum to use StringIOJay Mundrawala2015-09-281-1/+5
| | | |
| * | | Modify remote_file cache_control_data to use sha256 for its nameJay Mundrawala2015-09-281-4/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We want to support a fips mode, and doing MD5 with fips mode enabled is wrong/hard. In this case, the type of checksum does not matter, so let's just use sha256 since fips mode will be happy with that. For cases where the cache control data exists, we update it to provide a seamless upgrade.
* | | | Merge pull request #4014 from nsdavidson/issue-3923Thom May2015-11-124-0/+74
|\ \ \ \ | | | | | | | | | | Adding ksh resource for #3923
| * | | | updating copyright statment in ksh_spec.rbNolan Davidson2015-10-131-1/+1
| | | | |
| * | | | removing provider require in ksh resource and updating copyright infoNolan Davidson2015-10-071-2/+1
| | | | |
| * | | | Adding ksh resource and specsNolan Davidson2015-10-064-0/+75
| | | | |
* | | | | Merge pull request #4101 from johnbellone/masterThom May2015-11-121-0/+2
|\ \ \ \ \ | | | | | | | | | | | | package/solaris: Adds alias method for upgrade_package.
| * | | | | package/solaris: Adds alias method for upgrade_package.John Bellone2015-10-231-0/+2
| | | | | | | | | | | | | | | | | | This follows the same convention that the [AIX package provider](https://github.com/johnbellone/chef/blob/master/lib/chef/provider/package/aix.rb#L122) has done and adds an alias_method.
* | | | | | Merge pull request #4142 from tas50/masterThom May2015-11-122-2/+18
|\ \ \ \ \ \ | | | | | | | | | | | | | | 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-052-2/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-126-19/+21
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | | Use the initializer to avoid NoMethodError on nil.include?
| * | | | | | Route all tags access through Chef::Node#{tags,tag}Jos Backus2015-10-204-11/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This should avoid node tags not being initialized properly to an empty Array.
| * | | | | | Stop checking for nil hereJos Backus2015-10-201-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | node.tags should never be nil, always an Array. node.tags == nil would be a bug.
| * | | | | | 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
| | | | | | |
| * | | | | | Use the initializer to avoid NoMethodError on nil.include?Jos Backus2015-10-161-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Rename the argument so as to not collide with the tags method.
* | | | | | | Update DOC_CHANGES.mdjamescott2015-11-101-0/+1
| | | | | | | | | | | | | | | | | | | | | edits
* | | | | | | Update DOC_CHANGES.mdjamescott2015-11-101-0/+8
| | | | | | | | | | | | | | | | | | | | | add link to current/working release_notes page for 12.6 on docs.chef.io for metadata.rb settings and two resources
* | | | | | | Merge pull request #4138 from chef/jk/cleanupSteven Murawski2015-11-100-0/+0
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Minor tree cleanup I noticed
| * | | | | | | Remove now-unused function (already implemented by Chef::Resource)jk/cleanupJohn Keiser2015-11-041-4/+0
| | | | | | | |
| * | | | | | | Include pedant.gemfile.lock in .gitignoreJohn Keiser2015-11-041-1/+1
| | | | | | | |
* | | | | | | | Merge pull request #4120 from robbkidd/correct-capitalization-of-a-maintainerThom May2015-11-101-3/+3
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Correct capitalization of GitHub username of a maintainer
| * | | | | | | | Correct capitalization of GitHub username of a maintainerRobb Kidd2015-10-291-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Supermarket uses this file to track maintainers and is currently case-sensitive.
* | | | | | | | | Merge pull request #4151 from chef/logging_redux12.5.1Matt Wrock2015-11-092-10/+11
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ |/| | | | | | | | fix log location resolution in windows service
| * | | | | | | | fix log location resolution in windows serviceMatt Wrock2015-11-092-10/+11
|/ / / / / / / /
* | | | | | | | Remove now-unused function (already implemented by Chef::Resource)John Keiser2015-11-061-4/+0
| | | | | | | |
* | | | | | | | Include pedant.gemfile.lock in .gitignoreJohn Keiser2015-11-061-1/+1
| | | | | | | |
* | | | | | | | Merge pull request #4137 from chef/btm/knife-backtraceBryan McLellan2015-11-062-2/+15
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | 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-062-2/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.