neykov commented on this pull request.
> @@ -73,36 +93,41 @@ public long getDelay(TimeUnit unit) {
return unit.convert(expiryTime - timeSupplier.get(),
TimeUnit.MILLISECONDS);
}
}
- private Map<String, VagrantNode> nodes = new ConcurrentHashMap<String,
VagrantNode>();
- private DelayQueue<TerminatedNode> terminatedNodes = new
DelayQueue<TerminatedNode>();
+
+ private final DelayQueue<TerminatedNode> terminatedNodes = new
DelayQueue<TerminatedNode>();
Me neither before writing this code :). Turned out really handy.
If I remember correctly jclouds needs to see `terminated` status after
destroying the machine. I remove all traces of the machine on termination so I
won't be able to tell the difference between `terminated` and `not created` if
I don't keep this around.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/355
