updates aws docker file instructions includes necessary configuration…

… to bound UCX port assignment
cylondata · Aug 2, 2023 · 3064b04 · 3064b04
1 parent ab43bc8
commit 3064b04
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/aws/scripts/runCylon.sh b/aws/scripts/runCylon.sh
@@ -12,5 +12,5 @@ conda activate cylon_dev
 set -euo pipefail
 
 export LD_LIBRARY_PATH=/cylon/install/lib
-export UCX_TCP_PORT_RANGE="33000"
+
 exec python /cylon/aws/scripts/S3_run_script.py
diff --git a/docker/aws/Dockerfile b/docker/aws/Dockerfile
@@ -10,6 +10,14 @@ ENV CYLON_HOME=/cylon
 ENV CYLON_PREFIX=/cylon/install
 ENV UCC_HOME=/ucc
 
+#give ARG EXPOSE_ENV a default value
+ARG EXPOSE_ENV=33000
+
+#assign the EXPOSE_ENV arg to the EXPOSE_ENV ENV so that it can be accessed
+#by the subsequent RUN call within the container
+ENV EXPOSE_ENV $EXPOSE_ENV
+
+
 WORKDIR $CYLON_HOME
 
 RUN DEBIAN_FRONTEND=noninteractive apt-get update -y --fix-missing && \
@@ -76,6 +84,6 @@ RUN . /opt/conda/etc/profile.d/conda.sh && \
 
 RUN chmod 0755 /cylon/aws/scripts/runCylon.sh
 
-EXPOSE 33000
+EXPOSE $EXPOSE_ENV
 
 ENTRYPOINT ["/cylon/aws/scripts/runCylon.sh"]