Skip to content

Commit a17f03f

Browse files
committed
fix: poppler setup needs to be a dedicated layer
1 parent b2f109f commit a17f03f

3 files changed

Lines changed: 16 additions & 32 deletions

File tree

.github/workflows/build-release.yml

Lines changed: 14 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ jobs:
4646
run: |
4747
cp target/lambda/docbox-http-lambda/bootstrap.zip docbox-http-lambda-amd64.zip
4848
cp target/lambda/docbox-presigned-cleanup-lambda/bootstrap.zip docbox-presigned-cleanup-lambda-amd64.zip
49-
cp target/lambda/docbox-upload-completion-lambda/bootstrap.zip docbox-upload-completion-lambda.zip
49+
cp target/lambda/docbox-upload-completion-lambda/bootstrap.zip docbox-upload-completion-lambda-amd64.zip
5050
shell: bash
5151

5252
# Build the docker layers
@@ -56,20 +56,6 @@ jobs:
5656
bash ./layers/make-poppler-layer-amd64.sh
5757
shell: bash
5858

59-
# Merge processing lambda with poppler layer
60-
# (Required for processing pdf files)
61-
- name: Merge processing layers
62-
run: |
63-
mkdir /tmp/processing-lambda
64-
unzip docbox-upload-completion-lambda.zip -d /tmp/processing-lambda
65-
unzip poppler-lambda-layer.zip -d /tmp/processing-lambda
66-
(
67-
cd /tmp/processing-lambda
68-
zip -r /tmp/docbox-upload-completion-lambda-amd64.zip .
69-
)
70-
mv /tmp/docbox-upload-completion-lambda-amd64.zip .
71-
shell: bash
72-
7359
# Upload the built artifacts
7460
- name: Upload docbox-http-lambda.zip
7561
uses: actions/upload-artifact@v4
@@ -89,6 +75,12 @@ jobs:
8975
name: docbox-upload-completion-lambda-amd64.zip
9076
path: docbox-upload-completion-lambda-amd64.zip
9177

78+
- name: Upload poppler-lambda-layer-amd64.zip
79+
uses: actions/upload-artifact@v4
80+
with:
81+
name: poppler-lambda-layer-amd64.zip
82+
path: poppler-lambda-layer-amd64.zip
83+
9284
build-arm64:
9385
name: Build ARM64
9486
runs-on: ubuntu-latest
@@ -121,7 +113,7 @@ jobs:
121113
run: |
122114
cp target/lambda/docbox-http-lambda/bootstrap.zip docbox-http-lambda-arm64.zip
123115
cp target/lambda/docbox-presigned-cleanup-lambda/bootstrap.zip docbox-presigned-cleanup-lambda-arm64.zip
124-
cp target/lambda/docbox-upload-completion-lambda/bootstrap.zip docbox-upload-completion-lambda.zip
116+
cp target/lambda/docbox-upload-completion-lambda/bootstrap.zip docbox-upload-completion-lambda-arm64.zip
125117
shell: bash
126118

127119
- name: Set up QEMU
@@ -137,20 +129,6 @@ jobs:
137129
bash ./layers/make-poppler-layer-arm64.sh
138130
shell: bash
139131

140-
# Merge processing lambda with poppler layer
141-
# (Required for processing pdf files)
142-
- name: Merge processing layers
143-
run: |
144-
mkdir /tmp/processing-lambda
145-
unzip docbox-upload-completion-lambda.zip -d /tmp/processing-lambda
146-
unzip poppler-lambda-layer.zip -d /tmp/processing-lambda
147-
(
148-
cd /tmp/processing-lambda
149-
zip -r /tmp/docbox-upload-completion-lambda-arm64.zip .
150-
)
151-
mv /tmp/docbox-upload-completion-lambda-arm64.zip .
152-
shell: bash
153-
154132
# Upload the built artifacts
155133
- name: Upload docbox-http-lambda-arm64.zip
156134
uses: actions/upload-artifact@v4
@@ -170,6 +148,12 @@ jobs:
170148
name: docbox-upload-completion-lambda-arm64.zip
171149
path: docbox-upload-completion-lambda-arm64.zip
172150

151+
- name: Upload poppler-lambda-layer-arm64.zip
152+
uses: actions/upload-artifact@v4
153+
with:
154+
name: poppler-lambda-layer-arm64.zip
155+
path: poppler-lambda-layer-arm64.zip
156+
173157
release:
174158
name: Create Release
175159
runs-on: ubuntu-latest

layers/make-poppler-layer-amd64.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,5 +6,5 @@ docker buildx build --build-arg BASE_IMAGE=public.ecr.aws/lambda/provided:al2023
66

77
# Run a container and copy out the zip then delete it
88
CONTAINER_ID=$(docker create --platform linux/amd64 poppler-lambda-layer-amd64)
9-
docker cp $CONTAINER_ID:/tmp/poppler-lambda-layer.zip ./poppler-lambda-layer.zip
9+
docker cp $CONTAINER_ID:/tmp/poppler-lambda-layer.zip ./poppler-lambda-layer-amd64.zip
1010
docker rm $CONTAINER_ID

layers/make-poppler-layer-arm64.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,5 +6,5 @@ docker buildx build --build-arg BASE_IMAGE=public.ecr.aws/lambda/provided:al2023
66

77
# Run a container and copy out the zip then delete it
88
CONTAINER_ID=$(docker create --platform linux/arm64 poppler-lambda-layer-arm64)
9-
docker cp $CONTAINER_ID:/tmp/poppler-lambda-layer.zip ./poppler-lambda-layer.zip
9+
docker cp $CONTAINER_ID:/tmp/poppler-lambda-layer.zip ./poppler-lambda-layer-arm64.zip
1010
docker rm $CONTAINER_ID

0 commit comments

Comments
 (0)