Skip to content

ci: Reduce CI time#114

Merged
ArekBalysNordic merged 1 commit into
nrfconnect:mainfrom
ArekBalysNordic:reduce_ci
Jun 18, 2026
Merged

ci: Reduce CI time#114
ArekBalysNordic merged 1 commit into
nrfconnect:mainfrom
ArekBalysNordic:reduce_ci

Conversation

@ArekBalysNordic

Copy link
Copy Markdown
Contributor

Speed up PR sample builds with quarantine and parallel twister. Add integration quarantine rules to cut redundant platform coverage and shared CI across four jobs with ccache and west/pip caching.

@ArekBalysNordic ArekBalysNordic requested a review from a team as a code owner June 8, 2026 13:23

@edmont edmont left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

A few suggestions.

Comment thread .github/workflows/on_pr.yml Outdated

@edmont edmont left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Updated comments.

Comment thread scripts/quarantine_integration.yaml Outdated
Comment thread scripts/quarantine_integration.yaml Outdated
@mariusz-nordicsemi

Copy link
Copy Markdown
Contributor

Recently the approach in NCS was changed. They stopped using the qurantine_integration list to specify which configurations shouldn't be built. Now they use integration_platforms to specify the targets the sample configuration shlould be built for.
Perhaps we should follow the same approach?

@ArekBalysNordic ArekBalysNordic force-pushed the reduce_ci branch 8 times, most recently from 5243256 to 8bf7ba4 Compare June 18, 2026 12:03
- Speed up PR sample builds removing integration platforms.

- paralleled twister jobs across the samples and variants.
- Shared CI across four jobs with ccache and west/pip caching.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
@ArekBalysNordic ArekBalysNordic merged commit 804716f into nrfconnect:main Jun 18, 2026
6 checks passed
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.

3 participants