From eafa650465796fd0b8115e435ab762022dabce67 Mon Sep 17 00:00:00 2001 From: Kyle Shores Date: Fri, 14 Jun 2024 16:35:46 -0500 Subject: [PATCH] parallel builds, removing unused github actions steps --- .../workflows/fetch_content_integration.yml | 18 +----------------- docker/Dockerfile.fortran-gcc.integration | 4 ++-- docker/Dockerfile.fortran-intel | 5 ++--- docker/Dockerfile.fortran-nvhpc | 5 ++--- 4 files changed, 7 insertions(+), 25 deletions(-) diff --git a/.github/workflows/fetch_content_integration.yml b/.github/workflows/fetch_content_integration.yml index 3ec676b5..df04e847 100644 --- a/.github/workflows/fetch_content_integration.yml +++ b/.github/workflows/fetch_content_integration.yml @@ -41,20 +41,4 @@ jobs: run: docker build -t musica --build-arg MUSICA_GIT_TAG=${BRANCH_NAME} -f docker/${{ matrix.dockerfile }} . - name: Run tests in container - if: matrix.dockerfile != 'Dockerfile.coverage' - run: docker run --name test-container -t musica bash -c 'make test ARGS="--rerun-failed --output-on-failure -j8"' - - - name: Run coverage tests in container - if: matrix.dockerfile == 'Dockerfile.coverage' - run: docker run --name test-container -t musica bash -c 'make coverage ARGS="--rerun-failed --output-on-failure -j8"' - - - name: Copy coverage from container - if: matrix.dockerfile == 'Dockerfile.coverage' - run: docker cp test-container:build/coverage.info . - - - name: Upload coverage report - if: matrix.dockerfile == 'Dockerfile.coverage' - uses: codecov/codecov-action@v3 - with: - token: ${{ secrets.CODECOV_TOKEN }} - files: coverage.info \ No newline at end of file + run: docker run --name test-container -t musica bash -c 'make test ARGS="--rerun-failed --output-on-failure -j8"' \ No newline at end of file diff --git a/docker/Dockerfile.fortran-gcc.integration b/docker/Dockerfile.fortran-gcc.integration index 19a38b32..c4402811 100644 --- a/docker/Dockerfile.fortran-gcc.integration +++ b/docker/Dockerfile.fortran-gcc.integration @@ -39,7 +39,7 @@ RUN cd musica \ -B build \ -D CMAKE_BUILD_TYPE=Release \ && cd build \ - && make install + && make install -j # Build and install MUSICA RUN cd musica/fortran/test/fetch_content_integration \ @@ -49,7 +49,7 @@ RUN cd musica/fortran/test/fetch_content_integration \ -D MUSICA_GIT_TAG=${MUSICA_GIT_TAG} \ -D MUSICA_ENABLE_MICM=ON \ -D MUSICA_ENABLE_TUVX=OFF \ - && make + && make -j WORKDIR musica/fortran/test/fetch_content_integration/build RUN cp -r /musica/build/_deps/tuvx-src/examples/ . diff --git a/docker/Dockerfile.fortran-intel b/docker/Dockerfile.fortran-intel index 01994f40..d6f2fe61 100644 --- a/docker/Dockerfile.fortran-intel +++ b/docker/Dockerfile.fortran-intel @@ -50,10 +50,9 @@ COPY . musica RUN cd musica \ && cmake -S . \ -B build \ - -D MUSICA_ENABLE_TESTS=ON \ -D CMAKE_BUILD_TYPE=Release \ && cd build \ - && make install + && make install -j # Set environment variables to build MUSICA-Fortran using intel compilers ENV CC=icx @@ -66,7 +65,7 @@ RUN cd musica/fortran/test/fetch_content_integration \ -D CMAKE_BUILD_TYPE=Release \ -D CMAKE_EXE_LINKER_FLAGS="-Wl,--copy-dt-needed-entries" \ -D MUSICA_GIT_TAG=${MUSICA_GIT_TAG} \ - && make + && make -j WORKDIR musica/fortran/test/fetch_content_integration/build RUN cp -r /musica/build/_deps/tuvx-src/examples/ . diff --git a/docker/Dockerfile.fortran-nvhpc b/docker/Dockerfile.fortran-nvhpc index 7757ebb9..a9e8a220 100644 --- a/docker/Dockerfile.fortran-nvhpc +++ b/docker/Dockerfile.fortran-nvhpc @@ -50,10 +50,9 @@ COPY . musica RUN cd musica \ && cmake -S . \ -B build \ - -D MUSICA_ENABLE_TESTS=ON \ -D CMAKE_BUILD_TYPE=Release \ && cd build \ - && make install + && make install -j # Set environment variables to build MUSICA-Fortran using nvidia compilers ENV CXX=nvc++ @@ -66,7 +65,7 @@ RUN cd musica/fortran/test/fetch_content_integration \ -D CMAKE_BUILD_TYPE=Release \ -D CMAKE_EXE_LINKER_FLAGS="-Wl,--copy-dt-needed-entries" \ -D MUSICA_GIT_TAG=${MUSICA_GIT_TAG} \ - && make + && make -j WORKDIR musica/fortran/test/fetch_content_integration/build RUN cp -r /musica/build/_deps/tuvx-src/examples/ .