m

tooling/build_autotriage.sh

@@ -135,25 +135,25 @@ identifyFailedBuildsInTimerPipelines() {
     triageThesePlatforms=""
     for p in "${!temurinPlatforms[@]}"
     do
-      if [[ shorterListOfBuilds =~ .*\"buildName\"\:\"jdk${arrayOfAllJDKVersions[v]}${arrayOfUs[v]}\-${temurinPlatforms[p]}\-temurin\".* ]]; then
-        if [[ arrayOfAllJDKVersions[v] -lt ${platformStart[p]} ]]; then
+      if [[ $shorterListOfBuilds =~ .*\"buildName\"\:\"jdk${arrayOfAllJDKVersions[v]}${arrayOfUs[v]}\-${temurinPlatforms[p]}\-temurin\".* ]]; then
+        if [[ ${arrayOfAllJDKVersions[v]} -lt ${platformStart[p]} ]]; then
           errorLog "Error: Platform ${temurinPlatforms[p]} should not be built for jdk${arrayOfAllJDKVersions[v]}${arrayOfUs[v]}. Will not triage."
           continue
         fi
-        if [[ arrayOfAllJDKVersions[v] -gt ${platformEnd[p]} ]]; then
+        if [[ ${arrayOfAllJDKVersions[v]} -gt ${platformEnd[p]} ]]; then
           errorLog "Error: Platform ${temurinPlatforms[p]} should not be built for jdk${arrayOfAllJDKVersions[v]}${arrayOfUs[v]}. Will not triage."
           continue
         fi
       else
-        if [[ arrayOfAllJDKVersions[v] -ge ${platformStart[p]} ]]; then
-          if [[ arrayOfAllJDKVersions[v] -le ${platformEnd[p]} ]]; then
+        if [[ ${arrayOfAllJDKVersions[v]} -ge ${platformStart[p]} ]]; then
+          if [[ ${arrayOfAllJDKVersions[v]} -le ${platformEnd[p]} ]]; then
             errorLog "Error: Platform ${temurinPlatforms[p]} should be built for jdk${arrayOfAllJDKVersions[v]}${arrayOfUs[v]}, but was not launched. "
             continue
           fi
         fi
       fi
       # If we get to this stage of the loop, then this is a platform that was both *meant* to be built, and *was* built (or attempted).
-      triageThesePlatforms="${triageThesePlatforms},jdk${arrayOfAllJDKVersions[v]}${arrayOfUs[v]}-${temurinPlatforms[p]}-temurin,"
+      triageThesePlatforms+=",jdk${arrayOfAllJDKVersions[v]}${arrayOfUs[v]}-${temurinPlatforms[p]}-temurin,"
     done
 
     echo "Platforms validated. Identifying build numbers for these platforms: ${triageThesePlatforms:1:-1}"