Skip to content

Commit b00627f

Browse files
authored
chore: Add ca-certificates to the worker's runtime image to support reading signed S3 URLs in the CLP connector (fixes #56). (#55)
1 parent 32e6b12 commit b00627f

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

presto-native-execution/scripts/dockerfiles/prestissimo-runtime.dockerfile

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -39,12 +39,15 @@ FROM ${BASE_IMAGE}
3939
ENV BUILD_BASE_DIR=_build
4040
ENV BUILD_DIR=""
4141

42-
# Temporary fix for https://github.com/prestodb/presto/issues/25531
43-
# TODO: Update this code when there's a proper fix
44-
RUN apt-get update && \
45-
DEBIAN_FRONTEND=noninteractive apt-get install -y tzdata && \
46-
apt-get clean && \
47-
rm -rf /var/lib/apt/lists/*
42+
# NOTE:
43+
# - We need `ca-certificates` to support reads from signed S3 URLs.
44+
# - We need `tzdata` as a temporary workaround for https://github.com/prestodb/presto/issues/25531
45+
RUN apt-get update \
46+
&& DEBIAN_FRONTEND=noninteractive apt-get install -y \
47+
ca-certificates \
48+
tzdata \
49+
&& apt-get clean \
50+
&& rm -rf /var/lib/apt/lists/*
4851

4952
COPY --chmod=0775 --from=prestissimo-image /prestissimo/${BUILD_BASE_DIR}/${BUILD_DIR}/presto_cpp/main/presto_server /usr/bin/
5053
COPY --chmod=0775 --from=prestissimo-image /runtime-libraries/* /usr/lib64/prestissimo-libs/

0 commit comments

Comments
 (0)