summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Rewrite nested json test to not use stackjdm/json-nested-testJay Mundrawala2015-07-171-3/+4
| | | | | | | | | | | | | | | | | | | | Freebsd 10.1 i386 testers keep getting: ``` /opt/chef/embedded/lib/ruby/gems/2.2.0/gems/rspec-core-3.3.2/lib/rspec/core/configuration.rb:1327:in `load': stack level too deep (SystemStackError) from /opt/chef/embedded/lib/ruby/gems/2.2.0/gems/rspec-core-3.3.2/lib/rspec/core/configuration.rb:1327:in `block in load_spec_files' from /opt/chef/embedded/lib/ruby/gems/2.2.0/gems/rspec-core-3.3.2/lib/rspec/core/configuration.rb:1325:in `each' from /opt/chef/embedded/lib/ruby/gems/2.2.0/gems/rspec-core-3.3.2/lib/rspec/core/configuration.rb:1325:in `load_spec_files' from /opt/chef/embedded/lib/ruby/gems/2.2.0/gems/rspec-core-3.3.2/lib/rspec/core/runner.rb:102:in `setup' from /opt/chef/embedded/lib/ruby/gems/2.2.0/gems/rspec-core-3.3.2/lib/rspec/core/runner.rb:88:in `run' from /opt/chef/embedded/lib/ruby/gems/2.2.0/gems/rspec-core-3.3.2/lib/rspec/core/runner.rb:73:in `run' from /opt/chef/embedded/lib/ruby/gems/2.2.0/gems/rspec-core-3.3.2/lib/rspec/core/runner.rb:41:in `invoke' from /opt/chef/embedded/lib/ruby/gems/2.2.0/gems/rspec-core-3.3.2/exe/rspec:4:in `<top (required)>' from /opt/chef/embedded/bin/rspec:23:in `load' from /opt/chef/embedded/bin/rspec:23:in `<main>' ``` We seem to keep changing the depth of the json object. Instead, let's just not do it on the stack.
* Merge pull request #3680 from chef/tm/service_appveyor_onlyThom May2015-07-172-1/+2
|\ | | | | avoid windows service spec unless we're on appveyor
| * avoid windows service spec unless we're on appveyortm/service_appveyor_onlyThom May2015-07-172-1/+2
|/
* Update the URL in user agent string12.5.0.current.0danielsdeleo2015-07-161-1/+1
|
* Merge pull request #3242 from ranjib/chef_handlerRanjib Dey2015-07-166-1/+183
|\ | | | | [RFC-039] chef handler dsl
| * Merge remote-tracking branch 'origin/master' into chef_handlerRanjib Dey2015-07-0692-1611/+3399
| |\ | | | | | | | | | | | | Conflicts: lib/chef/exceptions.rb
| * \ Merge remote-tracking branch 'origin/master' into chef_handlerRanjib Dey2015-06-2469-765/+4290
| |\ \
| * | | add name class attribute for user defined handlers. more readable invalid ↵Ranjib Dey2015-06-235-18/+70
| | | | | | | | | | | | | | | | event handler exception name.
| * | | fix typoRanjib Dey2015-06-181-1/+1
| | | |
| * | | add spec for eventhandler dslRanjib Dey2015-06-131-0/+35
| | | |
| * | | use Chef.run_context to register handler if setRanjib Dey2015-06-131-1/+9
| | | |
| * | | Merge remote-tracking branch 'origin/master' into HEADRanjib Dey2015-06-12500-5875/+19315
| |\ \ \ | | | | | | | | | | | | | | | | | | | | Conflicts: lib/chef/chef_class.rb
| * | | | [RFC-039] chef handler dslRanjib Dey2015-04-185-1/+77
| | | | |
* | | | | Fix issue where DSL is not emitted if state_properties happens before propertyjk/state_attrsJohn Keiser2015-07-152-2/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If you call state_properties :x, and there is no property :x yet, it assumes you are working with a custom getter/setter like "def x". This is fine, but when you say "property :x", it won't override that (and will treat the property as if the getter/setter were already defined instead of creating a new one).
* | | | | Merge pull request #3592 from chef/ryan/archThom May2015-07-142-1/+7
|\ \ \ \ \ | | | | | | | | | | | | Add myself as an Archlinux maintainer
| * | | | | Add Ryan Cragun to Archlinux maintainersryan/archRyan Cragun2015-07-012-1/+7
| | | | | |
* | | | | | Merge pull request #3646 from chef/jdm/config-workstation-specJay Mundrawala2015-07-081-5/+10
|\ \ \ \ \ \ | | | | | | | | | | | | | | Try fix for failing config test
| * | | | | | Try fix for failing config testjdm/config-workstation-specJay Mundrawala2015-07-081-5/+10
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We keep seeing this randomly ``` 1) Chef::WorkstationConfigLoader loading the config file when the config file exists and raises a ruby exception during evaluation raises a ConfigurationError Failure/Error: expect { config_loader.load }.to raise_error(Chef::Exceptions::ConfigurationError) expected Chef::Exceptions::ConfigurationError, got #<Errno::ENOENT: No such file or directory @ rb_sysopen - /tmp/Chef-WorkstationConfigLoader-rspec-test20150707-35300-f9yfb6> with backtrace: # ./lib/chef/workstation_config_loader.rb:164:in `readlines' # ./lib/chef/workstation_config_loader.rb:164:in `highlight_config_error' # ./lib/chef/workstation_config_loader.rb:156:in `rescue in read_config' # ./lib/chef/workstation_config_loader.rb:137:in `read_config' # ./lib/chef/workstation_config_loader.rb:72:in `load' # ./spec/unit/workstation_config_loader_spec.rb:275:in `block (6 levels) in <top (required)>' # ./spec/unit/workstation_config_loader_spec.rb:275:in `block (5 levels) in <top (required)>' # ./spec/unit/workstation_config_loader_spec.rb:275:in `block (5 levels) in <top (required)>' ``` I think the issue is that the tempfile gets GC'd, and when it gets GC'd, it deletes the file. If it is cleaned up before it is used, then the test fails.
* | | | | | Merge pull request #3641 from grubernaut/issue_3640Noah Kantrowitz2015-07-072-2/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Decommission 12-stable
| * | | | | | Decommission 12-stableJake Champlin2015-07-072-2/+1
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | Removes 12-stable from travis.yml branch filter Updates appveyor.yml for the 12.4-stable branch
* | | | | | Update CHANGELOG for Chef 12.4.1Jay Mundrawala2015-07-061-3/+14
| | | | | |
* | | | | | Merge pull request #3630 from chef/tc/fix-chef-userJay Mundrawala2015-07-0651-1477/+2009
|\ \ \ \ \ \ | | | | | | | | | | | | | | Move Chef::OscUser back to Chef::User namespace and new user code to Chef::UserV1.
| * | | | | | Update changelog.tc/fix-chef-usertylercloke2015-07-061-0/+1
| | | | | | |
| * | | | | | Allow cookbooks_dir to use chef_rest instead of rest for now.tylercloke2015-07-062-10/+2
| | | | | | |
| * | | | | | Remove all the stupid json magic from Chef::ApiClientV1.tylercloke2015-07-066-111/+44
| | | | | | |
| * | | | | | Fix automated object generation from HTTP layer in ApiClient.tylercloke2015-07-061-0/+5
| | | | | | |
| * | | | | | Fixed old ApiClient tests.tylercloke2015-07-061-8/+3
| | | | | | |
| * | | | | | Set Chef::ApiClient to use API V0.tylercloke2015-07-061-2/+3
| | | | | | |
| * | | | | | Cleanup.tylercloke2015-07-062-1/+2
| | | | | | |
| * | | | | | Move ApiClient V1 supported code to Chef::ApiClientV1 and restore ↵tylercloke2015-07-0619-352/+944
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Chef::ApiClient. For backwards compatibility. ApiClientV1 will replace ApiClient when Chef 13 is released. Updated client_*.rb knife commands to use ApiClientV1.
| * | | | | | Default Chef::ChefFS::...::ChefServerRootDir.rest to use API V0 and ↵tylercloke2015-07-062-3/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | deprecated chef_rest. Unfortunately, chef_rest was using the now outdated Chef::HTTP so I could not default it to API V0. It was also being used anywhere except Chef::ChefFS::FileSystem::CookbooksDir, which can just as easially use rest instead of chef_rest as they provide the same interface. That update has been made.
| * | | | | | Remove _rest from http methods.tylercloke2015-07-062-14/+14
| | | | | | |
| * | | | | | Move Chef::OscUser back to Chef::User namespace and new user code to ↵tylercloke2015-07-0631-1118/+1122
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Chef::UserV1. Also, have Chef::User (formally Chef::OscUser) use API V0 to make requests.
* | | | | | Merge branch 'jk/3634'John Keiser2015-07-062-4/+34
|\ \ \ \ \ \
| * | | | | | Deprecate passing more than 1 argument to create a resourcejk/3634John Keiser2015-07-062-8/+18
| | | | | | |
| * | | | | | Don't accept multiple parameters in recipe DSL (just name)John Keiser2015-07-062-4/+24
|/ / / / / /
* | | | | | Merge branch 'jk/separate-priority-map'John Keiser2015-07-0634-517/+832
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | |
| * | | | | Memoize enabled_handlers so it is not recomputedjk/separate-priority-mapJohn Keiser2015-07-064-3/+39
| | | | | |
| * | | | | Re-separate priority map and DSL handler map so that provides has veto power ↵John Keiser2015-07-0634-517/+796
|/ / / / / | | | | | | | | | | | | | | | over priority
* | | | | Merge branch 'jk/property-state'John Keiser2015-07-0311-585/+1542
|\ \ \ \ \
| * | | | | Move property type derivation from "property_type" to "property"John Keiser2015-07-033-69/+42
| | | | | | | | | | | | | | | | | | | | | | | | to keep property_type simple
| * | | | | Add Property.create and Resource.property_type for type system overridingJohn Keiser2015-07-036-59/+137
| | | | | |
| * | | | | Bring set_or_return behavior back in line with what it used to beJohn Keiser2015-07-033-14/+26
| | | | | |
| * | | | | Only stick non-frozen values to the resourceJohn Keiser2015-07-032-6/+57
| | | | | |
| * | | | | Code review commentsJohn Keiser2015-07-031-6/+24
| | | | | |
| * | | | | Make required name attributes workJohn Keiser2015-07-0310-578/+1403
|/ / / / /
* | | | | Make required name attributes workjk/required_name_attributeJohn Keiser2015-07-022-6/+6
| | | | |
* | | | | Merge pull request #3626 from chef/revert-3606-jdm/pin-ffiJay Mundrawala2015-07-021-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Revert "FFI 1.9.9 is causing segfaults"
| * | | | | Revert "FFI 1.9.9 is causing segfaults"revert-3606-jdm/pin-ffiJay Mundrawala2015-06-301-1/+1
| | | | | |
* | | | | | Merge pull request #3623 from chef/jdm/simplify-lwrp1Jay Mundrawala2015-07-013-184/+153
|\ \ \ \ \ \ | | | | | | | | | | | | | | Simplify LWRP Deprecations Proposal 1
View Lorry Controller Status