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

chore(dot/rpc): Load test submit extrinsic #2089

Merged
merged 3 commits into from
Dec 1, 2021
Merged

Conversation

arijitAD
Copy link
Contributor

@arijitAD arijitAD commented Nov 30, 2021

Changes

Tests

Start 3 nodes and fix the authority as Alice, Bob and Charlie in genesis
Genesis:

    "Babe": {
                "Authorities": [
                    [
                        "5GrwvaEF5zXb26Fz9rcQpDWS57CtERHpNehXCPcNoHGKutQY",
                        1
                    ],
                    [
                        "5FHneW46xGXgs5mUiveU4sbTyGBzmstUspZC92UhjJM694ty",
                        1
                    ],
                    [
                        "5FLSigC9HGRKVhB9FiEo4Y3koPsNmBmLJbpXg2mp1hXcS59Y",
                        1
                    ]
                ]
            },
            "Grandpa": {
                "Authorities": [
                    [
                        "5DFNv4Txc4b88qHqQ6GG4D646QcT4fN3jjS2G3r1PyZkfDut",
                        1
                    ],
                    [
                        "5FRbF5E5xpBxaCdjuVkCAtoLgvvoL6wJoE7kAVTUdZBwpgi7",
                        1
                    ],
                    [
                        "5DYo8CvjQcBQFdehVhansDiZCPebpgqvNC8PQPi6K9cL9giT",
                        1
                    ]
                ]
            },

Command:

Alice
./bin/gossamer --chain gssmr init --force
./bin/gossamer --chain gssmr --key alice --babe-lead --rpc-unsafe-external --ws-unsafe-external --ws

Bob
./bin/gossamer --chain gssmr init --config ~/go/src/github.com/ChainSafe/gossamer/chain/gssmr/config.toml --basepath /tmp/TestSync_ManyProducers/bob --genesis ~/go/src/github.com/ChainSafe/gossamer/chain/gssmr/genesis.json --force
./bin/gossamer --chain gssmr --port 9590 --basepath /tmp/TestSync_ManyProducers/bob --rpchost localhost --rpcport 8541 --rpc --log info --roles 4 --key bob --wsport 9541 --ws --rpc-unsafe-external --ws-unsafe-external

Charlie
./bin/gossamer --chain gssmr init --config ~/go/src/github.com/ChainSafe/gossamer/chain/gssmr/config.toml --basepath /tmp/TestSync_ManyProducers/charlie --genesis ~/go/src/github.com/ChainSafe/gossamer/chain/gssmr/genesis.json --force
./bin/gossamer --chain gssmr --port 9591 --basepath /tmp/TestSync_ManyProducers/charlie --rpchost localhost --rpcport 8542 --rpc --log info --roles 4 --key charlie --wsport 9542 --ws --rpc-unsafe-external --ws-unsafe-external

Issues

Primary Reviewer

@codecov
Copy link

codecov bot commented Nov 30, 2021

Codecov Report

Merging #2089 (2fb2210) into development (7bf40e1) will increase coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##           development    #2089      +/-   ##
===============================================
+ Coverage        60.62%   60.67%   +0.04%     
===============================================
  Files              202      204       +2     
  Lines            27282    27331      +49     
===============================================
+ Hits             16541    16583      +42     
- Misses            8842     8843       +1     
- Partials          1899     1905       +6     
Flag Coverage Δ
unit-tests 60.67% <ø> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
dot/network/inbound.go 82.14% <0.00%> (-3.58%) ⬇️
dot/network/service.go 57.74% <0.00%> (-0.23%) ⬇️
lib/runtime/wasmer/imports.go 48.71% <0.00%> (-0.08%) ⬇️
dot/peerset/handler.go 54.11% <0.00%> (ø)
dot/telemetry/telemetry.go 70.23% <0.00%> (ø)
dot/telemetry/afg_finalized_blocks_up_to.go 100.00% <0.00%> (ø)
dot/telemetry/afg_authority_set.go 100.00% <0.00%> (ø)
lib/babe/build.go 60.23% <0.00%> (+0.15%) ⬆️
lib/grandpa/grandpa.go 58.83% <0.00%> (+0.34%) ⬆️
dot/peerset/peerset.go 54.79% <0.00%> (+0.54%) ⬆️
... and 1 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 7bf40e1...2fb2210. Read the comment docs.

tests/stress/stress_test.go Outdated Show resolved Hide resolved
tests/stress/stress_test.go Outdated Show resolved Hide resolved
tests/stress/stress_test.go Show resolved Hide resolved
@@ -569,3 +569,75 @@ func Test_SubmitAndWatchExtrinsic(t *testing.T) {
`"method":"author_extrinsicUpdate","params":{"result":{"inBlock":"`)

}

func TestSync_SubmitExtrinsicLoad(t *testing.T) {
t.Skip()
Copy link
Member

Choose a reason for hiding this comment

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

is this skip intentional?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, We don't want to load test in CI.

@arijitAD arijitAD merged commit 8c58282 into development Dec 1, 2021
@arijitAD arijitAD deleted the stress-txn branch December 1, 2021 13:57
@github-actions
Copy link

github-actions bot commented Dec 3, 2021

🎉 This PR is included in version 0.6.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

timwu20 pushed a commit to timwu20/gossamer that referenced this pull request Dec 6, 2021
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.

Transaction Load-Testing for Gossamer Devnet
3 participants