Skip to content

CAMEL-24318 - Use Maven parallelization for deploy on Jenkins - #25264

Merged
apupier merged 1 commit into
apache:mainfrom
apupier:24318-useMavenParallelForDeploy
Jul 31, 2026
Merged

CAMEL-24318 - Use Maven parallelization for deploy on Jenkins#25264
apupier merged 1 commit into
apache:mainfrom
apupier:24318-useMavenParallelForDeploy

Conversation

@apupier

@apupier apupier commented Jul 30, 2026

Copy link
Copy Markdown
Contributor
  • locally with local repo (take care to deactivate local build cache):
    • 8'32" ( ./mvnw -T1C -U -B -e -fae -V -Dnoassembly -Pdeploy,apache-snapshots -Dquickly clean deploy
      -DaltDeploymentRepository=your_repo_id::default::file:/tmp/alt-repo-2 )
    • 18'33" (./mvnw -U -B -e -fae -V -Dnoassembly
      -Pdeploy,apache-snapshots -Dquickly clean deploy
      -DaltDeploymentRepository=your_repo_id::default::file:/tmp/alt-repo-2)
  • on jenkins for Camel core:

Description

Target

  • I checked that the commit is targeting the correct branch (Camel 4 uses the main branch)

Tracking

  • If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

  • I checked that each commit in the pull request has a meaningful subject line and body.
  • I have run mvn clean install -DskipTests locally from root folder and I have committed all auto-generated changes.

AI-assisted contributions

  • If this PR includes AI-generated code, commits have proper co-authorship attribution (e.g., Co-authored-by trailers) and the PR description identifies the AI tool used.

@github-actions

Copy link
Copy Markdown
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟
🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run
  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot although they are normally detected and executed by CI.
  • You can label PRs using skip-tests and test-dependents to fine-tune the checks executed by this PR.
  • Build and test logs are available in the summary page. Only Apache Camel committers have access to the summary.

⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@github-actions github-actions Bot added the ci label Jul 30, 2026
* locally with local repo (take care to deactivate local build cache):
  * 8'32" ( ./mvnw -T1C -U -B -e -fae -V -Dnoassembly
-Pdeploy,apache-snapshots -Dquickly clean deploy
-DaltDeploymentRepository=your_repo_id::default::file:/tmp/alt-repo-2 )
  * 18'33" (./mvnw -U -B -e -fae -V -Dnoassembly
-Pdeploy,apache-snapshots -Dquickly clean deploy
-DaltDeploymentRepository=your_repo_id::default::file:/tmp/alt-repo-2)
* on jenkins for Camel core:
  * 25min instead of usually 3 hours, previous record was 2h35 , see
https://ci-builds.apache.org/job/Camel/job/Camel%20Daily%20Snapshot%20Deploy/job/main/1494/

Signed-off-by: Aurélien Pupier <apupier@ibm.com>
@apupier
apupier force-pushed the 24318-useMavenParallelForDeploy branch from 1a6b1bd to 634185e Compare July 30, 2026 21:16
@apupier
apupier marked this pull request as ready for review July 30, 2026 21:16

@gnodet gnodet left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Claude Code on behalf of Guillaume Nodet

Adding -T1C to the snapshot deploy pipeline is a well-tested, low-risk optimization. The deploy pipeline uses -Dquickly (skipping tests/formatting/enforcement), and all plugins involved (compiler, deploy, source, javadoc) are thread-safe. The PR provides good performance evidence — local measurements (8m32s vs 18m33s) and a Jenkins build link (build #1494 at 25min vs the typical ~3h baseline).

Key observations:

  • The main Jenkinsfile (full build with tests) is correctly left unchanged — full builds with test execution carry higher thread-safety risk
  • The existing -fae (fail-at-end) flag is the correct companion for -T1C
  • Maven reactor properly respects inter-module dependency ordering even with -T1C
  • Memory consumption may increase with parallel builds since multiple modules compile and generate javadoc simultaneously — worth monitoring after merge, though the Jenkins test (build #1494) demonstrates sufficient resources

@apupier
apupier merged commit 134d11d into apache:main Jul 31, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants