[chef-dev] 5/6 Opscode Code Review


Chronological Thread 
  • From: Steven Danna < >
  • To: < >
  • Subject: [chef-dev] 5/6 Opscode Code Review
  • Date: Mon, 6 May 2013 13:03:37 -0700

Ohai Chefs,

Code review notes are below.  Also, if you weren't able to make
ChefConf, the talks are currently up on YouTube:

https://www.youtube.com/playlist?list=PLrmstJpucjzXNMLcI5X-EjirpDd-SITd3

Cheers,

Steven

======
* To Merge
- CHEF-4070 Support DataBag.list on chef-solo

- CHEF-4130 knife * edit presents a .js file not a .json file

- CHEF-4135 Malformed arguments to notifies fail without a helpful
  message

- CHEF-3975 Searching for compound attributes in data bags will not
  yield results

- CHEF-4088 sorting the knife commands is done, but would be cool if
  you also sort the sub commands...

- CHEF-3976 chef_objects rejects "provides 'service[foo]'" in metadata


* Reopen
- CHEF-4115 The config file's path isn't shown in exceptions
  Please add a test for this here:
  https://github.com/opscode/chef/blob/master/spec/unit/application_spec.rb

- CHEF-4114 Broken raise of ConfigurationError when log_location not
  writable

  Please add a test for this here:
  https://github.com/opscode/chef/blob/master/spec/unit/config_spec.rb


- CHEF-2911 11: Yum Provider will not install correct version if
  version string is specified in the name

  Still looking for a deprecation message.

- CHEF-3599 extend include_recipe to be able to default to the current
  cookbook


* Other

- CHEF-4110 ruby_blocks should support why_run
  Conversation on this one ran long.  We aren't sure as to whether or
  not we should expose this to users.

-- 
Steven Danna
Systems Engineer, Opscode, Inc
GPG Key: http://stevendanna.github.com/downloads/code/public.key

Attachment: signature.asc
Description: OpenPGP digital signature



  • [chef-dev] 5/6 Opscode Code Review, Steven Danna, 05/06/2013

Archive powered by MHonArc 2.6.16.

§