Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Conversation

@radupopa2010
Copy link
Contributor

@radupopa2010 radupopa2010 commented Aug 17, 2021

Upgrade tests to simnet v8. This new version uses config for tests that live in this repo: https://github.com/paritytech/substrate/tree/master/simnet_tests.

At the moment tests run on a nightly schedule, on the master branch.
We have 2 kind of tests: "quick" and "long".
A "quick" test is a smoke test where basic check-expect tests run by checking values from metrics exposed by the app.
A "long" test is the load testing where we send 50K transactions into the network and check if all completed successfully.

We will do followup PR where we will run qucik test for all commits.

@radupopa2010 radupopa2010 marked this pull request as ready for review August 17, 2021 12:46
@radupopa2010 radupopa2010 requested a review from a team as a code owner August 17, 2021 12:46
@TriplEight
Copy link
Contributor

please write a PR description as well

@radupopa2010 radupopa2010 added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Aug 18, 2021
Co-authored-by: Denis Pisarev <[email protected]>
@TriplEight TriplEight self-requested a review August 18, 2021 11:10
Copy link
Contributor

@TriplEight TriplEight left a comment

Choose a reason for hiding this comment

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

good to go. Please merge as pipeline gets green and ping me to change the version variable in gitlab.
Also, please note to change that comment.

@radupopa2010 radupopa2010 requested a review from grbIzl August 18, 2021 13:06
.gitlab-ci.yml Outdated
- if: $CI_PIPELINE_SOURCE == "schedule"
- if: $CI_COMMIT_REF_NAME == "master"
- if: $CI_COMMIT_REF_NAME =~ /^v[0-9]+\.[0-9]+.*$/ # i.e. v1.0, v2.1rc1
variables:
Copy link
Contributor

Choose a reason for hiding this comment

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

what does it do?

@TriplEight TriplEight merged commit 287b5c5 into master Aug 18, 2021
@TriplEight TriplEight deleted the simnet-v8-move-tests branch August 18, 2021 15:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants