nacx commented on this pull request.


> +   OsImageToImage(@Memoized final Supplier<Set<Location>> locations,
+         Function<String, OsFamily> imageDescriptionToOsFamily) {
+      this.locations = locations;
+      this.imageDescriptionToOsFamily = imageDescriptionToOsFamily;
+   }
+
+   @Override
+   public Image apply(OsImage input) {
+
+      OsFamily osFamily = 
imageDescriptionToOsFamily.apply(input.description());
+      String osVersion = parseVersion(input.description());
+
+      OperatingSystem os = 
OperatingSystem.builder().name(input.name()).description(input.description())
+            
.family(osFamily).version(osVersion).is64Bit(is64bit(input)).build();
+
+      return new 
ImageBuilder().id(input.id()).name(input.name()).description(input.description())

I mean, you should think about it this way: calling 
`ComputeService.getImage(id)` with this ID will succeed? Or should the ID 
carry/encode additional info?

-- 
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/421#discussion_r153417324

Reply via email to