Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

go/oasis-test-runner: Improve txsource E2E test #2709

Merged
merged 5 commits into from
Feb 24, 2020

Conversation

kostko
Copy link
Member

@kostko kostko commented Feb 23, 2020

See #2506.
Fixes #2708.

Includes a test that successfully reproduces #2708.

TODO

@kostko kostko force-pushed the kostko/feature/e2e-txsource-restarts branch 2 times, most recently from b4b9673 to 48ea14c Compare February 24, 2020 10:02
@kostko kostko marked this pull request as ready for review February 24, 2020 10:10
@codecov
Copy link

codecov bot commented Feb 24, 2020

Codecov Report

Merging #2709 into master will decrease coverage by 0.18%.
The diff coverage is 35.61%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2709      +/-   ##
==========================================
- Coverage   63.21%   63.03%   -0.19%     
==========================================
  Files         378      378              
  Lines       35515    35505      -10     
==========================================
- Hits        22451    22379      -72     
- Misses      10274    10327      +53     
- Partials     2790     2799       +9
Impacted Files Coverage Δ
go/consensus/api/api.go 50% <ø> (ø) ⬆️
go/consensus/genesis/genesis.go 50% <ø> (ø) ⬆️
go/oasis-test-runner/oasis/args.go 0% <0%> (ø) ⬆️
go/oasis-test-runner/oasis/client.go 0% <0%> (ø) ⬆️
go/oasis-test-runner/oasis/seed.go 0% <0%> (ø) ⬆️
go/oasis-test-runner/oasis/sentry.go 0% <0%> (ø) ⬆️
go/oasis-test-runner/oasis/fixture.go 0% <0%> (ø) ⬆️
go/oasis-test-runner/oasis/compute.go 0% <0%> (ø) ⬆️
go/oasis-test-runner/oasis/validator.go 0% <0%> (ø) ⬆️
go/oasis-test-runner/oasis/keymanager.go 0% <0%> (ø) ⬆️
... and 50 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 233ea01...c19fc76. Read the comment docs.

@kostko kostko force-pushed the kostko/feature/e2e-txsource-restarts branch 2 times, most recently from 42992f2 to eac7c29 Compare February 24, 2020 10:38
This commit adds the following general txsource scenario features:

- Support for multiple parallel workloads.
- Restart random nodes on specified interval.
- Ensure consensus liveness for the duration of the test.

It also adds an oversized txsource workload which submits oversized
transactions periodically.
@kostko kostko force-pushed the kostko/feature/e2e-txsource-restarts branch from eac7c29 to c19fc76 Compare February 24, 2020 11:04
@kostko kostko merged commit bf9e4d9 into master Feb 24, 2020
@kostko kostko deleted the kostko/feature/e2e-txsource-restarts branch February 24, 2020 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Non-determinism due to incorrect parameter caching
3 participants