Nice update! It looks pretty good now. There are only a few minors. Apart from that, looking at how the ProfitBricks API returns the locations (lowercase region/zone), we'll have to configure some extra bits when it comes to implement the ComputeService abstraction (as the default jclouds implementation will just use the zone id without the "region/" prefix), but that is an easy change we'll do when we get there.
Nice job! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/225#issuecomment-166295135
