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

Support Triggering a Build that Runs ALL Tests in One Build #9003

Merged
merged 1 commit into from Apr 24, 2020

Conversation

kj4ezj
Copy link
Contributor

@kj4ezj kj4ezj commented Apr 24, 2020

Change Description

After merging pull request 8969, I created the eosio-full-test-suite pipeline to trigger a synchronous build on the eosio and eosio-build-unpinned pipelines simultaneously, each with RUN_ALL_TESTS set to true. Where the current pipelines only run some tests when certain conditions are met, this new pipeline Blockchain requested allows engineers to override those conditions to run ALL tests on suspect code with just one click.

This pull request adjust some logic that is only encountered during builds that were triggered by other builds.

Tested Working

Some builds show the pipeline step generation and then were cancelled so as not to waste agents, while other builds show pipeline step generation and passing tests.

See Also

Consensus Changes

  • Consensus Changes
    None.

API Changes

  • API Changes
    None.

Documentation Additions

  • Documentation Additions
    None.

Copy link
Contributor

@scottarnette scottarnette left a comment

Choose a reason for hiding this comment

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

squiggle

@kj4ezj kj4ezj marked this pull request as ready for review April 24, 2020 21:42
@kj4ezj kj4ezj merged commit b3ef396 into release/1.8.x Apr 24, 2020
@kj4ezj kj4ezj deleted the zach-all-tests-1.8 branch April 24, 2020 23:19
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.

None yet

2 participants