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

[develop] Add Trigger for LRTs and Multiversion Tests Post PR #7901

Merged
merged 52 commits into from
Sep 13, 2019

Conversation

scottarnette
Copy link
Contributor

@scottarnette scottarnette commented Sep 12, 2019

Change Description

Added functionality to .cicd/generate-pipeline.sh:

  • New trigger for LRTs.

    • Builds triggered by merging a PR into master/develop/release branches now have a step that asynchronously triggers a new pipeline: eosio-lrt.
    • This ensures that eosio PRs are not blocked by LRTs, but still ensures they run after a PR is merged.
    • Functionality remains unchanged non-PR or PR builds. You may still specify LRTs to run via SKIP_LONG_RUNNING_TESTS=false when creating a new build.
    • This trigger will run LRTs only against pinned builds.
    • This pipeline reuses artifacts from the eosio job that triggered it. Resources are not wasted to rebuild the same branch/commit on triggered builds.
    • The eosio-lrt pipeline can be used on its own. SKIP_BUILD is only provided to generate-pipeline.sh if the pipeline was triggered from another pipeline.
  • New trigger for Multiversion tests.

    • Builds triggered by merging a PR into master/develop/release branches now have a step that asynchronously triggers the eosio-multiversion-test pipeline.
    • This change ensures Multiversion tests are executed more frequently.
    • This change does not impact the functionality of the Multiversion test.

See:
Build 17478 | This PR's build.
Build 30 | Manual build on eosio-lrt that would build EOS since it was not triggered.
Build 35 | Triggered build on eosio-lrt that reuses the build artifacts from Build 17423.

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

@scottarnette scottarnette changed the title [develop] Add Trigger for LRTs and Multiversion Tests on PR [develop] Add Trigger for LRTs and Multiversion Tests Post PR Sep 13, 2019
.cicd/generate-pipeline.sh Outdated Show resolved Hide resolved
.cicd/generate-pipeline.sh Outdated Show resolved Hide resolved
.cicd/generate-pipeline.sh Show resolved Hide resolved
.cicd/generate-pipeline.sh Show resolved Hide resolved
Copy link
Contributor

@jerstern jerstern left a comment

Choose a reason for hiding this comment

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

🌯

@scottarnette scottarnette merged commit 7d001e1 into develop Sep 13, 2019
scottarnette added a commit that referenced this pull request Sep 13, 2019
scottarnette added a commit that referenced this pull request Sep 13, 2019
@kj4ezj kj4ezj deleted the cicd-lrts-dev branch September 18, 2019 19:11
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