For testing purpose it would be nice if the the user could destroy a specific or a random cluster instance.
I've just committed this. Thanks Tom.
I have updated the patch. Tom, thanks for reviewing.
Looks good. Thanks for adding the test. Looks like there are some spurious imports in DestroyInstanceCommand - e.g. Random. And some imports re-arranged in Service that don't need to be.
I have updated the patch and added basic unit tests.
> Do you need to expose the 'random' feature in the CLI?
Not really. It makes sense for services like ZooKeeper but for services like HDFS killing the namenode will bring down the whole cluster so it's not that useful. I now realize that it needs to be aware of the instance role.
I will submit an updated version of the patch soon (+ unit tests).
Do you need to expose the 'random' feature in the CLI? Can the test not choose this itself?
Please add a unit test for DestroyInstanceCommand.
Let me know what you think.