Update CI images to latest version #34194
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
PR #34194: Size comparison from 8b97b28 to 0e08f30 Full report (3 builds for nxp, stm32)
|
|
PR #34194: Size comparison from 9e057a3 to f28e812 Full report (88 builds for bl602, bl702, bl702l, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
|
PR #34194: Size comparison from 07789d4 to a166eec Full report (88 builds for bl602, bl702, bl702l, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
a166eec to
ed57673
Compare
|
PR #34194: Size comparison from 07789d4 to ed57673 Full report (88 builds for bl602, bl702, bl702l, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
ed57673 to
8afe1ba
Compare
|
PR #34194: Size comparison from 07789d4 to 8afe1ba Full report (88 builds for bl602, bl702, bl702l, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
|
Done in different PRs, e.g.: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem
There are few images in CI which are outdated. Update of them bring up few additional issues:
runner.pystopped to report success during intentionally failing tests.WriteLinefunction which break compilation-ccompilator argument as unused. Fix were merged into clang-tidy master but it will take some time to be available inCIPDCHIP_ERROR_POSIX(errno);fromGetSocketmethod. It will report an error that function related to sockets are called with-1argument.Changes
I updated the outdated CI images to one based on Ubuntu 24.04. Unfortunately, this required some code changes and updating of python packages (old version of packages required by some embedded targets do not want to be instaled on python 3.12) and pigweed submodule simultaneously.
_runwere throwing an error, due to bug in python3.9.5. As documentations states:If the finally clause executes a break, continue or return statement, exceptions are not re-raised.but the code to work properly expect the_runmethod to throw an error. I've fixed it by removing unnecessary try block.-cfrom command line arguments passed to it, after update of clang-tidy in pigweed submodule this change can be removed.PATHpython from silabs SDK to use it during calling of internal toolsbrew- cipd do not have python3.12 avaliable right now