[ 
https://issues.apache.org/jira/browse/WHIRR-152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12992295#comment-12992295
 ] 

Adrian Cole commented on WHIRR-152:
-----------------------------------

I think this patch can work, but could be a bit cleaner.  For example, I think 
we should consider not tainting code with any references to test case state. An 
alternate way could be to manage this with clarifying the operations to include 
things like start/stopProxy and start/stopCluster vs create/destroyCluster.

In the mean time, I think a better way would be to check for a system property 
like whirr.don'tshutdownmystuff in the tearDown() method, and conditionally 
shut down there.

> Add option to leave cluster running when integration tests fail
> ---------------------------------------------------------------
>
>                 Key: WHIRR-152
>                 URL: https://issues.apache.org/jira/browse/WHIRR-152
>             Project: Whirr
>          Issue Type: Improvement
>          Components: core
>            Reporter: Tom White
>         Attachments: WHIRR-152.patch, WHIRR-152.patch
>
>
> This would be useful for debugging why tests failed.

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to