Skip to content
Permalink
Browse files

Fix issue with public images (#1814)

Summary: This addresses #1814

Test Plan: manually tested new images

Reviewers: #ft, schrockn

Reviewed By: #ft, schrockn

Differential Revision: https://dagster.phacility.com/D1230
  • Loading branch information...
natekupp committed Oct 6, 2019
1 parent aa1490e commit 8b778472d0df6d9e3ff387f3de4ca93e2f09af08
Showing with 3 additions and 0 deletions.
  1. +1 −0 .buildkite/images/docker/Dockerfile
  2. +2 −0 .buildkite/images/docker/build.sh
@@ -57,6 +57,7 @@ RUN set -ex \
# Install deps
&& apt-get install -yqq --no-install-recommends \
bzip2 \
cron \
openjdk-8-jre-headless="$JAVA_DEBIAN_VERSION" \
openjdk-8-jdk-headless="$JAVA_DEBIAN_VERSION" \
nodejs \
@@ -34,13 +34,15 @@ rsync -av --exclude='*target*' --exclude='*.idea*' --exclude='*.class' $ROOT/sca
if [ $IMAGE_TYPE == "integration" ]; then
# Build the integration image
docker build . \
--no-cache \
--build-arg PYTHON_VERSION=$PYTHON_VERSION \
--build-arg PYTHON_MAJOR_VERSION=$PYTHON_MAJOR_VERSION \
--target dagster-integration-image \
-t "dagster/buildkite-integration:py${PYTHON_VERSION}-${IMAGE_VERSION}"
else
# Build the public image
docker build . \
--no-cache \
--build-arg PYTHON_VERSION=$PYTHON_VERSION \
--target dagster-public-image \
-t "dagster/dagster-py${PYTHON_MAJMIN}"

0 comments on commit 8b77847

Please sign in to comment.
You can’t perform that action at this time.