From 39033e692bee3219a0e88800e85528049f38e325 Mon Sep 17 00:00:00 2001 From: Howard Edwards Date: Mon, 19 Aug 2024 13:22:09 -0400 Subject: [PATCH] Address sandbox deployment issue (#1190) * Changes to pass sandbox deployment issues * Remove explicit install of npm during deploy --- deploy/roles/application/tasks/main.yml | 4 ++++ deploy/roles/nodejs/tasks/main.yml | 1 - 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/deploy/roles/application/tasks/main.yml b/deploy/roles/application/tasks/main.yml index b04d8b51c..ff6ef7434 100644 --- a/deploy/roles/application/tasks/main.yml +++ b/deploy/roles/application/tasks/main.yml @@ -17,6 +17,7 @@ path: '{{source_dir}}/server/scripts' mode: '0777' recurse: yes + become: yes when: deployment_mode != 'development' notify: "restart server" @@ -25,6 +26,7 @@ path: '{{source_dir}}/server/resources' mode: '0777' recurse: yes + become: yes when: deployment_mode != 'development' notify: "restart server" @@ -33,6 +35,7 @@ path: '{{source_dir}}/client/resources' mode: '0777' recurse: yes + become: yes when: deployment_mode != 'development' notify: "restart server" @@ -86,6 +89,7 @@ - name: Import latest tests and harness from w3c/aria-at shell: ../deploy/scripts/export-and-exec.sh {{environment_config.dest}} node ./scripts/import-tests/index.js + become: yes args: chdir: '{{source_dir}}/server' diff --git a/deploy/roles/nodejs/tasks/main.yml b/deploy/roles/nodejs/tasks/main.yml index d75dc0a62..e8d6c538c 100644 --- a/deploy/roles/nodejs/tasks/main.yml +++ b/deploy/roles/nodejs/tasks/main.yml @@ -25,7 +25,6 @@ name: - nodejs - yarn - - npm state: present become: yes