Skip to content

chore(pipeline): split Build and Test stage#562

Merged
lemeurherve merged 1 commit intojenkinsci:masterfrom
lemeurherve:split-build-and-test-stages
Oct 24, 2025
Merged

chore(pipeline): split Build and Test stage#562
lemeurherve merged 1 commit intojenkinsci:masterfrom
lemeurherve:split-build-and-test-stages

Conversation

@lemeurherve
Copy link
Copy Markdown
Member

This PR splits the existing stage into "Build" and "Test".

Having separate stages allows a better understanding of the potential gains on each of those part when improving the building process.

Notes:

  • Generated docker compose files are archived even if the tests fail
  • As the publish target includes building the images, no need to call it first when only that one is running on trusted.ci.jenkins.io
  • As there is no "test" phase for multi-arch build, no need to change it

Similar to:

Testing done

  • CI

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests that demonstrate the feature works or the issue is fixed

PS: easier to review without whitespaces, see https://github.com/jenkinsci/docker-agent/pull/1071/files?diff=unified&w=1

@lemeurherve lemeurherve requested a review from a team as a code owner October 22, 2025 09:09
@lemeurherve lemeurherve enabled auto-merge (squash) October 22, 2025 09:10
@lemeurherve lemeurherve disabled auto-merge October 22, 2025 09:10
@lemeurherve lemeurherve enabled auto-merge (squash) October 22, 2025 09:10
@lemeurherve lemeurherve merged commit 493cac6 into jenkinsci:master Oct 24, 2025
11 checks passed
@lemeurherve lemeurherve deleted the split-build-and-test-stages branch October 24, 2025 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants