summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | | | Merge pull request #5402 from chef/tas50/maintainersLamont Granquist2016-10-282-2/+14
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Add myself as a Maintainer on debian, ubuntu, suse, test tools and core
| * | | | | | | | | | | Add myself as a Maintainer on debian, ubuntu, suse, test tools and coretas50/maintainersTim Smith2016-10-032-2/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | | | | | | | | Bump version of chef to 12.16.24 by Chef Versioner.v12.16.24Chef Versioner2016-10-274-8/+8
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #5394 from ↵Bryan McLellan2016-10-272-6/+16
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | MsysTechnologiesllc/ali/package_resource_uninstall_fix Newly creating uninstall string using the product code value retrieved from the uninstall_string of the Windows registry to avoid using invalid uninstall_string.
| * | | | | | | | | | | Newly creating uninstall string using the product code value retrieved from ↵aliasgar162016-10-132-6/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the uninstall_string of the Windows registry to avoid using invalid uninstall_string. Signed-off-by: aliasgar16 <aliasgar.batterywala@msystechnologies.com>
* | | | | | | | | | | | Bump version of chef to 12.16.23 by Chef Versioner.v12.16.23Chef Versioner2016-10-264-8/+8
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #5378 from chef/lcg/node-path-trackingLamont Granquist2016-10-2611-295/+491
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / / |/| | | | | | | | | | | Add Chef::Node __path tracking, change root to __root, drop top_level_breadcrumb hack, fix long-standing Chef 12.0 deep merge cache bugs
| * | | | | | | | | | | rename __path to __path__ and __node to __node__lcg/node-path-trackingLamont Granquist2016-10-256-48/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | consistency with Unchain here: https://github.com/chef/chef/blob/master/lib/chef/decorator/unchain.rb#L23-L24 and with SimpleDelegator: https://ruby-doc.org/stdlib-2.1.0/libdoc/delegate/rdoc/SimpleDelegator.html#method-i-__getobj__ Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | | | | | | | | | | add tests from #4207Lamont Granquist2016-10-251-0/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | validates that we fixed the top_level_breadcrumb-state related bug which has been reported Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | | | | | | | | | | remove breadcrumb stateLamont Granquist2016-10-257-126/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | | | | | | | | | | standardize initializer args for VividMash+ImmutableMashLamont Granquist2016-10-255-32/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | if anyone is using these directly this might be a breaking change, although it makes it considerably easier to use VividMash and IMO ImmutableMash is an implementation detail of the deep merge cache and Chef::Node object. we definitely have never documented these APIs, so I think the onus is on the consumer to update their code. VividMash.new() should now work. VividMash.new({ foo: :bar }) should also now work. IDK what object people would have been passing in as the root object before. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | | | | | | | | | | green testsLamont Granquist2016-10-253-6/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | | | | | | | | | | fix immutablize to add __rootLamont Granquist2016-10-251-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | much less red now Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | | | | | | | | | | savegame for more aggressive refactoringLamont Granquist2016-10-255-29/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | | | | | | | | | | add specsLamont Granquist2016-10-251-0/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | | | | | | | | | | add a deep_merge_cache mixinLamont Granquist2016-10-252-83/+115
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | gets the inheritence chain correct Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | | | | | | | | | | s/path_tracking/state_tracking and add __root stateLamont Granquist2016-10-254-13/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | | | | | | | | | | add code to implement node path trackingLamont Granquist2016-10-256-75/+198
|/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | via node.__path variable using __underscore format because node["path"] could very well be used and we still have to deal with node.path method_missing horribleness. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | | | | | | | Bump version of chef to 12.16.22 by Chef Versioner.v12.16.22Chef Versioner2016-10-244-8/+8
| | | | | | | | | | |
* | | | | | | | | | | Merge pull request #5483 from chef/lcg/package-inline-resourcesLamont Granquist2016-10-242-8/+10
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|/ |/| | | | | | | | | | Core: set use_inline_resources in package superclass
| * | | | | | | | | | remove excessive mockinglcg/package-inline-resourcesLamont Granquist2016-10-211-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | we have too much collaboration and state in our objects to write pure unit tests. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | | | | | | | | | set use_inline_resources in package superclassLamont Granquist2016-10-211-5/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | allows subclasses to use_inline_resources (hab_package will most likely want this) Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | | | | | | | Bump version of chef to 12.16.21 by Chef Versioner.v12.16.21Chef Versioner2016-10-244-8/+8
| | | | | | | | | | |
* | | | | | | | | | | Merge pull request #5482 from chef/COOL-597/use-all-for-assertionTom Duffield2016-10-242-1/+2
|\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | | Use `all?` instead of `any?` to check for SUSE users
| * | | | | | | | | | Mock users in functional group testsCOOL-597/use-all-for-assertionTom Duffield2016-10-211-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because `Provider::Group::Suse` expects users to exist, we need to mock out the `Etc` call that checks for our non-existent spec users. Signed-off-by: Tom Duffield <tom@chef.io>
| * | | | | | | | | | Use `all?` instead of `any?` to check for SUSE usersTom Duffield2016-10-211-1/+1
|/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using `all?` will return `false` when the array is empty. Signed-off-by: Tom Duffield <tom@chef.io>
* | | | | | | | | | Bump version of chef to 12.16.20 by Chef Versioner.v12.16.20Chef Versioner2016-10-214-8/+8
| | | | | | | | | |
* | | | | | | | | | Merge pull request #5476 from tas50/copyrightsLamont Granquist2016-10-2121-44/+44
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Fix copyrights and update a few more Opscode references to Chef
| * | | | | | | | | | More Opscode -> Chef updatesTim Smith2016-10-2013-36/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | | | | | | | | | Fix a few bad copyrightsTim Smith2016-10-208-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I think this happened when we merged the Opscode and Chef copyrights Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | | | | | | | Merge pull request #5475 from tas50/spec_typoLamont Granquist2016-10-211-1/+1
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Fix a typo in the apt_repository spec
| * | | | | | | | | | | Fix spec typoTim Smith2016-10-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | | | | | | | | Merge pull request #5474 from tas50/yum_propertyLamont Granquist2016-10-211-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Use property vs. attribute in yum_repository
| * | | | | | | | | | | | Use property vs. attribute in yum_propertyTim Smith2016-10-191-1/+1
| |/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We just auto replace the attribute call with a property call here. We might as well use the right one to start with. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | | | | | | | | Bump version of chef to 12.16.19 by Chef Versioner.v12.16.19Chef Versioner2016-10-214-8/+8
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #5472 from chef/COOL-597/address-suse-test-failuresTom Duffield2016-10-213-6/+117
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Address test failures in SUSE 11.4 and 12.1
| * | | | | | | | | | | | Raise error if suse group manager adds non-existent userCOOL-597/address-suse-test-failuresTom Duffield2016-10-202-4/+113
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The expected behavior of chef, as defined by our tests, is that if a user specified in a group resource does not exist than chef should raise an error. On suse, we use groupmod which will not raise an error if you attempt to add a non-existent user to a group. This change adds in a simple check that will raise an error if the specified does not exist. Signed-off-by: Tom Duffield <tom@chef.io>
| * | | | | | | | | | | | Update useradd test to account for new SUSE 12.1 behaviorTom Duffield2016-10-201-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `usermod` binary in SUSE 12.1 no longer throws an error if you attempt to unlock a user account that has no password. Signed-off-by: Tom Duffield <tom@chef.io>
* | | | | | | | | | | | | Bump version of chef to 12.16.18 by Chef Versioner.v12.16.18Chef Versioner2016-10-204-8/+8
| | | | | | | | | | | | |
* | | | | | | | | | | | | Merge pull request #5479 from chef/lcg/rubocop-0.44.1Lamont Granquist2016-10-206-10/+11
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixes for rubocop 0.44.1 engine
| * | | | | | | | | | | | | fixes for rubocop 0.44.1 enginelcg/rubocop-0.44.1Lamont Granquist2016-10-206-10/+11
| | |_|/ / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a few cops we have enabled got a little more accurate/stricter Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | | | | | | | | | Bump version of chef to 12.16.17 by Chef Versioner.v12.16.17Chef Versioner2016-10-204-8/+8
| | | | | | | | | | | | |
* | | | | | | | | | | | | Merge pull request #5481 from chef/lcg/node-testsLamont Granquist2016-10-201-0/+28
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / |/| | | | | | | | | | | | Attributes: add tests for regression in 12.0 fixed recently
| * | | | | | | | | | | | Attributes: add tests for regression in 12.0 fixed recentlyLamont Granquist2016-10-201-0/+28
|/ / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tests for the issue that led up to https://github.com/chef/chef/issues/5447 The Hash tests actually pass against 12.13.37 so it appearly that only broke the Immutable array objects in 12.0. This both explains why we didn't have any existing tests that caught it, and why it went uncaught for so long. It also adds some ammunition to the argument that fixing this in 12.14.x was not a major breaking change to users that had been relying on the buggy 12.0 behavior. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | | | | | | | | Bump version of chef to 12.16.16 by Chef Versioner.v12.16.16Chef Versioner2016-10-204-8/+8
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #5471 from chef/lcg/make-scm-password-sensitiveThom May2016-10-201-7/+1
|\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | | convert svn_password to property + make sensitive
| * | | | | | | | | | | the password is also not desired_statelcg/make-scm-password-sensitiveLamont Granquist2016-10-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | should property-ize the rest of this as well Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | | | | | | | | | | convert scm_password to property + make sensitiveLamont Granquist2016-10-191-7/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | | | | | | | | Bump version of chef to 12.16.15 by Chef Versioner.v12.16.15Chef Versioner2016-10-194-8/+8
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #5466 from pwalz/fix-crontab-rewritesLamont Granquist2016-10-191-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | | Cron Provider: Eliminate crontab rewrite that happens with every chef-client run
View Lorry Controller Status