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.

Sign up for GitHub

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

Jump to bottom

Add jar/zip support #2280

Merged
merged 6 commits into from
Dec 21, 2023
Merged

Add jar/zip support #2280

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
98d1ba7
Add jar/zip support
showpune Dec 15, 2023
047e331
Use content type to decision the unzip
showpune Dec 16, 2023
59a47d5
Remove the message header file not found
showpune Dec 19, 2023
751348e
Use extension
showpune Dec 19, 2023
3b2c825
move sleep after extension check
showpune Dec 20, 2023
01280b3
Merge branch 'microsoft:main' into zhiyongli/jarsupport
showpune Dec 21, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions builders/container-apps-internal-registry-demo/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ RUN set -eux; \
sudo \
curl \
unzip \
grep \
showpune marked this conversation as resolved.
Show resolved Hide resolved
; \
rm -rf /var/lib/apt/lists/*

Expand Down
27 changes: 24 additions & 3 deletions builders/container-apps-internal-registry-demo/startup-script.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ file_upload_endpoint="$FILE_UPLOAD_CONTAINER_URL/$FILE_UPLOAD_BLOB_NAME"

temp_app_source_dir="/tmp/appsource"
temp_app_source_path="$temp_app_source_dir/$FILE_UPLOAD_BLOB_NAME"
temp_app_header_path="$temp_app_source_dir/header.txt"
mkdir $temp_app_source_dir

# List all the environment variables and filter the environment variable with prefix "ACA_CLOUD_BUILD_USER_ENV_", then write them to folder "/platform/env",
Expand All @@ -25,21 +26,41 @@ done

# write environment variable CORRELATION_ID to folder "/platform/env",
if [ -n "$CORRELATION_ID" ]; then
echo "$CORRELATION_ID" > "$build_env_dir/CORRELATION_ID"
echo -n "$CORRELATION_ID" > "$build_env_dir/CORRELATION_ID"
fi

file_extension=""
while [[ ! -f "$temp_app_source_path" || ! "$(file $temp_app_source_path)" =~ "compressed data" ]]
do
echo "Waiting for app source to be uploaded. Please upload the app source to the endpoint specified in the Build resource's 'uploadEndpoint' property."
curl -H "$auth_header" -H "$version_header" -H "$date_header" -X GET "$file_upload_endpoint" -o "$temp_app_source_path" -s
curl -H "$auth_header" -H "$version_header" -H "$date_header" -X GET "$file_upload_endpoint" -o "$temp_app_source_path" -D "$temp_app_header_path" -s
sleep 5
showpune marked this conversation as resolved.
Show resolved Hide resolved

if [[ -f "$temp_app_header_path" ]]; then
file_extension=$(grep -i x-ms-meta-FileExtension "$temp_app_header_path" | cut -d ' ' -f2)
# Check if the original file extension is .jar, .war, .zip or .tar.gz
if [[ "$file_extension" =~ ".tar.gz"
|| "$file_extension" =~ ".jar"
|| "$file_extension" =~ ".war"
|| "$file_extension" =~ ".zip" ]]; then
break
fi
fi
done

# Extract app code to CNB_APP_DIR directory.
echo "Found app source at '$temp_app_source_path'. Extracting to $CNB_APP_DIR"
mkdir -p $CNB_APP_DIR
cd $CNB_APP_DIR
tar -xzf "$temp_app_source_path"

if [[ "$file_extension" =~ ".jar"
|| "$file_extension" =~ ".war"
|| "$file_extension" =~ ".zip" ]]; then
unzip -qq "$temp_app_source_path"
else
# Keep compatibility with old logic
tar -xzf "$temp_app_source_path"
fi

fileCount=$(ls | wc -l)
if [ "$fileCount" = "1" ]; then
Expand Down