From e32dc86789541ff2b6b1acf032dea8a97a89c657 Mon Sep 17 00:00:00 2001
From: Anna Babu Palathingal <148897727+annaibm@users.noreply.github.com>
Date: Tue, 6 Feb 2024 09:32:15 -0500
Subject: [PATCH] Remove check_version in common_functions.sh (#4954)

- remove check_version function
- remove set_version function
- remove supported_versions
resolves : adoptium#4949

Signed-off-by: Anna Babu Palathingal <anna.bp@ibm.com>
---
 external/build_all.sh        | 45 +++++++++++++++++-------------------
 external/build_image.sh      |  2 --
 external/common_functions.sh | 30 +-----------------------
 3 files changed, 22 insertions(+), 55 deletions(-)

diff --git a/external/build_all.sh b/external/build_all.sh
index 7bcccb53d6..e22d0c81bb 100755
--- a/external/build_all.sh
+++ b/external/build_all.sh
@@ -25,27 +25,24 @@ cleanup_images
 # Loop through all possible images
 for test in ${supported_tests}
 do
-    for version in ${supported_versions}
-    do
-        for vm in ${supported_jvms}
-        do
-            for os in ${supported_os}
-            do
-                for package in ${supported_packages}
-                do
-                    for build in ${supported_builds}
-                    do
-                        echo "==============================================================================="
-                        echo "                                                                               "
-                        echo " Building Docker Images for ${test} ${version} ${vm} ${os} ${package} ${build} "
-                        echo "                                                                               "
-                        echo "==============================================================================="
-                        $(dirname "$0")/build_image.sh ${test} ${version} ${vm} ${os} ${package} ${build}
-                        echo
-                        echo
-                    done
-                done
-            done
-        done
-    done
-done
\ No newline at end of file
+	for vm in ${supported_jvms}
+	do
+		for os in ${supported_os}
+		do
+			for package in ${supported_packages}
+			do
+				for build in ${supported_builds}
+				do
+					echo "==============================================================================="
+					echo "                                                                               "
+					echo " Building Docker Images for ${test} ${version} ${vm} ${os} ${package} ${build} "
+					echo "                                                                               "
+					echo "==============================================================================="
+					$(dirname "$0")/build_image.sh ${test} ${version} ${vm} ${os} ${package} ${build}
+					echo
+					echo
+				done
+			done
+		done
+	done
+done
diff --git a/external/build_image.sh b/external/build_image.sh
index 30a6c5e914..99e8fa6bde 100755
--- a/external/build_image.sh
+++ b/external/build_image.sh
@@ -24,7 +24,6 @@ if [ $# -ne 9 ] && [ $# -ne 10 ]; then
 	echo
 	echo "usage: $0 test version vm os package build platform check_external_custom"
 	echo "test    = ${supported_tests}"
-	echo "version = ${supported_versions}"
 	echo "vm      = ${supported_jvms}"
 	echo "os      = ${supported_os}"
 	echo "package = ${supported_packages}"
@@ -42,7 +41,6 @@ check_external_custom=$9
 if [[ "${check_external_custom}" == "0" ]]; then
 	set_test $1
 fi
-set_version $2
 set_vm $3
 set_os $4
 set_package $5
diff --git a/external/common_functions.sh b/external/common_functions.sh
index da8e7eaa88..3d3b63b223 100755
--- a/external/common_functions.sh
+++ b/external/common_functions.sh
@@ -12,10 +12,6 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-
-# Supported versions
-supported_versions="8 11 17 21 22"
-
 # Supported JVMs
 supported_jvms="hotspot openj9"
 
@@ -32,30 +28,6 @@ supported_builds="full"
 # Supported tests
 supported_tests="external_custom camel criu-functional criu-portable-checkpoint criu-portable-restore criu-ubi-portable-checkpoint criu-ubi-portable-restore derby elasticsearch jacoco jenkins functional-test kafka lucene-solr openliberty-mp-tck payara-mp-tck quarkus quarkus_quickstarts scala system-test tomcat tomee wildfly wycheproof netty spring zookeeper"
 
-function check_version() {
-    version=$1
-
-    for current_version in ${supported_versions}
-    do
-        if [[ "${version}" == "${current_version}" ]]; then
-            return 1
-        fi
-    done
-
-    return 0
-}
-
-# Set a valid version
-function set_version() {
-    version=$1
-
-    if (check_version ${version}); then
-        echo "ERROR: Invalid Version: ${version}"
-        echo "Usage: $0 [${supported_versions}]"
-        exit 1
-    fi
-}
-
 function check_os() {
     os=$1
 
@@ -213,7 +185,7 @@ function set_test_info() {
     check_external_custom_test=$2
     cd ../
     path_to_file=$(pwd)
-    echo ${path_to_file}    
+    echo ${path_to_file}
     PROPERTY_FILE=${path_to_file}/${test}/test.properties
     github_url=$(getProperty "github_url")
     test_options=$(getProperty "test_options")