Skip to content
This repository has been archived by the owner on May 6, 2024. It is now read-only.

Commit

Permalink
feat: Add Atlas pull_translations after Makefile fixes
Browse files Browse the repository at this point in the history
This reverts commit 70eb142
and makes the following changes:

- No need to install atlas directly -- the openedx-atlas package
  actually has what we need already.
- After openedx/edx-platform#34306 we
  shouldn't encounter npm issues, so move task above NPM installs.
- Try dropping the DSM and CFG vars and see if it still works.
  • Loading branch information
timmc-edx committed Feb 28, 2024
1 parent 1ba584e commit 375755c
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions playbooks/roles/edxapp/tasks/deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -246,6 +246,18 @@
- install
- install:app-requirements

- name: "Pull translations using Atlas after Python dependencies installed"
shell: |
set -eu -o pipefail
source {{ edxapp_venv_dir }}/bin/activate
OPENEDX_ATLAS_PULL=true make pull_translations
args:
executable: /usr/bin/bash
chdir: "{{ edxapp_code_dir }}"
become_user: "{{ edxapp_user }}"
tags:
- install

# Set the npm registry
# This needs to be done as root since npm is weird about
# chown - https://github.com/npm/npm/issues/3565
Expand Down

0 comments on commit 375755c

Please sign in to comment.