Skip to content

Commit

Permalink
address PR comments
Browse files Browse the repository at this point in the history
Co-authored-by: Chesnay Schepler <chesnay@apache.org>
  • Loading branch information
rmetzger and zentol committed Jul 29, 2020
1 parent 4a1b98b commit b7ccc16
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 10 deletions.
2 changes: 1 addition & 1 deletion add-version.sh
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ for source_variant in "${SOURCE_VARIANTS[@]}"; do

mkdir "$dir"
generateDockerfile "${dir}" "${flink_tgz_url}" "${flink_asc_url}" ${gpg_key} true ${java_version} ${source_variant}
generateReleaseMetadata ${dir} ${flink_release} ${flink_version} ${scala_version} ${java_version} ${source_variant}
generateReleaseMetadata "${dir}" ${flink_release} ${flink_version} ${scala_version} ${java_version} ${source_variant}
done
done
done
Expand Down
14 changes: 5 additions & 9 deletions generator.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,9 @@

export SOURCE_VARIANTS=(debian )

export LATEST_SCALA="2.12"
export LATEST_JAVA="8"
export DEFAULT_SCALA="2.12"
export DEFAULT_JAVA="8"

# generate "Dockerfile" file
function generateDockerfile {
# define variables
dir=$1
Expand All @@ -18,8 +17,6 @@ function generateDockerfile {

from_docker_image="openjdk:${java_version}-jre"

source_file="Dockerfile-debian"

cp docker-entrypoint.sh "$dir/docker-entrypoint.sh"

# '&' has special semantics in sed replacement patterns
Expand All @@ -32,10 +29,9 @@ function generateDockerfile {
-e "s/%%GPG_KEY%%/$gpg_key/" \
-e "s/%%CHECK_GPG%%/${check_gpg}/" \
-e "s/%%FROM_IMAGE%%/${from_docker_image}/" \
"$source_file.template" > "$dir/Dockerfile"
"Dockerfile-$source_variant.template" > "$dir/Dockerfile"
}

# generate "release.metadata" file
function generateReleaseMetadata {
dir=$1
flink_release=$2
Expand All @@ -57,14 +53,14 @@ function generateReleaseMetadata {

tags="$full_tag, $short_tag, $scala_tag"

if [[ "$scala_version" == "$LATEST_SCALA" ]]; then
if [[ "$scala_version" == "$DEFAULT_SCALA" ]]; then
# we are generating the image for the latest scala version, add:
# "1.2.0-java11"
# "1.2-java11"
# "latest-java11"
tags="$tags, ${flink_version}${java_suffix}, ${flink_release}${java_suffix}, latest${java_suffix}"

if [[ "$java_version" == "$LATEST_JAVA" ]]; then
if [[ "$java_version" == "$DEFAULT_JAVA" ]]; then
# we are generating the image for the default java version, add tags w/o java tag:
# "1.2.0"
# "1.2"
Expand Down

0 comments on commit b7ccc16

Please sign in to comment.