summaryrefslogtreecommitdiff
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
* Revert lib/chef/api_client/registration.rb to use API V0 since it doesn't ↵tc/fix-api-client-registrationTyler Cloke2015-12-041-3/+7
| | | | support V1.
* Merge pull request #4196 from chef/lcg/dpkg-multipackageLamont Granquist2015-12-025-64/+209
|\ | | | | multipackage dpkg_package and bonus fixes
| * just use extendLamont Granquist2015-12-022-15/+10
| |
| * dpkg multipackage and bonus fixesLamont Granquist2015-12-025-64/+214
| | | | | | | | | | | | | | | | - multipackages dpkg_package - fixes edge conditions in load-current-resource around purging/removing packages that threw errors before - fixes the ability to purge packages that have been removed - adds a lot of functional tests for dpkg_package
* | Make to_hash include properties with defaults (fixes #4127)jk/4127John Keiser2015-12-021-3/+8
| |
* | Converge actions all together instead of in action :x (fixes #4124)jk/4124John Keiser2015-12-021-34/+15
|/
* Bump version to 12.6.0Salim Alam2015-12-011-1/+1
|
* extract declare_resource to a mixin and extend APILamont Granquist2015-12-013-78/+116
| | | | | | | - extract to its own mixin so it can be included without method_missing and the rest of the resources DSL - add ability to inject different run_context - add ability to create_if_missing into the resource_collection
* Add make_child_entry in ChefFS CookbookSubdirMaxime Brugidou2015-11-251-0/+5
| | | | Following a517fa8a we can't call `child` on cookbook subdirs.
* Merge pull request #4153 from matschaffer/patch-1Thom May2015-11-251-1/+1
|\ | | | | Require ShellOut before Knife::SSH definition
| * Require ShellOut before Knife::SSH definitionMat Schaffer2015-11-111-1/+1
| | | | | | | | | | | | | | | | | | I ran into a NameError when trying to run https://github.com/matschaffer/knife-solo/blob/master/test/knife_bootstrap_test.rb#L14 that I think this should fix. My suspicion is that under normal operation shell_out gets required via the application or platform. And prior to moving config into its own project it was being required in chef/config.rb In my tests since I end up loading knife ssh directly without the surrounding application things don't work out. I'll likely just add a line to my test helper for now but wanted to open this for comment or future improvement.
* | add better documentation to this methodLamont Granquist2015-11-241-10/+13
| | | | | | | | | | it violates our dont-violate-the-new-resource policy for a reason, but is likely still buggy if the resource gets reused.
* | Merge pull request #4200 from chef/jdm/dont-print-pscredJay Mundrawala2015-11-242-5/+22
|\ \ | | | | | | Prevent inspect on PsCredential from printing out plain text password
| * | Prevent inspect on PsCredential from printing out plain text passwordjdm/dont-print-pscredJay Mundrawala2015-11-232-5/+22
| | |
* | | Correcting regex for upstart_stateJustin Seubert2015-11-231-3/+3
| | |
* | | DTRT re: identity_fileJeff Blaine2015-11-232-14/+12
| | |
* | | Migrate to --ssh-identity-file instead of --identity-fileJeff Blaine2015-11-232-3/+13
|/ / | | | | | | | | | | | | | | | | | | | | | | Fixes #3434 Introduces --ssh-identity-file which is more like the other SSH arguments (they all begin with --ssh except this one). Behavior is unchanged. Migrates '-i' over with --ssh-identity-file Retains --identity-file with a deprecation statement.
* | dpkg provider cleanupLamont Granquist2015-11-201-60/+81
| | | | | | | | | | | | | | | | | | | | | | | | | | | | - :update and :install are now treated the same way and throw the same exceptions - :remove and :purge don't require the source at all, so don't do any checking on that - fix some convoluted side-effecty logic in load_current_resource - load_current_resource now correctly gets the dpkg state on :remove and :purge when the file does not exist (pretty sure the old logic did not) - fixed the FIXME about using en_US.UTF-8 (the default for shell_out!) - just use shell_out! to throw exceptions - clean up all the specs and remove all the instance vars from the code
* | Merge pull request #4188 from chef/salam/dsc-resource-fixSalim Alam2015-11-201-4/+9
|\ \ | | | | | | Update dsc_resource to use verbose stream output
| * | Update dsc_resource to use verbose stream outputsalam/dsc-resource-fixSalim Alam2015-11-191-4/+9
| | |
* | | Fix typo in commentGreg Karékinian2015-11-201-1/+1
|/ / | | | | | | | | | | s/vlaue/value/ Obvious fix.
* | Use repo_mode to decide whether to treat /users asJohn Keiser2015-11-181-7/+13
| | | | | | | | a list of users or as invites
* | Properly rescue ENOENT on delete and send DataNotFoundErrorJohn Keiser2015-11-181-6/+10
| | | | | | | | so that callers can 404 when deleting a nonexistent path
* | Fix /acls/environments and /acls/environments/x.jsonJohn Keiser2015-11-181-1/+14
| |
* | Support POST /organizations/NAME/association_requests and POST ↵John Keiser2015-11-181-0/+160
| | | | | | | | /organizations/NAME/users
* | WMF 5 RTM and Win 10 Threshold 2 allow the RefreshMode to be enabled.Steven Murawski2015-11-182-5/+40
| |
* | Add reboot notification to dsc_resourceSteven Murawski2015-11-182-32/+49
| |
* | 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
* | base package provider fixesLamont Granquist2015-11-174-47/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 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-171-0/+1
| | | | | | | | to be the cookbooks (see http://rspec.info/blog/2015/11/rspec-3-4-has-been-released/, Better Failure Source Detection, for more info)
* | Merge pull request #4040 from chef/tm/live_execute_streamingThom May2015-11-126-2/+47
|\ \ | | | | | | Implement live streaming for execute resources
| * | More code reviewtm/live_execute_streamingThom May2015-11-112-10/+3
| | |
| * | respond to review commentsThom May2015-11-092-3/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-093-7/+22
| | |
| * | Implement live streaming for execute resourcesAdam Jacob2015-11-094-1/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-4/+37
|\ \ \ | | | | | | | | Modify remote_file cache_control_data to use sha256 for its name
| * | | 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-123-0/+34
|\ \ \ \ | | | | | | | | | | Adding ksh resource for #3923
| * | | | removing provider require in ksh resource and updating copyright infoNolan Davidson2015-10-071-2/+1
| | | | |
| * | | | Adding ksh resource and specsNolan Davidson2015-10-063-0/+35
| | | | |
* | | | | 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-121-2/+12
|\ \ \ \ \ \ | | | | | | | | | | | | | | 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-2/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-10/+8
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | | Use the initializer to avoid NoMethodError on nil.include?
| * | | | | | Route all tags access through Chef::Node#{tags,tag}Jos Backus2015-10-202-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | | | | | 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.