ci: Update CI job to newer OS (drop ubuntu 20.04, centos7/8)

[AJPfleger]

As already mentioned in the last meetings, we would like to update our testing matrix.

Please check if you rely on Ubuntu 20.04 or Centos 7&8. We plan to remove those from the testing in the next few weeks, if no-one comes forward with strong reasons to keep them.

Updateplan

Change Ubuntu 22.04 -> 24.04
Change Ubuntu 20.04 -> 22.04
Remove Centos 7
Remove Centos 8

blocked by:

• fix: bump DD4hep version for python 3.12 machines#104

[andiwand]

Not sure if we want to go with 24 already. The majority of users might still be using 22 for a while

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.44%. Comparing base (cf9d872) to head (281d3cb).
Report is 16 commits behind head on main.

❗ Current head 281d3cb differs from pull request most recent head 83a05e4

Please upload reports for the commit 83a05e4 to get more accurate results.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3227      +/-   ##
==========================================
+ Coverage   47.24%   47.44%   +0.19%     
==========================================
  Files         508      508              
  Lines       30041    29066     -975     
  Branches    14586    14098     -488     
==========================================
- Hits        14192    13789     -403     
+ Misses       5375     5183     -192     
+ Partials    10474    10094     -380     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

📊: Physics performance monitoring for 079140e

Full contents

physmon summary

• ✅ CKF truth_smeared
• ✅ IVF truth_smeared
• ✅ AMVF truth_smeared
• ✅ Track Summary CKF truth_smeared
• ✅ Seeding truth_estimated
• ✅ CKF truth_estimated
• ✅ IVF truth_estimated
• ✅ AMVF truth_estimated
• ✅ Track Summary CKF truth_estimated
• ✅ Seeding seeded
• ✅ CKF seeded
• ✅ IVF seeded
• ✅ AMVF seeded
• ✅ AMVF (+grid seeder) seeded
• ✅ Track Summary CKF seeded
• ✅ Seeding orthogonal
• ✅ CKF orthogonal
• ✅ IVF orthogonal
• ✅ AMVF orthogonal
• ✅ Track Summary CKF orthogonal
• ✅ Ambisolver seeded
• ✅ Ambisolver orthogonal
• ✅ Seeding ttbar
• ✅ CKF ttbar
• ✅ Ambisolver
• ✅ Track Summary CKF ttbar
• ✅ AMVF ttbar
• ✅ AMVF (+grid seeder) ttbar
• ✅ Truth tracking (GSF)
• ✅ Truth tracking
• ✅ Truth tracking (GX2F)
• ✅ Particles fatras
• ✅ Particles geant4

[stephenswat]

I'd actually go so far as to be adventurous and add a C++23 CI job based on the Ubuntu 24.04 image. 😄

[paulgessinger]

We should definitely keep a 22.04 job for now.

[AJPfleger]

Yes, dropping 22.04 was never in question. That's why we plan:

• Change Ubuntu 20.04 -> 22.04

[AJPfleger]

@gagnonlg, we were discussing to remove the linux-nodeps-build. Would this be ok for you or do you think we should still test for the boost-edge-case?

[gagnonlg]

@gagnonlg, we were discussing to remove the linux-nodeps-build. Would this be ok for you or do you think we should still test for the boost-edge-case?

Fine by me @AJPfleger

[paulgessinger]

I bumped histcmp to v0.6.6, where the python upper limit is removed. Let's see what that does.

Aside from this, the hash changes are likely expected, and physmon changes likely as well.

[paulgessinger]

I have possible fixes:

• Use 51 instead of v51 in the GitLab CI config
• Use clang-tidy-18 directly from the Ubuntu 24.04 repository instead of installing from llvm. I believe this also simplifies the toolchain configuration. Let's see what the full CI run says.

[paulgessinger]

I updated the hashes again from Github, but it seems unfortunately they still disagree between Github and the Gitlab CI runs. I believe the matched previously. Not sure what to make of this yet.

[paulgessinger]

After another round they seem to agree now? Maybe I was looking at the wrong pair of jobs.

[paulgessinger]

Ok now we're down to a few FPEs that were not previously observed it seems.

[benjaminhuth]

Let's get it in!

[AJPfleger]

Sonar cloud complains about the fpe-fix, but since we had the nested logic before, we could ignore it?

[paulgessinger]

@AJPfleger I would ignore that for right now, yeah.

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
91.2% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[acts-project-service]

✅ Athena integration test results

✅ All tests successful

status	job	report
🟢	report_pull_request
🟢	run_art_test: test_data18_13TeV_1000evt
🟢	run_art_test: test_ttbarPU40_reco
🟢	test_ActsDumpGeometryIdentifiers
🟢	test_ActsCheckObjectCountsWorkflow
🟢	test_ActsEFTrackFit
🟢	test_ActsPersistifySeeds
🟢	test_ActsBenchmarkWithSpot
🟢	test_ActsAnalogueClustering
🟢	test_ActsWorkflowHeavyIons
🟢	test_ActsWorkflowFastTracking
🟢	test_ActsWorkflowCached
🟢	test_ActsWorkflow
🟢	test_ActsValidateAmbiguityResolution
🟢	test_ActsValidateResolvedTracks
🟢	test_ActsValidateTracks
🟢	test_ActsValidateActsCoreSpacePoints
🟢	test_ActsValidateActsSpacePoints
🟢	test_ActsValidateSeeds
🟢	test_ActsValidateOrthogonalSeeds
🟢	test_ActsValidateClusters
🟢	test_ActsPersistifyEDM
🟢	test_ActsGSFRefitting
🟢	test_ActsKfRefitting
🟢	test_ActsExtrapolationAlgTest
🟢	test_ActsITkTest
🟢	run_workflow_tests_run4_mc
🟢	run_workflow_tests_run2_mc
🟢	run_workflow_tests_run2_data
🟢	run_workflow_tests_run3_mc
🟢	run_workflow_tests_run3_data
🟢	run_unit_tests