chore(ci): add main branch build validation and concurrency control#20226
chore(ci): add main branch build validation and concurrency control#20226gnodet 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
left a comment
squakez
left a comment
There was a problem hiding this comment.
Some minor points you may want to consider.
| push: | ||
| branches: | ||
| - main | ||
| - camel-4.4.x |
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'] |
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.
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.
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.
|
Closing and recreating as a fresh PR with the rebased branch on current main. |
3 participants
Summary
main-build.ymlworkflow for push builds onmainand release branches (camel-4.10.x,camel-4.14.x) with JDK 21cancel-in-progress: trueto PR workflows (pr-build-main,depsreview,pr-doc-validation,pr-id) so outdated CI runs are automatically cancelled when a PR is updatedChanges from original PR
mainpr-comment.ymlwas reorganized intopr-commenter.yml/pr-id.yml; concurrency added topr-id.ymlTest plan
🤖 Generated with Claude Code