Skip to content
Snippets Groups Projects

Draft: CITest: add test for WorkDir configuration

Open Frank Winklmeier requested to merge fwinkl/athena:WorkDir_test into main

Add a minimal CI test to each project to make sure the WorkDir package can be configured against the built release. This test only works when run within the CI and for MRs that affect the Build domain. Otherwise it will be marked as "Skipped" in the ctest result.

This may help in discovering problems as seen with ATLINFR-5571 earlier.

Edited by Frank Winklmeier

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Frank Winklmeier changed title from Draft: CITest: add CI test for WorkDir configuration to Draft: CITest: add test for WorkDir configuration

    changed title from Draft: CITest: add CI test for WorkDir configuration to Draft: CITest: add test for WorkDir configuration

  • Frank Winklmeier changed the description

    changed the description

  • Author Maintainer

    Jenkins please retry a build

    • Author Maintainer

      @akraszna, this is a bit experimental. There may be better ways of achieving the same. But let's see...

    • Author Maintainer

      OK, this is probably more complicated than I thought as we don't actually install the projects during the CI.

    • Yeah... We neither install the build, nor do we set up the environment with that installation in it.

      If we wanted to do this, we'd need to dedicate some measurable additional resources to it in the CI. (I.e. the CI would have to become slower.) I agree that testing this would be great. But it might be just a little too difficult to do correctly.

      You know what...? There might be a way of hitting two birds with one stone here. 🤔 It could be possible to make the CI install a project before building it. Since as long as all our infrastructure works correctly, installation even without any binary files (or outdated ones), should work.

      But that's not great either. On the second run there would already be binaries from the first run. So the installation would be just as slow as "in a normal setup" anyway. 🤔 So I don't think I have a good proposal for this after all. 😦

    • Author Maintainer

      Is installing really that slow? I remember it taking maybe a few minutes. If we would execute the install as part of the test itself, and only run the test if changes to Projects/ were made, it may not be that bad. What do you think?

      Edited by Frank Winklmeier
    • Author Maintainer

      I pushed a new version that installs the project as part of the test. I used the "official" installation directory next to the already installed externals. If we are afraid that may interfere with sub-subsequent CI runs, we can also choose an entirely separate installation area.

    • Please register or sign in to reply
  • This merge request affects 1 package:

    • AtlasTest/CITest

    This merge request affects 6 files:

    • AtlasTest/CITest/AnalysisBase.cmake
    • AtlasTest/CITest/AthAnalysis.cmake
    • AtlasTest/CITest/AthGeneration.cmake
    • AtlasTest/CITest/AthSimulation.cmake
    • AtlasTest/CITest/Athena.cmake
    • AtlasTest/CITest/test/WorkDirConfig.sh

    Adding @fwinkl ,@pagessin ,@jchapman as watchers

  • CI Result FAILURE (hash 200c51c2)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon
    externals
    cmake
    make
    tests

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    Athena: number of compilation errors 0, warnings 0
    AthSimulation: number of compilation errors 0, warnings 0
    AthGeneration: number of compilation errors 0, warnings 0
    AnalysisBase: number of compilation errors 0, warnings 0
    AthAnalysis: number of compilation errors 0, warnings 0
    DetCommon: number of compilation errors 0, warnings 0
    📝 For experts only: Jenkins output (remote access info)

  • Frank Winklmeier added 244 commits

    added 244 commits

    Compare with previous version

  • Author Maintainer

    Jenkins please retry a build

  • This merge request affects 1 package:

    • AtlasTest/CITest

    This merge request affects 6 files:

    • AtlasTest/CITest/AnalysisBase.cmake
    • AtlasTest/CITest/AthAnalysis.cmake
    • AtlasTest/CITest/AthGeneration.cmake
    • AtlasTest/CITest/AthSimulation.cmake
    • AtlasTest/CITest/Athena.cmake
    • AtlasTest/CITest/test/WorkDirConfig.sh

    Adding @jchapman ,@pagessin ,@fwinkl as watchers

  • Frank Winklmeier added 2 commits

    added 2 commits

    • 86a20ddb - CITest: add CI test for WorkDir configuration
    • 324b6e13 - Temporarily disable most CI tests

    Compare with previous version

  • Author Maintainer

    Jenkins please retry a build

  • This merge request affects 1 package:

    • AtlasTest/CITest

    This merge request affects 6 files:

    • AtlasTest/CITest/AnalysisBase.cmake
    • AtlasTest/CITest/AthAnalysis.cmake
    • AtlasTest/CITest/AthGeneration.cmake
    • AtlasTest/CITest/AthSimulation.cmake
    • AtlasTest/CITest/Athena.cmake
    • AtlasTest/CITest/test/WorkDirConfig.sh

    Adding @jchapman ,@fwinkl ,@pagessin as watchers

  • CI Result FAILURE (hash 2d43edf8)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon
    externals
    cmake
    make
    tests

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    Athena: number of compilation errors 0, warnings 0
    AthSimulation: number of compilation errors 0, warnings 0
    AthGeneration: number of compilation errors 0, warnings 0
    AnalysisBase: number of compilation errors 0, warnings 0
    AthAnalysis: number of compilation errors 0, warnings 0
    DetCommon: number of compilation errors 0, warnings 0
    📝 For experts only: Jenkins output (remote access info)

  • CI Result FAILURE (hash 324b6e13)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon
    externals
    cmake
    make
    tests

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    Athena: number of compilation errors 0, warnings 0
    AthSimulation: number of compilation errors 0, warnings 0
    AthGeneration: number of compilation errors 0, warnings 0
    AnalysisBase: number of compilation errors 0, warnings 0
    AthAnalysis: number of compilation errors 0, warnings 0
    DetCommon: number of compilation errors 0, warnings 0
    📝 For experts only: Jenkins output (remote access info)

  • Frank Winklmeier added 2 commits

    added 2 commits

    • 8a094546 - CITest: add CI test for WorkDir configuration
    • bfbcf16a - Temporarily disable most CI tests

    Compare with previous version

  • Author Maintainer

    Jenkins please retry a build

  • This merge request affects 1 package:

    • AtlasTest/CITest

    This merge request affects 6 files:

    • AtlasTest/CITest/AnalysisBase.cmake
    • AtlasTest/CITest/AthAnalysis.cmake
    • AtlasTest/CITest/AthGeneration.cmake
    • AtlasTest/CITest/AthSimulation.cmake
    • AtlasTest/CITest/Athena.cmake
    • AtlasTest/CITest/test/WorkDirConfig.sh

    Adding @pagessin ,@fwinkl ,@jchapman as watchers

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading