andreaturli commented on this pull request.


> -               family = 
> OsFamily.fromValue(image.versionProperties().osDiskImage().operatingSystem().toUpperCase());
+            if (OsFamily.UNRECOGNIZED == family) {
+//               &&
+//            } image.versionProperties() != null
+//                  && image.versionProperties().osDiskImage() != null
+//                  && 
image.versionProperties().osDiskImage().operatingSystem() != null) {
+               
+               
+               
+               
+//               Version versionDetails = osImageApi.getVersion(publisherName, 
offer.name(), sku.name(), version.name());
+//                              VMImage vmImage = 
VMImage.azureImage().publisher(publisherName).offer(offer.name()).sku(sku.name())
+//                                    
.version(versionDetails.name()).location(location).versionProperties(versionDetails.properties())
+//                                    .build();
+
+//               family = 
OsFamily.fromValue(image.versionProperties().osDiskImage().operatingSystem().toUpperCase());
             }
 
             // only 64bit OS images are supported by Azure ARM

ops


-- 
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/351

Reply via email to