chore(ci): add main branch build validation and concurrency control#20226
Open
gnodet wants to merge 3 commits intoapache:mainfrom
Open
chore(ci): add main branch build validation and concurrency control#20226gnodet wants to merge 3 commits intoapache:mainfrom
gnodet wants to merge 3 commits intoapache:mainfrom
Conversation
Add a new workflow to validate commits pushed to main and LTS branches: - Runs full build and tests on push to main, camel-4.4.x, camel-4.8.x, camel-4.10.x, and camel-4.14.x branches - Tests with both Java 17 and Java 21 - Includes build artifact archival for debugging Add concurrency control to PR workflows to prevent resource waste: - pr-build-main.yml: Cancel in-progress runs when PR is updated - pr-comment.yml: Cancel in-progress runs for same issue - pr-doc-validation.yml: Cancel in-progress runs when PR is updated - depsreview.yaml: Cancel in-progress runs when PR is updated This ensures the main branch is always validated after merges and reduces CI queue times by cancelling obsolete workflow runs.
squakez
approved these changes
Dec 4, 2025
Contributor
squakez
left a comment
squakez
left a comment
There was a problem hiding this comment.
Some minor points you may want to consider.
.github/workflows/main-build.yml
Outdated
| push: | ||
| branches: | ||
| - main | ||
| - camel-4.4.x |
Contributor
There was a problem hiding this comment.
I'd keep only last 2 LTS instead. Altough this is not really affecting as we don't merge anything to older branches, it could be perceived as an argument for users which expects longer timeline maintenance for those older branches.
| runs-on: ubuntu-latest | ||
| strategy: | ||
| matrix: | ||
| java: ['17', '21'] |
Contributor
There was a problem hiding this comment.
For the merge I'd reduce the quantity of resources by only checking against the latest JVM supported instead.
Contributor
Author
There was a problem hiding this comment.
I think it would make more sense to keep the main build against all supported JVM and reduce the build in the PR to run only against the latest JVM rather than the opposite.
orpiske
approved these changes
Dec 16, 2025
Removed camel-4.4.x and camel-4.8.x branches from the main branch build workflow to focus on only the two most recent LTS releases.
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
3 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.
Add a new workflow to validate commits pushed to main and LTS branches:
main,camel-4.4.x,camel-4.8.x,camel-4.10.x, andcamel-4.14.xbranchesAdd concurrency control to PR workflows to prevent resource waste:
This ensures the main branch is always validated after merges and reduces CI queue times by cancelling obsolete workflow runs.