Revert "feat: replace openjdk with amazoncorretto:17.0.4 on connector…

…s for seсurity compliance (#17511)" (#17820) This reverts commit ab71f5b.
airbytehq · Oct 11, 2022 · e47c94e · e47c94e
1 parent 96f70ec
commit e47c94e
Show file tree

Hide file tree

Showing 7 changed files with 37 additions and 18 deletions.
diff --git a/airbyte-integrations/bases/base-java/Dockerfile b/airbyte-integrations/bases/base-java/Dockerfile
@@ -1,9 +1,7 @@
-ARG JDK_VERSION=17.0.4
-FROM amazoncorretto:${JDK_VERSION}
+ARG JDK_VERSION=17.0.1
+FROM openjdk:${JDK_VERSION}-slim
 COPY --from=airbyte/integration-base:dev /airbyte /airbyte
 
-RUN yum install -y tar openssl && yum clean all
-
 WORKDIR /airbyte
 
 COPY javabase.sh .

diff --git a/airbyte-integrations/bases/base-standard-source-test-file/Dockerfile b/airbyte-integrations/bases/base-standard-source-test-file/Dockerfile
@@ -1,12 +1,22 @@
-ARG JDK_VERSION=17.0.4
-FROM amazoncorretto:${JDK_VERSION}
+ARG JDK_VERSION=17.0.1
+FROM openjdk:${JDK_VERSION}-slim
 
 ARG DOCKER_BUILD_ARCH=amd64
 
 # Install Docker to launch worker images. Eventually should be replaced with Docker-java.
 # See https://gitter.im/docker-java/docker-java?at=5f3eb87ba8c1780176603f4e for more information on why we are not currently using Docker-java
-RUN amazon-linux-extras install -y docker
-RUN yum install -y openssl jq tar && yum clean all
+RUN apt-get update && apt-get install -y \
+                          apt-transport-https \
+                          ca-certificates \
+                          curl \
+                          gnupg-agent \
+                          software-properties-common
+RUN curl -fsSL https://download.docker.com/linux/debian/gpg | apt-key add -
+RUN add-apt-repository \
+       "deb [arch=${DOCKER_BUILD_ARCH}] https://download.docker.com/linux/debian \
+       $(lsb_release -cs) \
+       stable"
+RUN apt-get update && apt-get install -y docker-ce-cli jq
 
 ENV APPLICATION base-standard-source-test-file
 

diff --git a/airbyte-integrations/bases/base/Dockerfile b/airbyte-integrations/bases/base/Dockerfile
@@ -1,4 +1,4 @@
-FROM amazonlinux:2022.0.20220831.1
+FROM debian:10.5-slim
 
 WORKDIR /airbyte
 

diff --git a/airbyte-integrations/bases/standard-source-test/Dockerfile b/airbyte-integrations/bases/standard-source-test/Dockerfile
@@ -1,12 +1,22 @@
-ARG JDK_VERSION=17.0.4
-FROM amazoncorretto:${JDK_VERSION}
+ARG JDK_VERSION=17.0.1
+FROM openjdk:${JDK_VERSION}-slim
 
 ARG DOCKER_BUILD_ARCH=amd64
 
 # Install Docker to launch worker images. Eventually should be replaced with Docker-java.
 # See https://gitter.im/docker-java/docker-java?at=5f3eb87ba8c1780176603f4e for more information on why we are not currently using Docker-java
-RUN amazon-linux-extras install -y docker
-RUN yum install -y openssl jq tar && yum clean all
+RUN apt-get update && apt-get install -y \
+                          apt-transport-https \
+                          ca-certificates \
+                          curl \
+                          gnupg-agent \
+                          software-properties-common
+RUN curl -fsSL https://download.docker.com/linux/debian/gpg | apt-key add -
+RUN add-apt-repository \
+       "deb [arch=${DOCKER_BUILD_ARCH}] https://download.docker.com/linux/debian \
+       $(lsb_release -cs) \
+       stable"
+RUN apt-get update && apt-get install -y docker-ce-cli jq
 
 ENV APPLICATION standard-source-test
 

diff --git a/airbyte-integrations/connectors/destination-s3/Dockerfile b/airbyte-integrations/connectors/destination-s3/Dockerfile
@@ -19,11 +19,12 @@ RUN /bin/bash -c 'set -e && \
     ARCH=`uname -m` && \
     if [ "$ARCH" == "x86_64" ] || [ "$ARCH" = "amd64" ]; then \
        echo "$ARCH" && \
-       yum install lzop lzo lzo-dev -y; \
+       apt-get update; \
+       apt-get install lzop liblzo2-2 liblzo2-dev -y; \
     elif [ "$ARCH" == "aarch64" ] || [ "$ARCH" = "arm64" ]; then \
       echo "$ARCH" && \
-      yum group install -y "Development Tools" \
-      yum install lzop lzo lzo-dev wget curl unzip zip maven git -y; \
+      apt-get update; \
+      apt-get install lzop liblzo2-2 liblzo2-dev wget curl unzip zip build-essential maven git -y; \
       wget http://www.oberhumer.com/opensource/lzo/download/lzo-2.10.tar.gz -P /tmp; \
       cd /tmp && tar xvfz lzo-2.10.tar.gz; \
       cd /tmp/lzo-2.10/ && ./configure --enable-shared --prefix /usr/local/lzo-2.10; \

diff --git a/airbyte-integrations/connectors/destination-snowflake/Dockerfile b/airbyte-integrations/connectors/destination-snowflake/Dockerfile
@@ -1,7 +1,7 @@
 FROM airbyte/integration-base-java:dev
 
 # uncomment to run Yourkit java profiling
-#RUN yum install -y curl zip
+#RUN apt-get update && apt-get install -y curl zip
 #
 #RUN curl -o /tmp/YourKit-JavaProfiler-2021.3-docker.zip https://www.yourkit.com/download/docker/YourKit-JavaProfiler-2021.3-docker.zip && \
 #  unzip /tmp/YourKit-JavaProfiler-2021.3-docker.zip -d /usr/local && \

diff --git a/tools/bin/build_image.sh b/tools/bin/build_image.sh
@@ -42,7 +42,7 @@ if [ "$FOLLOW_SYMLINKS" == "true" ]; then
   # to use as the build context
   tar cL "${exclusions[@]}" . | docker build - "${args[@]}"
 else
-  JDK_VERSION="${JDK_VERSION:-17.0.4}"
+  JDK_VERSION="${JDK_VERSION:-17.0.1}"
   if [[ -z "${DOCKER_BUILD_PLATFORM}" ]]; then
     docker build --build-arg JDK_VERSION="$JDK_VERSION" --build-arg DOCKER_BUILD_ARCH="$DOCKER_BUILD_ARCH" . "${args[@]}"
   else