Skip to content

[CI] Migrate from ubuntu-22.04 to ubuntu-24.04 - #2567

Merged
soumeh01 merged 7 commits into
mainfrom
update-ubuntu-runner
Aug 20, 2026
Merged

[CI] Migrate from ubuntu-22.04 to ubuntu-24.04#2567
soumeh01 merged 7 commits into
mainfrom
update-ubuntu-runner

Conversation

@soumeh01

@soumeh01 soumeh01 commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Fixes

Changes

  • Readiness for upcoming ubuntu-22.04 deprecation

Checklist

  • 🤖 This change is covered by unit tests (if applicable).
  • 🤹 Manual testing has been performed (if necessary).
  • 🛡️ Security impacts have been considered (if relevant).
  • 📖 Documentation updates are complete (if required).
  • 🧠 Third-party dependencies and TPIP updated (if required).

@soumeh01
soumeh01 marked this pull request as ready for review August 19, 2026 07:42
@github-actions

Copy link
Copy Markdown

Test Results

 4 files   -    39  10 suites   - 117   0s ⏱️ - 13m 42s
30 tests  - 1 081  30 ✅  - 1 064  0 💤  - 17  0 ❌ ±0 
60 runs   - 2 301  60 ✅  - 2 243  0 💤  - 58  0 ❌ ±0 

Results for commit d78c892. ± Comparison against base commit 3a8b92d.

This pull request removes 1081 tests.
AuxCmdTests ‑ MkdirCmdTest
AuxCmdTests ‑ RmdirCmdTest
AuxCmdTests ‑ TouchCmdTest
BuildSystemGeneratorTests ‑ GenAuditFile
BuildSystemGeneratorTests ‑ GenAuditFile_WithOut_Existing_Audit_File
BuildSystemGeneratorTests ‑ GenAuditFile_With_Existing_Audit_File
BuildSystemGeneratorTests ‑ GetString
BuildSystemGeneratorTests ‑ StrConv
BuildSystemGeneratorTests ‑ StrNorm
CBuildGCCTests ‑ Asm
…

@github-actions

Copy link
Copy Markdown

Test Results

 11 files   -    32   25 suites   - 102   3s ⏱️ - 13m 39s
174 tests  -   937  174 ✅  -   920  0 💤  - 17  0 ❌ ±0 
174 runs   - 2 187  174 ✅  - 2 129  0 💤  - 58  0 ❌ ±0 

Results for commit d78c892. ± Comparison against base commit 3a8b92d.

This pull request removes 937 tests.
AuxCmdTests ‑ MkdirCmdTest
AuxCmdTests ‑ RmdirCmdTest
AuxCmdTests ‑ TouchCmdTest
BuildSystemGeneratorTests ‑ GenAuditFile
BuildSystemGeneratorTests ‑ GenAuditFile_WithOut_Existing_Audit_File
BuildSystemGeneratorTests ‑ GenAuditFile_With_Existing_Audit_File
BuildSystemGeneratorTests ‑ GetString
BuildSystemGeneratorTests ‑ StrConv
BuildSystemGeneratorTests ‑ StrNorm
CBuildGCCTests ‑ Asm
…

@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown

Test Results

  5 files   39 suites   3m 44s ⏱️
185 tests 168 ✅ 17 💤 0 ❌
509 runs  458 ✅ 51 💤 0 ❌

Results for commit eb907b9.

♻️ This comment has been updated with latest results.

@soumeh01 soumeh01 left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@edriouk @brondani Here are some concerns raised by cpplinter tool
https://github.com/Open-CMSIS-Pack/devtools/actions/runs/32253146244?pr=2567
let me know what do you think about them

@brondani

Copy link
Copy Markdown
Collaborator

let me know what do you think about them

Apart those under tools/buildmgr which is deprecated, I believe we should address them all. It should be straightforward.

@codecov

codecov Bot commented Aug 20, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 72.09%. Comparing base (ff65562) to head (eb907b9).

Files with missing lines Patch % Lines
tools/packgen/src/PackGen.cpp 66.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2567      +/-   ##
==========================================
- Coverage   72.14%   72.09%   -0.06%     
==========================================
  Files         185      185              
  Lines       30126    30129       +3     
  Branches    17785    17753      -32     
==========================================
- Hits        21735    21722      -13     
- Misses       6090     6099       +9     
- Partials     2301     2308       +7     
Flag Coverage Δ
buildmgr-cov 79.71% <ø> (-0.17%) ⬇️
ctrace-cov 99.80% <ø> (-0.20%) ⬇️
packchk-cov 69.85% <ø> (+0.03%) ⬆️
packgen-cov 81.92% <66.66%> (-0.10%) ⬇️
projmgr-cov 88.04% <ø> (-0.05%) ⬇️
svdconv-cov 46.82% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
tools/packgen/src/PackGen.cpp 81.92% <66.66%> (-0.10%) ⬇️

... and 10 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@soumeh01

Copy link
Copy Markdown
Contributor Author

let me know what do you think about them

Apart those under tools/buildmgr which is deprecated, I believe we should address them all. It should be straightforward.

@brondani here is the fix for cppcheck findings

Fixed cpp check findings

@soumeh01
soumeh01 requested a review from brondani August 20, 2026 11:35
@soumeh01
soumeh01 merged commit 9fc7a07 into main Aug 20, 2026
119 of 121 checks passed
@soumeh01
soumeh01 deleted the update-ubuntu-runner branch August 20, 2026 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants