remove extra CMD from containers (#390)

* add entrypoint to all containers

* remove -e for pip installs

* add keyring to fix bad key nvidia issue

* trying to move env call to see if it helps fix the install of distributed embeddings after sok install

* fix extra CVE

* fixing CI routes

* fix merlin merge

* remove extra CMD that replaces previous entrypoint

Co-authored-by: Karl Higley <kmhigley@gmail.com>

docker/dockerfile.ctr

@@ -277,6 +277,3 @@ RUN rm /usr/local/cuda/lib64/stubs/libcuda.so.1
 RUN rm -rf /repos
 RUN rm -rf /usr/local/share/jupyter/lab/staging/node_modules/marked
 RUN rm -rf /usr/local/share/jupyter/lab/staging/node_modules/node-fetch
-
-HEALTHCHECK NONE
-CMD ["/bin/bash"]

docker/dockerfile.tf

@@ -61,7 +61,3 @@ RUN if [ "$INSTALL_DISTRIBUTED_EMBEDDINGS" == "true" ]; then \
         cd /distributed_embeddings && git checkout ${TFDE_VER} && \
         make pip_pkg && pip install artifacts/*.whl && make clean; \
     fi
-
-
-HEALTHCHECK NONE
-CMD ["/bin/bash"]

docker/dockerfile.torch

@@ -35,6 +35,3 @@ COPY --chown=1000:1000 --from=dlfw /opt/conda/lib/python3.8/site-packages/torch
 
 RUN pip install --no-deps fastai fastcore fastprogress fastdownload
 RUN pip install matplotlib
-
-HEALTHCHECK NONE
-CMD ["/bin/bash"]