Skip to content

Commit

Permalink
GEODE-8763: run benchmark tests up to 5 times (apache#5864)
Browse files Browse the repository at this point in the history
* run benchmark tests up to 5 times, this time correctly reporting failure if still didn't pass on the 5th try
* increase benchmark timeouts
* run benchmarks on every commit even when multiple commits come in close together
  • Loading branch information
onichols-pivotal authored Dec 18, 2020
1 parent b3867a8 commit 941aa50
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 2 deletions.
2 changes: 2 additions & 0 deletions ci/pipelines/geode-build/jinja.template.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@
trigger: true
passed:
- Build
version: every
{% endmacro %}

{%- macro deep_merge(a, b): %}
Expand Down Expand Up @@ -476,6 +477,7 @@ jobs:
- get: geode-build-version
trigger: true
passed: *benchmark-inputs
version: every
- put: concourse-metadata-resource
- do:
- task: run_benchmarks{{ run_var.title }}
Expand Down
4 changes: 2 additions & 2 deletions ci/pipelines/shared/jinja.variables.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,12 +29,12 @@ benchmarks:
flag: '-PwithSsl -PtestJVM=/usr/lib/jvm/bellsoft-java11-amd64'
options: '--tests=*GetBenchmark --tests=*PutBenchmark'
max_in_flight: 1
timeout: 3h
timeout: 8h
- title: '_with_security_manager'
flag: '-PwithSecurityManager'
options: '--tests=Partitioned*'
max_in_flight: 2
timeout: 5h
timeout: 12h

build_test:
ARTIFACT_SLUG: build
Expand Down
4 changes: 4 additions & 0 deletions ci/scripts/run_benchmarks.sh
Original file line number Diff line number Diff line change
Expand Up @@ -85,10 +85,14 @@ do
./run_on_cluster.sh -t ${CLUSTER_TAG} -- rm /home/geode/locator10334view.dat;

if ./run_against_baseline.sh -t ${CLUSTER_TAG} -b ${GEODE_SHA} -r ${GEODE_REPO} -p ${BENCHMARKS_REPO} ${BASELINE_OPTION} -e ${BENCHMARKS_BRANCH} -o ${RESULTS_DIR} -m "'source':'geode-ci',${METADATA_BASELINE},'baseline_branch':'${BASELINE_BRANCH}','geode_branch':'${GEODE_SHA}'" --ci -- ${FLAGS} ${TEST_OPTIONS} ; then
STATUS=0
break
else
STATUS=1
fi

set -e
done

popd
exit $STATUS

0 comments on commit 941aa50

Please sign in to comment.