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

fix: Fixing stress.py #2220

Merged
merged 1 commit into from Mar 4, 2020
Merged

fix: Fixing stress.py #2220

merged 1 commit into from Mar 4, 2020

Commits on Mar 3, 2020

  1. fix: Fixing stress.py

    The test was not updated after some config parameters were renamed.
    Also because of #2195, tx status of lost transactions times out, so
    adding a workaround into the test for now
    
    Disabling the version of stress that messes up with network, because the
    nightly runner is currently not configured to support the utility I use
    to stop network between processes
    
    Couple other changes:
    1. Made `node_restart` worker not restart the node if no blocks were
    produced in the meantime. It is needed because with only two nodes after
    a long restart doomslug can take a while to recover (it is equivalent to
    half the network shutting down and restarting after some delay).
    Block production worker will fail the test if the block production
    actually stalls
    2. For the same reason increased the tolerated delays for block
    production
    3. Limited how many txs are sent per iteration of tx worker, since due
    to #2195 it takes one second to query one transaction, and if the test
    finished in the middle of querying, the allowed one minute for workers
    to stop is not sufficient for the tx worker.
    4. Also generally increasing the allowance from 1m to 2m at the end,
    since the tx worker at the end of the test might take some time before
    it even starts querying the transaction outcomes
    SkidanovAlex committed Mar 3, 2020
    Copy the full SHA
    e7b5492 View commit details
    Browse the repository at this point in the history