Skip to content

Commit

Permalink
fixup! feat: simplify args and include atlas
Browse files Browse the repository at this point in the history
  • Loading branch information
OmarIthawi committed Aug 6, 2023
1 parent b3158f6 commit 1fdce16
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions tutordiscovery/templates/discovery/build/discovery/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,11 @@ RUN pip install nodeenv==1.7.0
RUN nodeenv /openedx/nodeenv --node=16.14.2 --prebuilt
ENV PATH /openedx/nodeenv/bin:${PATH}

# Install the Open edX Atlas translation tool
RUN curl -L https://github.com/openedx/openedx-atlas/releases/download/v0.1.1/atlas \
-o /openedx/venv/bin/atlas && \
chmod +x /openedx/venv/bin/atlas

# Install python and nodejs requirements
# This is identical to "make production-requirements" but it was split in multiple
# instructions to benefit from docker image caching
Expand All @@ -64,11 +69,6 @@ RUN {% if is_buildkit_enabled() %}--mount=type=cache,target=/openedx/.cache/pip,

# Update i18n and collect static assets
{% if DISCOVERY_ATLAS_PULL %}
# Install the Open edX Atlas translation tool
RUN curl -L https://github.com/openedx/openedx-atlas/releases/v0.1.1/atlas \
-o /openedx/venv/bin/atlas && \
chmod +x /openedx/venv/bin/atlas

RUN find course_discovery/conf/locale -mindepth 1 -maxdepth 1 -type d -exec rm -r {} \; \
&& cd course_discovery/conf \
&& atlas pull {{ DISCOVERY_ATLAS_ARGS }} \
Expand Down

0 comments on commit 1fdce16

Please sign in to comment.