Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update al2 image checks for 22 and remove al2023 from generic linux #200

Merged
merged 8 commits into from
Mar 25, 2024

Conversation

elifaslan1
Copy link
Contributor

Update docker file to not check al2 for 22

@elifaslan1 elifaslan1 requested review from lutkerd and Rudometov March 25, 2024 18:00
@@ -4,7 +4,7 @@
ALPINE_VERSIONS = ['3.16', '3.17', '3.18', '3.19']

LTS_VERSIONS = [ "8", "11", "17", "21"]
AL2023_GENERIC_VERSIONS = ["20"]
AL2023_GENERIC_VERSIONS = ["22"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Original intention of this was to have a AL2023 image with the generic linux package, which we don't need anymore since we have the native packages.

@@ -5,6 +5,8 @@ SED="sed -i"
sed --version 2>/dev/null || SED="sed -i.bkp"

LTS_VERSIONS=("8" "11" "17" "21")
AL2_VERSIONS=("8" "11" "17" "21")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you rename this to AL2_GENERIC_LINUX_VERSIONS? Other option would be to remove this and change the check on line 57 to see if the file exists, like we do for al2023-generic.

${SED} "s/ARG version=.*/ARG version=${jdk_version}.${jdk_build}-${corretto_version}/g" ./${MAJOR_RELEASE}/jdk/al2-generic/Dockerfile
if [[ "${AL2_VERSIONS[*]}" =~ ${MAJOR_RELEASE} ]]; then
${SED} "s/ARG version=.*/ARG version=${jdk_version}.${jdk_build}-${corretto_version}/g" ./${MAJOR_RELEASE}/jdk/al2-generic/Dockerfile
fi
if [[ -f ./${MAJOR_RELEASE}/jdk/al2023-generic/Dockerfile ]]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can remove this as we don't think al2023 with generic linux packages anymore.

@elifaslan1 elifaslan1 changed the title Update al2for22 Update al2 image checks for 22 and remove al2023 from generic linux Mar 25, 2024
@elifaslan1 elifaslan1 merged commit 725a51c into main Mar 25, 2024
116 checks passed
@elifaslan1 elifaslan1 deleted the updateAL2for22 branch March 25, 2024 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants