Skip to content

Set forkCount to speed up builds #451

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jglick merged 8 commits into from
Mar 11, 2021
Merged

Set forkCount to speed up builds #451

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
3945c73
Set forkCount to speed up builds?
jglick Mar 10, 2021
79e172d
Weird error in ASTParserUtilsTest.prettyPrintTransformed, hoping reus…
jglick Mar 10, 2021
9158dff
org.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTr…
jglick Mar 10, 2021
0adbc48
Revert "Bump pipeline-model-definition-plugin.version in /bom-latest"
jglick Mar 10, 2021
dfd2c2d
reuseForks=false did not in fact work around the failure, so not goin…
jglick Mar 10, 2021
f37cd79
forkCount=.5C makes things run a bit faster; what about 1C?
jglick Mar 10, 2021
e6eef68
1C seems too much, let us try .75C
jglick Mar 11, 2021
8566255
Going back to 1.8.4 but excluding failure pending https://github.com/…
jglick Mar 11, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions pct.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,7 @@ else
PCT_S_ARG=
fi

# TODO use -ntp if there is a PCT option to pass Maven options
MAVEN_PROPERTIES=org.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn:jth.jenkins-war.path=$(pwd)/megawar.war
MAVEN_PROPERTIES=jth.jenkins-war.path=$(pwd)/megawar.war:forkCount=.75C
Copy link
Member Author

@jglick jglick Mar 11, 2021

Choose a reason for hiding this comment

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

With current ACI agents, this seems to be effectively 3×.

if [ -v EXTRA_MAVEN_PROPERTIES ]
then
MAVEN_PROPERTIES="$MAVEN_PROPERTIES:$EXTRA_MAVEN_PROPERTIES"
Expand Down Expand Up @@ -114,4 +113,7 @@ rm -fv pct-work/workflow-basic-steps/target/surefire-reports/TEST-org.jenkinsci.
# TODO until dropping 2.235.x so can rely on https://github.com/jenkinsci/workflow-basic-steps-plugin/pull/120
rm -fv pct-work/workflow-basic-steps/target/surefire-reports/TEST-org.jenkinsci.plugins.workflow.steps.TimeoutStepTest.xml

# TODO https://github.com/jenkinsci/pipeline-model-definition-plugin/pull/417
Copy link
Member Author

@jglick jglick Mar 11, 2021

Choose a reason for hiding this comment

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

jenkinsci/pipeline-model-definition-plugin#417

rm -fv pct-work/pipeline-model-definition/pipeline-model-definition/target/surefire-reports/TEST-org.jenkinsci.plugins.pipeline.modeldefinition.parser.ASTParserUtilsTest.xml

# produces: **/target/surefire-reports/TEST-*.xml
5 changes: 2 additions & 3 deletions prep.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,8 @@ do
done

# TODO find a way to encode this in some POM so that it can be managed by Dependabot
SNAPSHOT_VERSION=0.5.1-SNAPSHOT # TODO incrementals broken till https://github.com/jenkinsci/plugin-compat-tester/pull/260 is merged
version=0.5.1-20201017.070523-3 # TODO https://github.com/jenkinsci/plugin-compat-tester/pull/261
pct=$HOME/.m2/repository/org/jenkins-ci/tests/plugins-compat-tester-cli/${SNAPSHOT_VERSION}/plugins-compat-tester-cli-${version}.jar
version=0.5.1-rc1068.959674bdeffd # TODO https://github.com/jenkinsci/plugin-compat-tester/pull/275
Copy link
Member Author

@jglick jglick Mar 10, 2021

Choose a reason for hiding this comment

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

jenkinsci/plugin-compat-tester#275

pct=$HOME/.m2/repository/org/jenkins-ci/tests/plugins-compat-tester-cli/${version}/plugins-compat-tester-cli-${version}.jar
[ -f $pct ] || $MVN dependency:get -Dartifact=org.jenkins-ci.tests:plugins-compat-tester-cli:${version}:jar -DremoteRepositories=https://repo.jenkins-ci.org/public/,https://repo.jenkins-ci.org/incrementals/ -Dtransitive=false
cp $pct target/pct.jar

Expand Down