chore: Add more build artifacts for debugging (#749)

* chore: Add more build artifacts for debugging

* Not always

* npm only on failure

Author: lorensr

.github/workflows/ci.yml

@@ -63,6 +63,14 @@ jobs:
           # TODO: Run integration tests on MacOS / Windows probably using temporalite
           RUN_INTEGRATION_TESTS: ${{ startsWith(matrix.os, 'ubuntu') }}
 
+      - uses: actions/upload-artifact@v3
+        if: ${{ failure() }}
+        with:
+          name: npm-logs
+          path: |
+            ~/.npm/_logs/
+            C:\npm\cache\_logs\
+
       # Do docs stuff (only on one host)
       - name: Build docs
         if: ${{ matrix.docsTarget }}

.github/workflows/stress.yml

@@ -54,11 +54,10 @@ jobs:
       - run: mkdir -p $TEMPORAL_TESTING_LOG_DIR/tails
       - run: mkdir -p $TEMPORAL_TESTING_MEM_LOG_DIR
       - run: 'timeout ${{ inputs.test-timeout-minutes }}m npm run ${{ inputs.test-type }}'
-      # Tail the logs and upload as artifact on cancel or failure
       - run: for f in $TEMPORAL_TESTING_LOG_DIR/*.log; do tail -20000 $f > $TEMPORAL_TESTING_LOG_DIR/tails/$(basename $f); done
-        if: ${{ cancelled() || failure() }}
+        if: ${{ always() }}
       - uses: actions/upload-artifact@v3
-        if: ${{ cancelled() || failure() }}
+        if: ${{ always() }}
         with:
           name: worker-logs
           path: ${{ env.TEMPORAL_TESTING_LOG_DIR }}/tails