You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@whirr.apache.org by "Adrian Cole (JIRA)" <ji...@apache.org> on 2011/02/09 03:27:57 UTC

[jira] Commented: (WHIRR-152) Add option to leave cluster running when integration tests fail

    [ 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