For testing purpose it would be nice if the the user could destroy a specific or a random cluster instance.
Let me know what you think.
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.
> 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).
I have updated the patch and added basic unit tests.
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. Tom, thanks for reviewing.
I've just committed this. Thanks Tom.