2/3 Chef Code Review


#1

Merge:
OHAI-461 - Provide networks information on rackspace nodes
CHEF-4953 - Add information on 3rd party providers for chef-solo
–format option
CHEF-4905 - Arch Linux - Systemd must be used for service resource
CHEF-2334 - Search fails with multiple AND NOT clauses
CHEF-4901 - Upgrade Chef Zero to 2.0
CHEF-3714 - Implement a method in Chef::Util::FileEdit that returns
if a file was edited
CHEF-1752 - Certain characters (e.g. ‘:’) in package version numbers
cause Dpkg provider to install things repeatedly.
CHEF-4848 - Cron resource accepts a symbol as weekday
COOK-4123 - (munin) extend multi-environment monitoring. accepted
by cookbook maintainer
COOK-4206 - (mysql) create root@127.0.01 as well as root@localhost
COOK-3441 - (database) database_user password should not be required
COOK-4213 - (aws) aws_s3_file does not notify on resource updated.
Accepted for merge with caveat that this is a job for Chef 11’s
use_inline_resources. Anyone want to up-convert this provider?
COOK-4188 - (windows) rewrite powershell_installed? helper to handle
Powershells other than 2.0.
Other:
CHEF-4972 - Upstart service provider should obey supports[:reload]
Reopened - upstart always supports reload/restart when it makes sense
COOK-3370 - (tomcat) don’t install the tomcat manager apps package if
we’re not going to use it. Needs rebase against master.
COOK-2100 - (mysql) Changing innodb_log_file_zie tunable results in
inability to start MySQL. Have the test-kitchen tests run
successfully?


Bryan McLellan | chef | software engineer
© 206.607.7108 | (t) @btmspox | (www) http://getchef.com


#2

For COOK-2100 I did run the affected suites in test-kitchen and they were successful.


Kannan Manickam

On February 4, 2014 at 12:21:22 PM, Bryan McLellan (btm@getchef.com) wrote:

Merge:
OHAI-461 - Provide networks information on rackspace nodes
CHEF-4953 - Add information on 3rd party providers for chef-solo
–format option
CHEF-4905 - Arch Linux - Systemd must be used for service resource
CHEF-2334 - Search fails with multiple AND NOT clauses
CHEF-4901 - Upgrade Chef Zero to 2.0
CHEF-3714 - Implement a method in Chef::Util::FileEdit that returns
if a file was edited
CHEF-1752 - Certain characters (e.g. ‘:’) in package version numbers
cause Dpkg provider to install things repeatedly.
CHEF-4848 - Cron resource accepts a symbol as weekday
COOK-4123 - (munin) extend multi-environment monitoring. accepted
by cookbook maintainer
COOK-4206 - (mysql) create root@127.0.01 as well as root@localhost
COOK-3441 - (database) database_user password should not be required
COOK-4213 - (aws) aws_s3_file does not notify on resource updated.
Accepted for merge with caveat that this is a job for Chef 11’s
use_inline_resources. Anyone want to up-convert this provider?
COOK-4188 - (windows) rewrite powershell_installed? helper to handle
Powershells other than 2.0.
Other:
CHEF-4972 - Upstart service provider should obey supports[:reload]
Reopened - upstart always supports reload/restart when it makes sense
COOK-3370 - (tomcat) don’t install the tomcat manager apps package if
we’re not going to use it. Needs rebase against master.
COOK-2100 - (mysql) Changing innodb_log_file_zie tunable results in
inability to start MySQL. Have the test-kitchen tests run
successfully?


Bryan McLellan | chef | software engineer
© 206.607.7108 | (t) @btmspox | (www) http://getchef.com