-
Notifications
You must be signed in to change notification settings - Fork 6.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[v3.5-branch] ci: Use zephyr-runner v2 #70133
Merged
stephanosio
merged 26 commits into
zephyrproject-rtos:v3.5-branch
from
stephanosio:v3.5-branch-ci-zephyr-runner-v2
Jun 10, 2024
Merged
[v3.5-branch] ci: Use zephyr-runner v2 #70133
stephanosio
merged 26 commits into
zephyrproject-rtos:v3.5-branch
from
stephanosio:v3.5-branch-ci-zephyr-runner-v2
Jun 10, 2024
Conversation
This file contains 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
stephanosio
force-pushed
the
v3.5-branch-ci-zephyr-runner-v2
branch
from
March 13, 2024 01:50
c20ef74
to
e66ddfe
Compare
This comment was marked as outdated.
This comment was marked as outdated.
stephanosio
requested review from
dcpleung,
andyross,
nashif and
galak
as code owners
June 10, 2024 17:37
zephyrbot
added
area: Kernel
area: Continuous Integration
area: Testsuite
Testsuite
labels
Jun 10, 2024
zephyrbot
requested review from
aaronemassey,
asemjonovs,
ceolin,
cfriedt,
jeremybettis,
peter-mitsis and
yperess
June 10, 2024 17:38
This commit updates the twister workflow to use the new zephyr-runner v2 CI runner deployment. It also updates the workflow to use the `ci-repo-cache` Docker image, which includes the Zephyr repository cache, because the node level repository cache is no longer available in the zephyr-runner v2. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit 7c19bc7)
This commit updates the twister workflow jobs that run on the zephyr-runner v2 to print the underlying cloud service information in the logs to help trace and debug potential runner issues. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit 9a9bebb)
This commit updates the twister workflow to store ccache data in the zephyr-runner v2 node cache. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit 1be3aac)
This commit updates the twister workflow to, when available, use Redis remote storage backend for the ccache compilation cache data. The Redis cache server is hosted in the Kubernetes cluster in which the zephyr-runner pods run -- the Redis remote storage backend will be ignored if the server is unavailable. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit 3823f1f)
This commit adds the compiler `--specs=*` flag to the ccache ignore option list because ccache is unable to resolve the toolchain-provided specs file path and will consider source files to be uncacheable if it is unable to read the specified specs file. Note that adding `--specs=*` to the ignore option list is not a problem because it is unlikely for the content of the toolchain libc spec file to change without the compiler executable itself changing. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit d3f9f39)
This commit updates the twister workflow such that ccache only uses remote Redis cache storage when available. The purpose of this to reduce the individual runner local disk IOPS requirement; thereby, reducing the overall load on the SAN. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit 527435d)
This commit sets the twister timeout multiplier to 2, which effectively increases the default test timeout from 60 to 120 seconds, because the new cost-effective Zephyr runners may take longer to execute tests and the default timeout is not sufficient for some tests to complete. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit de68ea7)
This commit increases the twister build job timeout from the default value of 6 hours to 24 hours because scheduled (weekly) build runs take longer than 6 hours to complete. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit 7df7e83)
Increase matrix size to 20 from 15 on push events. Signed-off-by: Anas Nashif <[email protected]> (cherry picked from commit 9970724)
This commit updates the doc-build workflow to use the new zephyr-runner v2 CI runner deployment. It also installs additional system packages that are not available by default in the zephyr-runner v2. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit 2819c35)
This commit updates the bsim-tests workflow to use the new zephyr-runner v2 CI runner deployment. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit 9838633)
This commit updates the codecov workflow to run on all forks under the zephyrproject-rtos organisation. The purpose of this is mainly to simplify the process of testing of this workflow under the zephyr-testing repository. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit c1bd5a6)
This commit updates the codecov workflow to use the new zephyr-runner v2 CI runner deployment. It also updates the workflow to use the `ci-repo-cache` Docker image, which includes the Zephyr repository cache, because the node level repository cache is no longer available in the zephyr-runner v2. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit 354e290)
This commit updates the codecov workflow to store ccache data in the zephyr-runner v2 node cache. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit 36b0b10)
This commit updates the codecov workflow to, when available, use Redis remote storage backend for the ccache compilation cache data. The Redis cache server is hosted in the Kubernetes cluster in which the zephyr-runner pods run -- the Redis remote storage backend will be ignored if the server is unavailable. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit b57f1b5)
This commit adds the compiler `--specs=*` flag to the ccache ignore option list because ccache is unable to resolve the toolchain-provided specs file path and will consider source files to be uncacheable if it is unable to read the specified specs file. Note that adding `--specs=*` to the ignore option list is not a problem because it is unlikely for the content of the toolchain libc spec file to change without the compiler executable itself changing. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit ab9f6b4)
This commit updates the codecov workflow such that ccache only uses remote Redis cache storage when available. The purpose of this to reduce the individual runner local disk IOPS requirement; thereby, reducing the overall load on the SAN. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit a636c52)
This commit sets the codecov workflow twister timeout multiplier to 2, which effectively increases the default test timeout from 60 to 120 seconds, because the new cost-effective Zephyr runners may take longer to execute tests and the default timeout is not sufficient for some tests to complete. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit 550bb4e)
This commit updates the clang workflow to use the new zephyr-runner v2 CI runner deployment. It also updates the workflow to use the `ci-repo-cache` Docker image, which includes the Zephyr repository cache, because the node level repository cache is no longer available in the zephyr-runner v2. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit 64ca699)
This commit updates the clang workflow to store ccache data in the zephyr-runner v2 node cache. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit cd83f07)
This commit updates the clang workflow to, when available, use Redis remote storage backend for the ccache compilation cache data. The Redis cache server is hosted in the Kubernetes cluster in which the zephyr-runner pods run -- the Redis remote storage backend will be ignored if the server is unavailable. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit 4a2884c)
This commit updates the clang workflow such that ccache only uses remote Redis cache storage when available. The purpose of this to reduce the individual runner local disk IOPS requirement; thereby, reducing the overall load on the SAN. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit 95e7eb3)
This commit updates the bsim-tests workflow to use the new zephyr-runner v2 CI runner deployment. It also updates the workflow to use the `ci-repo-cache` Docker image, which includes the Zephyr repository cache, because the node level repository cache is no longer available in the zephyr-runner v2. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit 9f9a6c5)
This commit updates the footprint workflow to use the new zephyr-runner v2 CI runner deployment. It also updates the workflow to use the `ci-repo-cache` Docker image, which includes the Zephyr repository cache, because the node level repository cache is no longer available in the zephyr-runner v2. Signed-off-by: Stephanos Ioannidis <[email protected]>
This commit increases the default value of `ZTEST_TEST_DELAY_MS` from 3000 to 5000 milliseconds because the current value of 3000ms may not be sufficient for the hosts with lower CPU clock frequency (e.g. new Zephyr CI runners with cost-effective processors). Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit 1bf7510)
This commit relaxes the idle event statistics test precision requirement for emulated QEMU targets because the cycle counts may be inaccurate when the host CPU is overloaded (e.g. when running tests with twister) and a high failure rate is observed for this test in the CI. Signed-off-by: Stephanos Ioannidis <[email protected]> (cherry picked from commit 2b2dd01)
stephanosio
force-pushed
the
v3.5-branch-ci-zephyr-runner-v2
branch
from
June 10, 2024 17:38
e66ddfe
to
677e109
Compare
stephanosio
commented
Jun 10, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merging since zephyr-runner v1 is no longer available.
stephanosio
merged commit Jun 10, 2024
48ba8b4
into
zephyrproject-rtos:v3.5-branch
8 of 14 checks passed
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.
This series migrates the CI workflows in the v3.5-branch to use the zephyr-runner v2.
Backport of #69806, #70014 and #70015
"Backport Issue Check" failure should be overridden since this is strictly a CI housekeeping change and backport rules do not apply.