Address sandbox deployment issue (#1190)

* Changes to pass sandbox deployment issues * Remove explicit install of npm during deploy
w3c · Aug 19, 2024 · 39033e6 · 39033e6
1 parent 721d04e
commit 39033e6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git 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
@@ -25,7 +25,6 @@
     name:
       - nodejs
       - yarn
-      - npm
     state: present
   become: yes
-Original file line number
+Diff line change
@@ Expand Up / @@ -25,7 +25,6 @@ @@
         name:
           - nodejs
           - yarn
-          - npm
         state: present
       become: yes
@@ Expand Down @@