The previous attempt (#7117) was failing with an error in the call to paver i18n_compilejs after the Atlas-specific part of pull_translations, in the part where it runs npm ci. I don't have access to the npm log, but I bet it's just that our npm setup wasn't ready yet at this point in the playbook.
I think we just need to move this call to after the npm setup.
Make sure that the following steps are done before merging:
[x] Have a Site Reliability Engineer review the PR if you don't own all of the services impacted.
[x] If you are adding any new default values that need to be overridden when this change goes live, update internal repos and add an entry to the top of the CHANGELOG.
[ ] Performed the appropriate testing.
[x] Think about how this change will affect Open edX operators and update the wiki page for the next Open edX release if needed
The previous attempt (#7117) was failing with an error in the call to
paver i18n_compilejs
after the Atlas-specific part of pull_translations, in the part where it runsnpm ci
. I don't have access to the npm log, but I bet it's just that our npm setup wasn't ready yet at this point in the playbook.I think we just need to move this call to after the npm setup.
Part of OEP-58 and https://github.com/edx/edx-arch-experiments/issues/548
Configuration Pull Request
Make sure that the following steps are done before merging: