[PATCH v2] github_ci: remove redundant and low value build jobs to speed up CI - #2401
Open
JannePeltonen wants to merge 3 commits into
Open
JannePeltonen wants to merge 3 commits into
JannePeltonen wants to merge 3 commits into
Conversation
github-actions
Bot
changed the title
TuomasTaipale
approved these changes
Sep 17, 2026
Comment on lines
137
to
+138
| matrix: | ||
| conf: ['', '--enable-abi-compat'] | ||
| conf: [''] |
Collaborator
There was a problem hiding this comment.
The matrix block in some jobs now doesn't do anything leaving some bloat. I guess they could be left for easy additions in the future but on the other hand, the config listing doesn't seem to vary much historically.
Collaborator
Author
There was a problem hiding this comment.
I left the empty matrices there because I did not want to touch all the run lines that refer to them.
Remove build testing with -Os and -O1 as redundant. Other optimization levels, including -O0, -O2 (the default) and -O3, are still tested and a build breakage that would show up only with -Os or -O1 is unlikely. Signed-off-by: Janne Peltonen <janne.peltonen@nokia.com> Reviewed-by: Tuomas Taipale <tuomas.taipale@nokia.com>
ABI compatibility is not a widely used feature but gets tested rather heavily. Remove --enable-abi-compat from jobs that exercise different compiler versions, since problems in the ABI compatibility implementation are unlikely to show up only with certain compiler versions. Signed-off-by: Janne Peltonen <janne.peltonen@nokia.com> Reviewed-by: Tuomas Taipale <tuomas.taipale@nokia.com>
Exclude the combination of Clang and ABI compatibility from the cross compilation build jobs for arm64, armhf, i386 and ppc64el. Building the ABI compat variants with both GCC and Clang adds little value over testing ABI compat build with GCC only in these jobs. Signed-off-by: Janne Peltonen <janne.peltonen@nokia.com> Reviewed-by: Tuomas Taipale <tuomas.taipale@nokia.com>
JannePeltonen
force-pushed
the
ci-job-cleanup
branch
from
September 18, 2026 12:01
4327753 to
3b9837c
Compare
github-actions
Bot
changed the title
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Remove a bunch of --enable-abi-compat build jobs as well as -Os and -O1 build jobs as they do not add much coverage, if at all.