summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
* minimal_ohai: Add init_package plugin as a required plugininit_package_pluginTim Smith2018-11-221-1/+1
| | | | | | | This is a pretty critical bit of data and we're using it for decisions in the timezone resource now. Signed-off-by: Tim Smith <tsmith@chef.io>
* add tests for apt-get --allow-downgradeslcg/apt-get-allow-downgradesLamont Granquist2018-11-201-2/+79
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* add --allow-downgrades to apt-getLamont Granquist2018-11-201-6/+6
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Merge pull request #7953 from chef/lcg/chef-15-allow-downgradeLamont Granquist2018-11-193-26/+38
|\ | | | | Chef-15: switch default of allow_downgrade to true
| * fix one of the yum func testslcg/chef-15-allow-downgradeLamont Granquist2018-11-191-1/+2
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * Chef-15: switch default of allow_downgrade to trueLamont Granquist2018-11-192-25/+36
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | Chef-15: require instead of load librariesLamont Granquist2018-11-191-1/+5
|/ | | | | | | | This reverts a change which was working around problems caused by unforked interval runs. Fixes issues with chefspec, should make it faster and reduce warning spam. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Merge pull request #7916 from chef/testingTim Smith2018-11-191-0/+27
|\ | | | | windows_workgroup: Coerce the provided reboot property and add more tests
| * windows_workgroup: Coerce the provided reboot property and add more testsTim Smith2018-11-141-0/+27
| | | | | | | | | | | | | | | | Instead of converting the passed in reboot property when we use it coerce it when it's passed in. This allows us to easily test the conversion and it makes it so the docs don't include the legacy values. I also added moved command generation into a method and added tests for that. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #7946 from chef/propertiesTim Smith2018-11-193-97/+142
|\ \ | | | | | | Convert service resource to use properties
| * | Expand windows_service specsTim Smith2018-11-151-6/+29
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Simplify the windows_service specTim Smith2018-11-151-8/+0
| | | | | | | | | | | | | | | | | | No need to configure the provider mapping for this spec Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Add a basic macosx_service specTim Smith2018-11-151-0/+37
| | | | | | | | | | | | | | | | | | Might as well test this Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Simplify the service resource specsTim Smith2018-11-151-83/+59
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Convert service resource to use propertiesTim Smith2018-11-151-15/+32
| | | | | | | | | | | | | | | | | | Also expand the testing a bit while I'm here. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Merge pull request #7947 from chef/properties2Tim Smith2018-11-191-9/+22
|\ \ \ | | | | | | | | Fully convert remote_directory to use properties
| * | | Fully convert remote_directory to use propertiesproperties2Tim Smith2018-11-161-9/+22
| |/ / | | | | | | | | | | | | | | | Also add descriptions, default_descriptions, and expand the tests a bit. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Merge pull request #7917 from chef/certTim Smith2018-11-171-0/+30
|\ \ \ | |/ / |/| | windows_certificate: Add testing of the defaults and allowed properties
| * | windows_certificate: Add testing of the defaults and allowed propertiesTim Smith2018-11-141-0/+30
| |/ | | | | | | | | | | These should have been tested when I merged this in. It's easy stuff to verify with unit tests. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #7928 from chef/gem_spec_cleanupTim Smith2018-11-151-3/+0
|\ \ | | | | | | Don't set chef_gem_compile_time config in the chef_gem spec
| * | Don't set chef_gem_compile_time config in the chef_gem specgem_spec_cleanupTim Smith2018-11-141-3/+0
| |/ | | | | | | | | | | | | We removed this config in Chef 13. There's no need to set it in the spec anymore. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #7914 from EugenMayer/patch-1Tim Smith2018-11-141-3/+3
|\ \ | |/ |/| use --no-tty during apt-keys fro gpg - fixes #7913
| * fix code stylesem2018-11-121-1/+1
| | | | | | | | Signed-off-by: em <eugen.mayer@kontextwork.de>
| * restore spec name - acidently defacedEugen Mayer2018-11-121-2/+1
| | | | | | | | Signed-off-by: em <eugen.mayer@kontextwork.de>
| * Fix specs for including --no-ttyEugen Mayer2018-11-121-4/+5
| | | | | | | | Signed-off-by: Eugen Mayer <eugen.mayer@kontextwork.de>
* | windows_feature: Move provider logic into the default of the install_method ↵windows_featureTim Smith2018-11-091-0/+4
|/ | | | | | | | property This eliminates the need for setting the subresource variable and also allows us to auto-generate docs that are aware of the default. Signed-off-by: Tim Smith <tsmith@chef.io>
* Always run policy_file if a policy_file or policy_group existspolicy_files_alwaysTim Smith2018-11-091-12/+0
| | | | | | Remove the config to turn it off Signed-off-by: Tim Smith <tsmith@chef.io>
* Chef-15: add nillability to attribute deep mergingLamont Granquist2018-11-091-0/+30
| | | | | | | | | | Changes to consistently uses the "NIL" object internally to mean "not present" instead of abusing nil for that purpose. So now "nil means nil" and we can deep merge that so it will effectively override lower precedence levels if you write a nil. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* fix the knife integration spec timeoutsLamont Granquist2018-11-083-13/+19
| | | | | | | | | rely on killing the thread to terminate the server, and rely on thread#join to correctly signal to the main thread that the server thread is dead, don't leak the server object out of the helper. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Merge pull request #7886 from chef/lcg/yum-version-with-package-sourceLamont Granquist2018-11-071-0/+18
|\ | | | | add tests for yum version with package_source bug
| * spelling is a thinglcg/yum-version-with-package-sourceLamont Granquist2018-11-071-2/+2
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * add tests for yum version with package_source bugLamont Granquist2018-11-071-0/+18
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | Merge pull request #7866 from chef/provisioningTim Smith2018-11-071-20/+0
|\ \ | |/ |/| Remove Chef provisioning lazy loading
| * Remove Chef provisioning lazy loadingprovisioningTim Smith2018-11-021-20/+0
| | | | | | | | | | | | Remove the lazy loading of chef provisioning resources Signed-off-by: Tim Smith <tsmith@chef.io>
* | Coerce arrays of ints to an array of stringsfirewallTim Smith2018-11-061-4/+9
| | | | | | | | | | | | Otherwise it won't compare to the PS output Signed-off-by: Tim Smith <tsmith@chef.io>
* | Allow passing multiple ports in windows_firewallTim Smith2018-11-061-10/+45
| | | | | | | | | | | | | | | | | | Allow ports to be an integer Allow ports to be a comma separated list or an array Workaround a bug in PowerShell where arrays that are converted to json turn into hashes with the count and the value The quotes around the port are removed because you can't pass '80,8080' in the cmdlet. It has to be a bare 80,8080 Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #7852 from chef/cookbook_createTim Smith2018-11-051-42/+0
|\ \ | | | | | | Fully remove knife cookbook create command
| * | Fully remove knife cookbook create commandcookbook_createTim Smith2018-11-021-42/+0
| | | | | | | | | | | | | | | | | | We already removed all functionality and replaced it with an error message. This prevents the command from showing up in help. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Merge pull request #7842 from Intility/windows_firewall_ruleTim Smith2018-11-051-0/+361
|\ \ \ | | | | | | | | Add windows_firewall_rule
| * | | add testsTor Magnus Rakvåg2018-11-011-0/+361
| | | | | | | | | | | | | | | | Signed-off-by: Tor Magnus Rakvåg <tor.magnus@outlook.com>
* | | | Remove OSC Chef specsTim Smith2018-11-047-332/+1
| | | | | | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | Remove knife user support for open source Chef Server < 12Tim Smith2018-11-045-103/+0
| |_|/ |/| | | | | | | | | | | | | | We shipped Chef 12 server on 2014-11-25. We've maintained backwards compatibility here by shipping double plugins, constantly checking for Chef 11 and then rewriting passed args. We deprecated it with Chef 14. This fully removes support. chef-client will continue to work on chef 11, but users cannot be managed with knife from a workstation running the latest version of Chef/ChefDK. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Merge pull request #7840 from chef/lcg/chef-15-attribute-array-fixesTim Smith2018-11-022-2/+106
|\ \ \ | |_|/ |/| | Chef 15 node attribute array fixes
| * | Chef 15 node attribute array fixeslcg/chef-15-attribute-array-fixesLamont Granquist2018-10-312-2/+106
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Doing something like: ``` node.default["foo"] = [] node.default["foo"] << { "bar" => "baz } ``` Would result in a Hash instead of a VividMash inserted into the AttrArray, so that: ``` node.default["foo"][0]["bar"] # gives the correct result node.default["foo"][0][:bar] # does not work due to the sub-Hash not # converting keys ``` This fixes that behavior, but is likely a breaking change to someone who expects to ingest hashes with symbol keys and get back out symbol keys, they will now get converted to strings. This breaking change is fully 100% intentional. This makes the node attribute structure more consistent and predictable. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | Merge pull request #7323 from chef/cookbook_testTim Smith2018-11-011-84/+0
|\ \ | | | | | | Remove knife cookbook test feature
| * | Remove knife cookbook test featureTim Smith2018-11-011-84/+0
| | | | | | | | | | | | | | | | | | We deprecated this during the Chef 13 cycle. It's time to fully remove it. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Merge pull request #7777 from ↵Bryan McLellan2018-11-011-0/+39
|\ \ \ | |/ / |/| | | | | | | | MsysTechnologiesllc/Kapil/MSYS-913_add_description_property Added property `description` on windows_task resource
| * | - Added `description` property on windows_task resourcekapil chouhan2018-10-301-0/+39
| | | | | | | | | | | | Signed-off-by: kapil chouhan <kapil.chouhan@msystechnologies.com>
* | | Merge pull request #7787 from chef/ohai_nameTim Smith2018-11-011-4/+0
|\ \ \ | |_|/ |/| | Remove the deprecated ohai_name property from the ohai resource
| * | Remove the deprecated ohai_name property from the ohai resourceohai_nameTim Smith2018-10-291-4/+0
| | | | | | | | | | | | | | | | | | This did nothing and we deprecated it already. Signed-off-by: Tim Smith <tsmith@chef.io>