> +                      .id("")
> +                      .ram(container.getConfig().getMemory())
> +                      .processor(new 
> Processor(container.getConfig().getCpuShares(), 
> container.getConfig().getCpuShares()))
> +                      .build());
> +      // TODO Set up location properly
> +      LocationBuilder locationBuilder = new LocationBuilder();
> +      locationBuilder.description("");
> +      locationBuilder.id("");
> +      locationBuilder.scope(LocationScope.HOST);
> +      builder.location(locationBuilder.build());
> +      builder.status(toPortableStatus.apply(container.getState()));
> +      builder.imageId(container.getImage());
> +      builder.loginPort(getLoginPort(container));
> +      builder.publicAddresses(getPublicIpAddresses());
> +      builder.privateAddresses(getPrivateIpAddresses(container));
> +      
> builder.operatingSystem(OperatingSystem.builder().description("linux").family(OsFamily.LINUX).build());

But you have the `imageId` (you're setting it a few lines before). You could 
use that to get the corresponding `Image` from the `ImageSupplier` and populate 
the details of its operating system? That should be closer to the real one than 
the hardcoded Linux.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/57/files#r12953837

Reply via email to