Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Created test scripts with xUnit, auto-scaling parallelism, and empty test detection; and added them to pipeline #6963

Merged
merged 8 commits into from
Mar 20, 2019

Conversation

kj4ezj
Copy link
Contributor

@kj4ezj kj4ezj commented Mar 18, 2019

Change Description

I moved the test steps for serial, parallel, and long-running tests into bash scripts. This enables us to use features not supported by raw Buildkite commands, which run in shell.

Tests now:

  • Fail if there are zero tests to run
  • Automatically detect the number of CPU cores and scale across all of them for parallelizable tests (previously we assumed 8 cores)
  • Produces an xUnit-formatted XML file containing test results
  • Uploads the XML file as a test artifact

Also increased LRT time limit from 60 to 90 minutes, tested in build 2218.

Zane asked me to delete the following files, which have already been migrated to our central Buildkite repo:
/.buildkite/coverage.yml
/.buildkite/debug.yml
/.buildkite/docker.yml
/.buildkite/sanitizers.yml

Consensus Changes

  • Consensus Changes

None.

API Changes

  • API Changes

None.

Documentation Additions

  • Documentation Additions

None.

Copy link
Contributor

@zreyn zreyn left a comment

Choose a reason for hiding this comment

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

Only thing is maybe remove the && \ line continuations

@kj4ezj kj4ezj merged commit 756f3c7 into develop Mar 20, 2019
@kj4ezj kj4ezj deleted the zcb-xunit branch March 20, 2019 15:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants