[ https://issues.apache.org/jira/browse/WHIRR-49?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13133633#comment-13133633 ]
Andrei Savu commented on WHIRR-49: ---------------------------------- Nice work David! I really like the tests based on dry run. We have to use 2 spaces for indentation just to keep the code base consistent and remove the @author javadoc. Two tests are commented (testChefRunRecipeFromURL and testChefRunRecipesFromProvidedCookbooks) - do we need them? What do you think about creating something like services/common for common scripts? (e.g. install_ruby for chef and puppet) > Allow Whirr to use Chef for configuration management > ---------------------------------------------------- > > Key: WHIRR-49 > URL: https://issues.apache.org/jira/browse/WHIRR-49 > Project: Whirr > Issue Type: New Feature > Components: core > Reporter: Jeff Hammerbacher > Attachments: WHIRR-49-cc.patch, WHIRR-49-pom.patch, WHIRR-49.patch, > WHIRR-49.patch, WHIRR-49.patch, WHIRR-49.patch, WHIRR-49.patch, > WHIRR-49.patch, WHIRR-49.patch, WHIRR-49.patch, WHIRR-49.patch, WHIRR-49.patch > > > As discussed in > https://cwiki.apache.org/confluence/display/WHIRR/WhirrDesign, Whirr should > be agnostic to the tool used to bring the images up to a state where they're > ready to run the service. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira