andreaturli commented on this pull request.


> +      if (hardware.isPresent()) {
+         builder.hardware(hardware.get());
+      } else {
+         logger.info(">> hardware with id %s for instance %s was not found. "
+                         + "This might be because the image that was used to 
create the instance has a new id.",
+                 from.instanceType(), from.instanceId());
+      }
+
+      builder.id(RegionAndId.slashEncodeRegionAndId(from.regionId(), 
from.instanceId()));
+      builder.providerId(from.instanceId());
+      builder.name(from.instanceName());
+      builder.hostname(String.format("%s", from.hostname()));
+      builder.group(groupNamingConvention.extractGroup(from.instanceName()));
+      builder.status(toPortableStatus.apply(from.status()));
+      
builder.privateAddresses(from.innerIpAddress().entrySet().iterator().next().getValue());
+      
builder.publicAddresses(from.publicIpAddress().entrySet().iterator().next().getValue());

it's a weird data structure, getValue returns a List! :D

-- 
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/443#discussion_r207135700

Reply via email to