Skip to content

Commit

Permalink
[devops] Try parallelize less during the API diff build. (#15471)
Browse files Browse the repository at this point in the history
For some reason the C# compilers crash a lot during the build in src/ when
building for the API diff (but not the normal build!). So test the theory that
we're overloading the bot in question (OOM maybe?) by slowing down a bit.

I have to say that if this works and the theory is proven, it's kind of sad
that after over a decade doing -j8 the bot situation has gotten worse...

Co-authored-by: Manuel de la Pena <[email protected]>
  • Loading branch information
rolfbjarne and mandel-macaque authored Jul 18, 2022
1 parent 4221018 commit 0021abb
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ steps:
gitHubToken: ${{ parameters.gitHubToken }}
xqaCertPass: ${{ parameters.xqaCertPass }}
enableDotnet: ${{ parameters.enableDotnet }}
makeParallelism: '4'

# detect changes
- bash: $(Build.SourcesDirectory)/xamarin-macios/tools/devops/automation/scripts/bash/compare.sh
Expand Down
12 changes: 10 additions & 2 deletions tools/devops/automation/templates/build/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,10 @@ parameters:
type: stepList
default: []

- name: makeParallelism
type: string
default: '8'

steps:

- template: ../common/checkout.yml
Expand Down Expand Up @@ -252,8 +256,12 @@ steps:
MAKE_FLAGS="V=1 -w"
fi
time make -C $(Build.SourcesDirectory)/xamarin-macios/ all -j8 $MAKE_FLAGS
time make -C $(Build.SourcesDirectory)/xamarin-macios/ install -j8 $MAKE_FLAGS
if test -z "$makeParallelism"; then
makeParallelism=8
fi
time make -C $(Build.SourcesDirectory)/xamarin-macios/ all -j$makeParallelism $MAKE_FLAGS
time make -C $(Build.SourcesDirectory)/xamarin-macios/ install -j$makeParallelism $MAKE_FLAGS
name: build
displayName: 'Build'
timeoutInMinutes: 300
Expand Down

5 comments on commit 0021abb

@vs-mobiletools-engineering-service2

This comment was marked as outdated.

@vs-mobiletools-engineering-service2

This comment was marked as outdated.

@vs-mobiletools-engineering-service2

This comment was marked as outdated.

@vs-mobiletools-engineering-service2

This comment was marked as outdated.

@vs-mobiletools-engineering-service2

This comment was marked as outdated.

Please sign in to comment.