Skip to content

chore(windows): no --parallel on build#2280

Merged
lemeurherve merged 2 commits intojenkinsci:masterfrom
lemeurherve:no-parallel
Mar 23, 2026
Merged

chore(windows): no --parallel on build#2280
lemeurherve merged 2 commits intojenkinsci:masterfrom
lemeurherve:no-parallel

Conversation

@lemeurherve
Copy link
Copy Markdown
Member

@lemeurherve lemeurherve commented Mar 22, 2026

This change removes the parallel docker compose argument as it seems it wasn't properly configured since the beginning (no amount specified cf https://docs.docker.com/reference/cli/docker/compose/#configuring-parallelism), and did not result in any gain, but instead seems the cause of Windows builds stucks at the end of the build without any obvious error or cause. (cf multiple releases replayed)

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

Copy link
Copy Markdown
Contributor

@dduportal dduportal left a comment

Choose a reason for hiding this comment

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

https://docs.docker.com/reference/cli/docker/compose/#configuring-parallelism: this flag would have required an argument. But which value?

Since it has no performance gain but generate random issues, LGTM!

@lemeurherve lemeurherve marked this pull request as ready for review March 23, 2026 15:43
@lemeurherve lemeurherve requested a review from a team as a code owner March 23, 2026 15:43
@lemeurherve lemeurherve merged commit 326c337 into jenkinsci:master Mar 23, 2026
13 checks passed
@lemeurherve lemeurherve deleted the no-parallel branch March 23, 2026 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants