nacx commented on this pull request.
> +@Singleton
+public class DatacenterToLocation implements Function<Datacenter, Location> {
+
+ private final JustProvider justProvider;
+
+ @Inject
+ public DatacenterToLocation(JustProvider justProvider) {
+ this.justProvider = checkNotNull(justProvider, "justProvider");
+ }
+
+ @Override
+ public Location apply(final Datacenter datacenter) {
+ return new
LocationBuilder().id(datacenter.id()).description(datacenter.displayName())
+
.parent(getOnlyElement(justProvider.get())).scope(LocationScope.ZONE)
+ .iso3166Codes(ImmutableSet.<String>of()).metadata(
+ ImmutableMap.<String, Object>of("name",
datacenter.displayName(), "city", datacenter.city(), "state",
Yes. And now the locations are not going to be obtained by calling the
ComputeService adapter, so it is unlikely we'll need this function. I'd remove
it.
--
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/423#discussion_r182998406
