summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* remove errant debugginglcg/nil-sets-defaultLamont Granquist2018-03-221-1/+0
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Setting nil to properties with implicit nil sets default valueLamont Granquist2018-03-223-18/+65
| | | | | | | | | | | | | | | | | | | | | | - This makes converting core resources to properties safer - This makes it easier to apply wrapping properties to subresources property :foo, String, default: "foo" This is where the change lies, and writing a nil here will now actually write a "foo" to the variable. property :foo, [ String, nil ], default: "foo" This is unchanged. Writing nil writes nil. property :foo, String Technically this is changed, since it writes the default value, but since nil.equal?(nil) in a very deep way no behavior changes. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Bump version to 14.0.154 by Expeditorv14.0.154Chef Expeditor2018-03-225-11/+12
|
* Merge pull request #7025 from chef/lcg/remove-dead-codeLamont Granquist2018-03-211-25/+0
|\ | | | | remove dead code from property declaration
| * remove dead code from property declarationlcg/remove-dead-codeLamont Granquist2018-03-211-25/+0
|/ | | | | | | | | | | this is effectively dead because of forcing new_resource.thingy i stuck a hard `raise` in here and we have no tests around this any more. the conditional also just makes my head hurt looking at it. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Bump version to 14.0.153 by Expeditorv14.0.153Chef Expeditor2018-03-215-11/+12
|
* Merge pull request #7023 from chef/lcg/rubocop-fixesLamont Granquist2018-03-2112-30/+30
|\ | | | | rubocop fixes from engine bump to 0.54.0
| * rubocop fixes from engine bump to 0.54.0lcg/rubocop-fixesLamont Granquist2018-03-2112-30/+30
| | | | | | | | | | | | some cops got a little stricter. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | Bump version to 14.0.152 by Expeditorv14.0.152Chef Expeditor2018-03-215-11/+12
| |
* | Merge pull request #7016 from chef/tm/chef_guidThom May2018-03-219-2/+99
|\ \ | | | | | | Save the node's UUID as an attribute
| * | Save the node's UUID as an attributetm/chef_guidThom May2018-03-219-2/+99
| | | | | | | | | | | | | | | | | | | | | We generate the UUID as part of the data collector report, but we didn't make that available to the node or the chef server otherwise. Signed-off-by: Thom May <thom@chef.io>
* | | Bump version to 14.0.151 by Expeditorv14.0.151Chef Expeditor2018-03-215-11/+12
| | |
* | | Merge pull request #7015 from chef/windows_feature_fixTim Smith2018-03-211-5/+15
|\ \ \ | | | | | | | | Support installing removed windows features from source
| * | | Support installing removed windows features from sourcewindows_feature_fixTim Smith2018-03-201-5/+15
| | |/ | |/| | | | | | | | | | | | | This issue was reported on the cookbook after refactoring the resource. The issue is that we have a method to determine the features to install given a list of features we want to install. It wasn't taking into account that a removed feature can be installed if the user specifies a source. Now it properly takes that into account and should build the correct array of features to install. It also builds the install command in a cleaner way that avoids funky extra spaces at the end of the command. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Bump version to 14.0.150 by Expeditorv14.0.150Chef Expeditor2018-03-215-11/+12
| | |
* | | Merge pull request #7019 from chef/pagefile_docsTim Smith2018-03-211-2/+2
|\ \ \ | |/ / |/| | Pagefile sizes are in megabytes not bytes
| * | Pagefile sizes are in megabytes not bytespagefile_docsTim Smith2018-03-211-2/+2
|/ / | | | | | | | | | | @s1mark reported this in the Windows cookbook Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 14.0.149 by Expeditorv14.0.149Chef Expeditor2018-03-205-11/+12
| |
* | Merge pull request #6661 from chef/lcg/package_class_perf_tweakLamont Granquist2018-03-201-9/+12
|\ \ | | | | | | memoize some work in the package class
| * | memoize some work in the package classLamont Granquist2018-03-201-9/+12
| | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | Bump version to 14.0.148 by Expeditorv14.0.148Chef Expeditor2018-03-205-11/+12
| | |
* | | Merge pull request #7005 from chef/macos_build_essentialTim Smith2018-03-201-15/+30
|\ \ \ | | | | | | | | Add missing installed logic for macos in build_essential
| * | | Add commentTim Smith2018-03-201-0/+1
| | | | | | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | | Simplify the if installed logic for macosTim Smith2018-03-201-4/+1
| | | | | | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | | Remove rubocop commentTim Smith2018-03-201-1/+0
| | | | | | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | | Add missing installed logic for macos in build_essentialTim Smith2018-03-201-15/+33
| |/ / | | | | | | | | | | | | | | | | | | | | | Add the missing installed? check that makes this fast. Also use tail instead of head in the execute so we install the latest not the oldest xcode release if more than 1 is available on the system (upgrades). Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Bump version to 14.0.147 by Expeditorv14.0.147Chef Expeditor2018-03-205-11/+12
| | |
* | | Merge pull request #6963 from chef/macos_resourcesTim Smith2018-03-209-8/+471
|\ \ \ | |/ / |/| | Add dmg_package, homebrew_cask, and homebrew_tap resources
| * | Update to the code from the homebrew 9.0 cookbookTim Smith2018-03-206-56/+180
| | | | | | | | | | | | | | | | | | This converts the Chef resources to shell_out and adds new properties for dealing with edge case installations. It also adds validation messaging / testing and lazy evals the stuff that uses the mixin. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Add new find_homebrew_username method in the Homebrew mixinTim Smith2018-03-201-3/+13
| | | | | | | | | | | | | | | | | | We need the username about a billion times in both providers. The current helper grabs the UID. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Add dmg_package, homebrew_cask, and homebrew_tap resourcesTim Smith2018-03-205-0/+329
|/ / | | | | | | | | | | Pull in resources from the dmg and homebrew cookbooks Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 14.0.146 by Expeditorv14.0.146Chef Expeditor2018-03-205-11/+12
| |
* | Merge pull request #6998 from chef/cleanupTim Smith2018-03-202-9/+45
|\ \ | | | | | | Remove Chef 12-isms from the apt_repository resource
| * | Remove Chef 12-isms from the apt_repository resourceTim Smith2018-03-202-9/+45
|/ / | | | | | | | | | | We set a lot of nil defaults and did coersions here that are no longer necessary. Since we did accept nil in a lot of these properties at one point we need to keep it so I wrote a note to future me about that. I also added some specs for the remaining coersion and the defaults. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 14.0.145 by Expeditorv14.0.145Chef Expeditor2018-03-205-11/+12
| |
* | Merge pull request #7000 from chef/repo_filenamesTim Smith2018-03-206-3/+28
|\ \ | | | | | | Fail with a warning if users specify apt/yum/zypper repos with slashes
| * | Fail with a warning if users specify apt/yum/zypper repos with slashesrepo_filenamesTim Smith2018-03-196-3/+28
| | | | | | | | | | | | | | | | | | We can't write out a filename like foo/bar.repo so we should properly warn the user if they try to give us that. There's probably other things, but this one makes particular sense for apt since people want to use the slash in PPA repo names. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Merge pull request #7003 from chef/minor_resource_cleanupTim Smith2018-03-208-79/+38
|\ \ \ | | | | | | | | Add more resource descriptions and convert more resources to use properties
| * | | Add more resource descriptions and convert more resources to use propertiesminor_resource_cleanupTim Smith2018-03-198-79/+38
| |/ / | | | | | | | | | | | | | | | More minor cleanup Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Merge pull request #7004 from chef/amazon_linux_build_essentialTim Smith2018-03-203-2/+28
|\ \ \ | | | | | | | | Don't try to install old GCC packages on Amazon Linux 2
| * | | Test no Amazon Linux 2.0amazon_linux_build_essentialTim Smith2018-03-192-1/+27
| | | | | | | | | | | | | | | | | | | | | | | | It's enough different and we're starting to get a good number of users on the new version. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | | Don't try to install old GCC packages on Amazon Linux 2Tim Smith2018-03-191-1/+1
| |/ / | | | | | | | | | | | | | | | This worked until Amazon released 2.0 Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Bump version to 14.0.144 by Expeditorv14.0.144Chef Expeditor2018-03-205-11/+12
| | |
* | | Bump version to 14.0.143 by Expeditorv14.0.143Chef Expeditor2018-03-204-8/+8
| | |
* | | Merge pull request #7013 from chef/tm/remove_bundle_auditThom May2018-03-203-12/+0
|\ \ \ | | | | | | | | Remove Bundle Audit
| * | | remove bundle audittm/remove_bundle_auditThom May2018-03-203-12/+0
|/ / / | | | | | | | | | Signed-off-by: Thom May <thom@chef.io>
* | | Merge pull request #7009 from chef/fix_zypperTim Smith2018-03-192-1/+9
|\ \ \ | |/ / |/| | Detect new "automatically" installed string in Zypper
| * | Just look for Yes and anything in the regexTim Smith2018-03-191-1/+1
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Detect new "automatically" installed string in ZypperTim Smith2018-03-192-1/+9
|/ / | | | | | | | | | | | | | | | | | | This resolves issue #6836 which explains the issue in great detail. TLDR: Packages that are installed as deps on another package have a different string when you get info on them now. This updates the regex to detect both. Based on the zypper changelog I believe this was introduced in 1.13.17. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #6910 from chef/tm/inject_logThom May2018-03-1918-236/+187
|\ \ | | | | | | Add the ability to inject log objects
View Lorry Controller Status