jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Svet <notificati...@github.com>
Subject Re: [jclouds/jclouds-labs] Load existing machines info on startup (#355)
Date Tue, 31 Jan 2017 22:26:54 GMT
>Could you also elaborate a bit the motivation behind having split the Vagrant Facade in
two different interfaces?

The `<B>` started leaking in classes that don't really deal with boxes at all. Looking
at the changes that must've been code that I've removed since as it was additional to `VagrantComputeServiceAdapter`.
Still I think it's useful as `<B>` is now confined just to `ImageSupplier` which indeed
does handle `B`'s.

-- 
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#issuecomment-276513508
Mime
View raw message