jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Zack Shoylev <notificati...@github.com>
Subject Re: [jclouds/jclouds] Updates the default template so live tests pass again. (#1041)
Date Mon, 21 Nov 2016 17:56:48 GMT
zack-shoylev commented on this pull request.



> @@ -80,9 +80,9 @@ public OperatingSystem apply(final Image from) {
             is64Bit = matcher.group(2).equals("x64");
          }
       } else {
-         if (imageName.contains("Red Hat EL")) {
+         if (imageName.contains("Red Hat")) {

Well, the imageName string changed from "EL" to "Enterprise Linux" at some point, which is
why the match was failing, but I don't know the original reasoning for being that specific.
Seems safe, though.

-- 
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/pull/1041
Mime
View raw message