-
Notifications
You must be signed in to change notification settings - Fork 53
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.
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
Include Docker client executable in the PCT Docker image #207
Conversation
|
||
RUN curl -L --show-error https://download.java.net/java/GA/jdk11/13/GPL/openjdk-11.0.1_linux-x64_bin.tar.gz --output openjdk.tar.gz && \ | ||
echo "7a6bb980b9c91c478421f865087ad2d69086a0583aeeb9e69204785e8e97dcfd openjdk.tar.gz" | sha256sum -c && \ | ||
tar xvzf openjdk.tar.gz && \ | ||
mv jdk-11.0.1/ /usr/lib/jvm/java-11-openjdk-amd64 && \ | ||
rm openjdk.tar.gz | ||
|
||
RUN curl -fsSL https://download.docker.com/linux/debian/gpg | apt-key add - |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you move this before line 48 you can use the apt call in line 48 to prevent calling apt twice
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is not possible as you need first install the packages to be able to add properly the repository
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am fine with this change. Note that you are going into the Docker-in-Docker spaces, and all considerations should be taken into account. But I see no problem with getting the executable included
comments were addressed
Testing PCT for new version 1.33 for the durable-task plugin, we need to have docker installed in the PCT docker image to make test can run properly
cc/ @oleg-nenashev @dwnusbaum @raul-arabaolaza @batmat