mdavidsaver / ci-core-dumper

Utility to automate analysis of core dumps from crashes during CI builds and test runs
GNU General Public License v3.0
5 stars 2 forks source link

Update to node 20 doesn't work with Centos 7 #4

Open simon-ess opened 3 months ago

simon-ess commented 3 months ago

See https://github.com/actions/runner/issues/2906 or https://github.com/epics-base/epics-base/actions/runs/8279118587/job/22652857166?pr=468 for an example.

mdavidsaver commented 3 months ago

For what it may be worth, I have created a node16 branch initially pointing to 96219fd5fbe37e99841e205ee61d14b410bf7aed the revision prior to merging #3.

mdavidsaver commented 3 months ago

A temporary workaround... (mdavidsaver/pvxs@3e93627f375776e843b5e2980ec7e8ee723bd868)

     - name: Automatic core dumper analysis
       uses: mdavidsaver/ci-core-dumper@master
+      if: matrix.image!='centos:7'
+    - name: Automatic core dumper analysis
+      uses: mdavidsaver/ci-core-dumper@node16
+      if: matrix.image=='centos:7'
mdavidsaver commented 3 months ago

Unless the github folks start injecting compatible libraries along with their Node binaries, the options I see are:

  1. Abandon centos:7 testing
  2. Switch to manual invocation of docker

Since sadly centos seven seems set to be with us for some time, I suspect I will end up with option 2. For an example of what this could look like, see the epicscorelibs builds for old manylinux1 images.

https://github.com/mdavidsaver/epicscorelibs/blob/cfdc449eb0e4e4efd6beac3eb323051c9a2128aa/.github/workflows/build.yml#L238