Merge pull request #642 from intersystems/master-ci-package-timeout

fix(ci): add timeout for each package, remove NS switch, bump action ver
intersystems · Dec 4, 2024 · ac53d10 · ac53d10
2 parents 84e7c93 + 45388ab
commit ac53d10
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/.github/workflows/packages.yml b/.github/workflows/packages.yml
@@ -53,8 +53,8 @@ jobs:
           CONTAINER=$(docker run -d --rm -v `pwd`:/home/irisowner/zpm/ zpm)
           docker exec $CONTAINER /usr/irissys/dev/Cloud/ICM/waitISC.sh
           docker exec -i $CONTAINER iris session IRIS << EOF
-              zpm "list":1
               zn "%SYS"
+              zpm "list":1
               zpm "test zpm -v -only":1:1
           EOF
           docker container stop $CONTAINER
@@ -78,7 +78,7 @@ jobs:
       - name: Set up Docker Buildx
         uses: docker/setup-buildx-action@v1
       - name: Download Artifact
-        uses: actions/download-artifact@v2
+        uses: actions/download-artifact@v4
         with:
           name: zpmimage
           path: /tmp
@@ -114,9 +114,10 @@ jobs:
           
             echo "::group::Test package $package"
             set +e
-            docker exec -i $CONTAINER iris session IRIS <<- EOF
-                  zpm "install $package":1
-                zpm "$package test -only ${{ env.test-flags }}":1:1
+            timeout 60s docker exec -i $CONTAINER iris session IRIS <<- EOF
+              zpm "install $package":1
+              zpm "$package test -only ${{ env.test-flags }}":1:1
+              halt
           EOF
 
             if [ $? -ne 0 ]; then