[CI] Overlaps exceptions and CI, fix for #2931 and sub-issues
- Re-enables the setup overlaps check in the CI to prevent new overlaps from being introduced due to usage of incompatible detectors geometries in setups
- Introduces agreed upon exceptions for the existing default setups
Draft status due to following points:
-
Missing a solution to run the overlap checks only when there is an update of the geometry repository hash to avoid waste of CI CPU power and time
(test cannot be performed in geo repo CI as CBM libraries are needed to create the setup from the individual detector geometries) -
Missing a geometry hash update to fix the problems with the APR21
setups, either CbmSoft/cbmroot_geometry!237 (closed) or CbmSoft/cbmroot_geometry!238 (merged) (#2936)
=> Hash update included inadvertently in !1482 (merged)
=> This MR will therefore not trigger anymore the new CI tests for most recent pipelines as it does not include anymore a geo Hash change
=> See pipeline #25533 for the last successful pipeline with these tests -
Missing either an exception for the RICH vs magnet overlap in sis100_electron_DEC21
or a hash update with the removal (deprecation) of theDEC21
setups (#2935)
Redmine Refs #2931, #2932, #2933, #2934, #2935, #2936
Of interest to
- @e.clerkin @v.friese @i.selyuzhenkov (Exceptions and setups modifications/removals)
- @f.uhlig (CI chain tuning)
Merge request reports
Activity
requested review from @e.clerkin
assigned to @f.uhlig
added 27 commits
-
81c89e79...d330f18b - 22 commits from branch
computing:master
- 00e09389 - [CI] Enable detection of overlaps in all geo-setups
- cf4af9a7 - Add exception for MVD internal overlap in setups check macro Refs #2934
- ea28d7cb - Add exception for beampipe internal overlap in setups check macro Refs #2933
- 93a49d81 - Add exception for RICH-beampipe overlap in setups check macro Refs #2933
- 12280ad7 - Add exception for RICH v23a internals overlaps in setups check macro Refs #2932
Toggle commit list-
81c89e79...d330f18b - 22 commits from branch
added 1 commit
- 7d6072c3 - [CI/CDASH] Run geo setups tests only for Nightly/Weekly or on run4 if geo hash change
added 1 commit
- 9d829ba5 - [CI/CDASH] Run geo setups tests only for Nightly/Weekly or on run4 if geo hash change
added 1 commit
- ae809666 - [CI/CDASH] Run geo setups tests only for Nightly/Weekly or on run4 if geo hash change
added 1 commit
- eb1c3329 - [CI/CDASH] Run geo setups tests only for Nightly/Weekly or on run4 if geo hash change
added 1 commit
- 40cb16eb - [CI/CDASH] Run geo setups tests only for Nightly/Weekly or on run4 if geo hash change
added 1 commit
- 94d23d19 - [CI/CDASH] Run geo setups tests only for Nightly/Weekly or on run4 if geo hash change
added 1 commit
- 995b1436 - [CI/CDASH] Run geo setups tests only for Nightly/Weekly or on run4 if geo hash change
added 1 commit
- cb2cd639 - [CI/CDASH] Run geo setups tests only for Nightly/Weekly or on run4 if geo hash change
added 1 commit
- a67af222 - [CI/CDASH] Run geo setups tests only for Nightly/Weekly or on run4 if geo hash change
added 1 commit
- 2c61e9c1 - [CI/CDASH] Run geo setups tests only for Nightly/Weekly or on run4 if geo hash change
added 1 commit
- 999f2cdb - [CI/CDASH] Run geo setups tests only for Nightly/Weekly or on run4 if geo hash change
added 1 commit
- 835e0f7f - [CI/CDASH] Run geo setups tests only for Nightly/Weekly or on run4 if geo hash change
added 8 commits
-
6f097990 - 1 commit from branch
computing:master
- eba1fcc1 - [CI] Enable detection of overlaps in all geo-setups
- f89aa131 - Add exception for MVD internal overlap in setups check macro Refs #2934
- e6d66915 - Add exception for beampipe internal overlap in setups check macro Refs #2933
- 0daac3f6 - Add exception for RICH-beampipe overlap in setups check macro Refs #2933
- 6217eeff - Add exception for RICH v23a internals overlaps in setups check macro Refs #2932
- b312d496 - [CI/CDASH] Run geo setups tests only for Nightly/Weekly or on run4 if geo hash change
- 3bb0376b - DO NOT MERGE! [Temp] change geo repo hash to fork to setup checks enabling
Toggle commit list-
6f097990 - 1 commit from branch
Sorry for the spam with the multiple force-pushes and failed pipelines today, I had to fight more than I had expected with the Gitlab CI config and runners.
Nonetheless I managed to achieve the target:
- The setup level geometry checks are not run by default on all CI runners (which avoids always running the material checks as before)
- There is a check done only for
run4
to see if the Install cmake file of the Geometry repository was modified relative the main master - Only if this is the case does
run4
check both the materials and overlaps consistencies ofall and any setup
provided in the repository.
This should allow to both keep the typical CI pipeline duration under control while catching potential troubles with the setups.
The only thing remaining to remove the
Draft
status is to replace theTEMP
commit with a valid geometry repository hash update, once the decisions on which changes to apply to theAPR21
andDEC21
setups are taken.