summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* add a comment about the ruby warning level suppressionautoloadmwrock2020-09-081-0/+4
| | | | Signed-off-by: mwrock <matt@mattwrock.com>
* use autoloading for many required gemsmwrock2020-09-0868-157/+220
| | | | Signed-off-by: mwrock <matt@mattwrock.com>
* Bump version to 16.5.35 by Chef Expeditorv16.5.35Chef Expeditor2020-09-087-19/+20
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #10393 from chef/VSingh/data-collector-output-locationsTim Smith2020-09-087-131/+235
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Config validation updatesVivek Singh2020-09-092-29/+28
| | | | | | | | Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
| * Fix chefstyleVSingh/data-collector-output-locationsVivek Singh2020-09-081-1/+1
| | | | | | | | Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
| * Fixes urls configurations catches files optionsVivek Singh2020-09-081-1/+1
| | | | | | | | Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
| * Fixes integration specsVivek Singh2020-09-082-3/+9
| | | | | | | | Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
| * Fixes build failureVivek Singh2020-09-082-1/+2
| | | | | | | | Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
| * Data collector docs minor updatesVivek Singh2020-09-081-2/+3
| | | | | | | | Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
| * Data collector multiple fixesVivek Singh2020-09-086-125/+222
| | | | | | | | | | | | | | - Fix invalid output_locations raise start_time error. - Improve validate_output_locations! Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
* | Bump version to 16.5.34 by Chef Expeditorv16.5.34Chef Expeditor2020-09-087-19/+20
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #10394 from chef/interpolationTim Smith2020-09-0810-15/+15
|\ \ | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Resolve RuboCop Style/RedundantInterpolation warningsTim Smith2020-09-0810-15/+15
|/ / | | | | | | | | | | Resolve the issues that pete pointed out plus a few others Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 16.5.33 by Chef Expeditorv16.5.33Chef Expeditor2020-09-087-19/+20
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #10390 from chef/launchd_restartTim Smith2020-09-081-4/+34
|\ \ | |/ |/| Signed-off-by: Tim Smith <tsmith@chef.io>
| * Minor fixes with a fresh eyelaunchd_restartTim Smith2020-09-051-3/+7
| | | | | | | | | | | | | | Only use launchd create action on the service itself. Enable will trigger restarts which we want to avoid since that will prematurely hang things Make sure we disable the restarter plist as well if someone runs the disable action otherwise any manual plist work will turn the client back on Signed-off-by: Tim Smith <tsmith@chef.io>
| * Improve comments on the restarterTim Smith2020-09-051-0/+5
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Explain why we're sleepingTim Smith2020-09-051-0/+4
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * chef_client_launchd: create a launchd daemon to handle the client restartTim Smith2020-09-051-3/+20
|/ | | | | | | | Launchd doesn't have the concept of a reload aka restart. Instead to update a daemon config you have to unload it and then reload the new plist. That's usually fine (actually just like upstart), but not if Chef is trying to restart itself. Currently if you change the nice level or interval the client will unload its own config which kills the run and it never gets started back up. So to work around this we're going to install a launchd daemon that has a watcher on the chef-client daemon plist file. It's only job to to unload and reload the client. That way when the daemonized chef-client process updates the plist the restarter job will handle the restart. To ensure we're not in the middle of the next recipe we use a chef_sleep resource to sleep a bit post launchd update. I also updated the resource to work better with cinc while I was in here. Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #10387 from chef/debug-windows-habTim Smith2020-09-041-2/+15
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Fix habitat argument passing in Windows test script.debug-windows-habPete Higgins2020-09-041-2/+15
| | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* | Bump version to 16.5.32 by Chef Expeditorv16.5.32Chef Expeditor2020-09-057-19/+20
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #10389 from chef/apply_loggingTim Smith2020-09-041-0/+5
|\ \ | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Add --logfile to chef-apply commandapply_loggingTim Smith2020-09-041-0/+5
|/ / | | | | | | | | | | | | | | | | | | I needed this for testing and it just sorta seems like something we should support. Yes that means people may be using chef-apply with some sort of init system, but if that's something they wanna do then why not just let them roll with that. This just copies the option from the base.rb file and it works fine for me locally. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 16.5.31 by Chef Expeditorv16.5.31Chef Expeditor2020-09-047-19/+20
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #10388 from chef-davin/windows_ad_join-updateTim Smith2020-09-041-0/+7
|\ \ | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Add a :reboot_delay property to the windows_ad_join resourceDavin Taddeo2020-09-041-0/+7
|/ / | | | | | | Signed-off-by: Davin Taddeo <davin@chef.io>
* | Bump version to 16.5.30 by Chef Expeditorv16.5.30Chef Expeditor2020-09-047-19/+20
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #10382 from chef/lcg/fix-nil-deep-mergeLamont Granquist2020-09-042-12/+15
|\ \ | |/ |/| Signed-off-by: Lamont Granquist <lamont@chef.io>
| * Fix nil deep_merginglcg/fix-nil-deep-mergeLamont Granquist2020-09-032-12/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While this behavior was fixed for attributes a long, long time ago it turns out we don't use this deep_merge mixin for constructing the deep merge of attribute precedence levels any more. This bug however surfaces because we still use deep_merge to merge json attributes into normal attributes. That means that it is not possible to pass a nil / null value in through -j and have it override a normal attribute which is pulled back in from the node object on the chef-server. So this fixes that behavior to be consistent with the behavior of attributes that we've had for several years now. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | Bump version to 16.5.29 by Chef Expeditorv16.5.29Chef Expeditor2020-09-047-19/+20
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #10386 from chef/remove_profileTim Smith2020-09-041-4/+5
|\ \ | | | | | | Allow removing profiles in osx_profile on Big Sur
| * | Allow removing profiles in osx_profile on Big SurTim Smith2020-09-041-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | Users may want to cleanup old profiles after upgrading a Catalina box to Big Sur. We should allow for that. It's just installing new profiles where we need to fail hard. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Bump version to 16.5.28 by Chef Expeditorv16.5.28Chef Expeditor2020-09-047-19/+20
| | | | | | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | | Merge pull request #10385 from chef/bumpsTim Smith2020-09-042-12/+12
|\ \ \ | |/ / |/| | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Update all deps to currentTim Smith2020-09-042-12/+12
|/ / | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 16.5.27 by Chef Expeditorv16.5.27Chef Expeditor2020-09-047-19/+20
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #10381 from chef/niceTim Smith2020-09-033-1/+44
|\ \ | |/ |/| Signed-off-by: Tim Smith <tsmith@chef.io>
| * Update kitchen-tests/cookbooks/end_to_end/recipes/linux.rb niceTim Smith2020-09-031-1/+1
| | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io> Co-authored-by: pete higgins <pete@peterhiggins.org>
| * chef_client_systemd_timer: Add the ability to set CPUQuota on the ↵Tim Smith2020-09-033-1/+44
|/ | | | | | | | chef-client unit This allows us to set a quota like 50% on everything handled by the chef-client including all the sub-processes like ohai or inspec. I looked at using systemd's nice functionality since that would give us a unified property for all the *nix processes, but that only applies to the chef-client process itself. This is much more powerful and gives users what user's probably really want which is "chef and all its stuff should not consume all the CPU on my host". Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #10372 from chef/shell-test-cleanupTim Smith2020-09-023-86/+97
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Set correct config value.shell-test-cleanupPete Higgins2020-09-011-1/+1
| | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
| * Cleanup tests to make test setup more consistent.Pete Higgins2020-09-011-41/+34
| | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
| * Move code to where it is used and remove top level constants.Pete Higgins2020-09-013-44/+62
| | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* | Bump version to 16.5.26 by Chef Expeditorv16.5.26Chef Expeditor2020-09-017-19/+20
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #10369 from chef/remove_duplicate_requiresTim Smith2020-09-011-4/+0
|\ \ | |/ |/| Signed-off-by: Tim Smith <tsmith@chef.io>
| * Remove duplicate requires in the Provider classTim Smith2020-09-011-4/+0
|/ | | | | | We're requiring these already at the top of the file Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 16.5.25 by Chef Expeditorv16.5.25Chef Expeditor2020-08-287-19/+20
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #10362 from chef/client_updatesTim Smith2020-08-285-50/+103
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>