11/20 Opscode Cookbook Code Review


#1

More!

Merge:
COOK-3852 - Ubuntu receipe triggers an apt-get update before
replacing the default sources.list
COOK-3885 - launchd_service template missing client_bin
COOK-3267 - Improve testing suite in runit cookbook
COOK-3851 - Postgresql: reload after config change does not pick up
certain configuration changes
COOK-3892 - sql_server cookbook uses deprecated windows_registry LWRP
COOK-2672 - EULA for package is displayed instead accepted
COOK-3843 - Make cookbook ‘sudo’ compatible with Mac OS X
COOK-3868 - add mod_dav to subversion cookbooks
COOK-2073 - Support for MatchGroup in openssh cookbook

Other:
COOK-3818 - haproxy cookbook uses old-style node attributes
Reopened - Don’t overload the attribute
COOK-3899 - Allow action override in service block
Reviewed - Those present don’t know anything about fail2ban.
COOK-3368 - windows_task will fail on non-english version of windows
Reviewed - Adam Edwards?


Bryan McLellan | opscode | technical program manager, open source
© 206.607.7108 | (t) @btmspox | (b) http://blog.loftninjas.org


#2

Bryan,

On 20 November 2013 23:04, Bryan McLellan btm@opscode.com wrote:

More!

Merge:
COOK-3885 - launchd_service template missing client_bin

This one line fix has been provided 2 months ago, with a review action to
merge one month ago. When will this actually be merged and released?

Ringo


#3

On Sat, Dec 14, 2013 at 3:29 AM, Ringo De Smet ringo.desmet@gmail.com wrote:

On 20 November 2013 23:04, Bryan McLellan btm@opscode.com wrote:

COOK-3885 - launchd_service template missing client_bin

This one line fix has been provided 2 months ago, with a review action to
merge one month ago. When will this actually be merged and released?

Looks like Sean got to it today.

https://tickets.opscode.com/browse/COOK-3885


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