Skip to content
This repository has been archived by the owner on Apr 17, 2019. It is now read-only.

Add performance test based on locust.io #2087

Merged

Conversation

lebdron
Copy link
Contributor

@lebdron lebdron commented Feb 9, 2019

Description of the Change

Performance test that sends a transaction using asynchronous API and waits until the final status - committed or rejected - is received from the status stream.

How to run

README.md

Specify locustfile in LOCUSTFILE_PATH in Compose file

@lebdron lebdron changed the title Feature/locust performance test Add performance test based on locust.io Feb 9, 2019
grpc_gevent.init_gevent()

import grpc
from irohalib import Iroha, IrohaGrpc
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's move to Iroha from pypi.

pip install iroha

and

from iroha import Iroha, IrohaGrpc, IrohaCrypto

The second preparation step from test/load/README.md will be much simpler.

Copy link
Contributor

@MBoldyrev MBoldyrev left a comment

Choose a reason for hiding this comment

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

Launched - seems working.

@task
def send_tx(self):
iroha = Iroha('admin@test')
admin_private_key = 'b60e30b7af0f0d233473ca8afe10084c321abc254a911b68c7bb49e924c579bb'
Copy link
Contributor

@MBoldyrev MBoldyrev Feb 9, 2019

Choose a reason for hiding this comment

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

Why the key does not match the one in genesis block example?
Also, if this script is supposed to be run from within the repo - could we read the private key matching the public one from genesis block that irohad uses? Or if you consider it too complicated - please at least put this to the beginning of the file to ease adjustment.

Igor Egorov added 2 commits February 9, 2019 17:49
@neewy
Copy link
Contributor

neewy commented Feb 13, 2019

Encountered the following problem:

ERROR: yaml.constructor.ConstructorError: while constructing a mapping
  in "./docker-compose.yml", line 18, column 7
expected a mapping or list of mappings for merging, but found scalar
  in "./docker-compose.yml", line 6, column 16

@neewy
Copy link
Contributor

neewy commented Feb 14, 2019

The problem still persists on macOS and Ubuntu 18.10 x64

@neewy
Copy link
Contributor

neewy commented Feb 14, 2019

Ubuntu
docker-compose version 1.22.0, build unknown
Docker version 18.09.2, build 6247962

@neewy
Copy link
Contributor

neewy commented Feb 14, 2019

macOS
Docker version 18.09.1, build 4c52b90
docker-compose version 1.23.2, build 1110ad01

Signed-off-by: Andrei Lebedev <[email protected]>
@lebdron lebdron force-pushed the feature/locust-performance-test branch from 5ca5c68 to 1b72b52 Compare February 18, 2019 12:59
Copy link
Contributor

@neewy neewy left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Andrei Lebedev <[email protected]>
@lebdron lebdron merged commit 4c65417 into hyperledger-iroha:develop Feb 18, 2019
@lebdron lebdron deleted the feature/locust-performance-test branch February 18, 2019 14:24
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.

4 participants