The reason for this override is because, when I do:
```
ComputeService compute = ContextBuilder.newBuilder("profitbricks")
.credentials("identity", "credential")
.buildView(ComputeServiceContext.class)
.getComputeService();
Template template = compute.templateBuilder().build();
```
The `template` produced has a location of type `REGION` (`us/lasdev`, `us/las`,
etc.,). ProfitBricks provisions a node to a datacenter though (`ZONE`). Then,
there's no method like `TemplateBuilder#getLocationId()` to check whether the
locationId was already set, so I was *forced* to parse again the property.
Both `BaseTemplateBuilderLiveTest` and `BaseComputeServiceLiveTest` passes with
or without this method, but then again I am passing a template, so the context
uses the location of the image.
What'ya think?
P.S.: I removed the `ProfitBricksTemplateBuilder#build()`. It turns out, it
wasn't needed. Also I added this
[catch](https://github.com/devcsrj/jclouds-labs/commit/f4b1311c3cb4f4caf8b17c31e6cb6597d151bce9#diff-11ea89320c87f906e05816c31c5773a1R90),
since weirdly enough, it threw `Unparseable date: "2015-06-18T14:16:51+0000"`,
even when the actual value returned by the server is `2015-06-18T14:16:51Z`.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/145#issuecomment-115143706
