Workaround CI workflow for vfx 2022/2021 (#1931)

* Workaround CI workflow for vfx 2022/2021 Build for VFX reference platform prior to 2023, which requires a workaround to run an old version of glibc required by the ASWF Docker images. This doesn't easily integrate with the existing CI workflow, and it can be deprecated entirely once we've moved beyond support for the old CentOS-based VFX reference platform years. See this discussion for details: https://academysoftwarefdn.slack.com/archives/C0169RX7MMK/p1732574400981949 Signed-off-by: Cary Phillips <[email protected]> * remove centos7 label Signed-off-by: Cary Phillips <[email protected]> --------- Signed-off-by: Cary Phillips <[email protected]>
AcademySoftwareFoundation · Dec 2, 2024 · b51239d · b51239d
1 parent a393987
commit b51239d
Show file tree

Hide file tree

Showing 2 changed files with 105 additions and 34 deletions.
diff --git a/.github/workflows/ci_workflow.yml b/.github/workflows/ci_workflow.yml
@@ -55,7 +55,7 @@ jobs:
   # TODO: Add ARM build. Add sanitize build.
 
   linux:
-    name: 'Linux ${{ matrix.label }}VFXP-${{ matrix.vfx-cy }} centos7
+    name: 'Linux ${{ matrix.label }}VFXP-${{ matrix.vfx-cy }} 
        <${{ matrix.compiler-desc }},
        C++${{ matrix.cxx-standard }},
        config=${{ matrix.build-type }},
@@ -69,7 +69,7 @@ jobs:
       image: aswf/ci-openexr:${{ matrix.vfx-cy }}
     strategy:
       matrix:
-        build: [1, 2, 3, 4, 5, 6, 7, 8, 9]
+        build: [1, 2, 3, 4, 5, 6, 7]
         include:
           # -------------------------------------------------------------------
           # VFX CY2024 - GCC
@@ -174,38 +174,6 @@ jobs:
             vfx-cy: 2023
             exclude-tests:
 
-          # -------------------------------------------------------------------
-          # VFX CY2022 - GCC, Release
-          # -------------------------------------------------------------------
-          # Shared, Release
-          - build: 8
-            build-type: Release
-            build-shared: 'ON'
-            cxx-standard: 17
-            cxx-compiler: g++
-            cc-compiler: gcc
-            compiler-desc: gcc9.3.1
-            label: 
-            threads-enabled: 'ON'
-            vfx-cy: 2022
-            exclude-tests:
-
-          # -------------------------------------------------------------------
-          # VFX CY2021 - GCC, Release
-          # -------------------------------------------------------------------
-          # Shared, Release
-          - build: 9
-            build-type: Release
-            build-shared: 'ON'
-            cxx-standard: 17
-            cxx-compiler: g++
-            cc-compiler: gcc
-            compiler-desc: gcc9.3.1
-            label: 
-            threads-enabled: 'ON'
-            vfx-cy: 2021
-            exclude-tests:
-
     env:
       CXX: ${{ matrix.cxx-compiler }}
       CC: ${{ matrix.cc-compiler }}

diff --git a/.github/workflows/ci_workflow_old.yml b/.github/workflows/ci_workflow_old.yml
@@ -0,0 +1,103 @@
+# SPDX-License-Identifier: BSD-3-Clause
+# Copyright (c) Contributors to the OpenEXR Project.
+#
+# GitHub Actions workflow file
+# https://help.github.com/en/actions/reference/workflow-syntax-for-github-actions
+
+name: CI-old
+
+# Build for VFX reference platform prior to 2023, which requires a
+# workaround to run an old version of glibc required by the ASWF Docker
+# images.
+#
+# This doesn't easily integrate with the existing CI workflow, and it
+# can be depreciated entirely once we've moved beyond support for the
+# old CentOS-based VFX reference platform years.
+#
+# See this discussion for details:
+# https://academysoftwarefdn.slack.com/archives/C0169RX7MMK/p1732574400981949
+
+
+# Run on all changes except:
+# - doc file changes
+# - changes to the website, *except* for "website/src", since that
+#   code needs validation.  The website has a separate workflow
+# - changes to the bazel config, since it has its own workflow
+# - changes to the python bindings
+# - changes to workflows other than this one
+
+on:
+  push:
+    paths:
+      - '**'
+      - '!**.md'
+      - '!website/**'
+      - 'website/src/**'
+      - '!bazel/**'
+      - '!src/wrappers/**'
+      - '!.github/workflows/**'
+      - '.github/workflows/ci_workflow_old.yml'
+  pull_request:
+    paths:
+      - '**'
+      - '!**.md'
+      - '!website/**'
+      - 'website/src/**'
+      - '!bazel/**'
+      - '!src/wrappers/**'
+      - '!.github/workflows/**'
+      - '.github/workflows/ci_workflow_old.yml'
+
+permissions:
+  contents: read
+
+jobs:
+
+  linux:
+    name: 'Linux vfx${{ matrix.vfx-cy }}' 
+    runs-on: ubuntu-latest
+    container:
+      image: aswf/ci-openexr:${{ matrix.vfx-cy }}
+      volumes:
+        - /node20217:/node20217:rw,rshared
+        - /node20217:/__e/node20:ro,rshared
+
+    strategy:
+      matrix:
+        include:
+
+          - build: 2022
+            vfx-cy: 2022
+
+          - build: 2021
+            vfx-cy: 2021
+
+    steps:
+      - name: install nodejs20glibc2.17
+        run: |
+          curl --silent https://unofficial-builds.nodejs.org/download/release/v20.18.1/node-v20.18.1-linux-x64-glibc-217.tar.xz | tar -xJ --strip-components 1 -C /node20217 -f -
+      - name: Checkout
+        uses: actions/checkout@f43a0e5ff2bd294095638e18286ca9a3d1956744 # v3.6.0
+      - name: Create build directories
+        run: |
+          mkdir _install
+          mkdir _build
+      - name: Configure
+        run: |
+          cmake -B _build -S . \
+                -DCMAKE_INSTALL_PREFIX=_install \
+                -DCMAKE_VERBOSE_MAKEFILE:BOOL='ON'
+      - name: Build
+        run: |
+          cmake --build _build \
+                --target install \
+                --config Release
+      - name: Test
+        run: |
+          ctest -T Test \
+                -C Release \
+                --timeout 7200 \
+                --output-on-failure \
+                -VV
+        working-directory: _build
+