Hi @nacx ! Sorry for the delay, I was preoccupied with other things.

- *Does it fail if you don't add it to the jclouds closer?* I tried commenting 
out the `closer.add(..)` line, but still threw the error.
- *"..not 100% sure that will be guice friendly."* Yeah it wasn't. Too bad. 
Kinda messy type casts.

But more importantly, because of the `ProvisioningManagerTest`, the exception 
was reproduced. It was because of the [executors 
map](https://github.com/devcsrj/jclouds-labs/commit/0e783857c09d94b3770583bb52a23fbee1d894ca#diff-c67d5d74a6059f235c249f2e9a00a176L47)
 being a static variable. The live tests creates multiple contexts; and 
afterwards closed. Since it's a static variable, the rest of the instances held 
a closed executor. Phew, you're a code wizard. \m/

Anyway, before I squash them (and fix conflicts), I'll re-run a live test in a 
few moments and re-post the latest results. :)

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/145#issuecomment-113077243

Reply via email to