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

[api] add the api integration/smoke tests to smoke-test suite #9209

Merged
merged 1 commit into from Sep 17, 2021

Conversation

xli
Copy link
Contributor

@xli xli commented Sep 17, 2021

Add one simple api integration test to ensure diem-node integration works as expected.

see #9193 for more context of new api service.

Related PRs

#9144

@bors-libra bors-libra added this to In Review in bors Sep 17, 2021
@xli xli requested a review from bmwill September 17, 2021 17:34
bmwill
bmwill previously approved these changes Sep 17, 2021
bmwill
bmwill previously approved these changes Sep 17, 2021
@xli xli changed the title [api] add the api integration-tests [api] add the api integration/smoke tests to smoke-test suite Sep 17, 2021
@xli
Copy link
Contributor Author

xli commented Sep 17, 2021

/land

@bors-libra bors-libra moved this from In Review to Queued in bors Sep 17, 2021
@bors-libra bors-libra moved this from Queued to Testing in bors Sep 17, 2021
@github-actions
Copy link

Cluster Test Result

Test runner setup time spent 286 secs
Compatibility test results for land_3ba55ff1 ==> land_6124648d (PR)
1. All instances running land_3ba55ff1, generating some traffic on network
2. First full node land_3ba55ff1 ==> land_6124648d, to validate new full node to old validator node traffic
3. First Validator node land_3ba55ff1 ==> land_6124648d, to validate storage compatibility
4. First batch validators (14) land_3ba55ff1 ==> land_6124648d, to test consensus and traffic between old full nodes and new validator node
5. First batch full nodes (14) land_3ba55ff1 ==> land_6124648d
6. Second batch validators (15) land_3ba55ff1 ==> land_6124648d, to upgrade rest of the validators
7. Second batch of full nodes (15) land_3ba55ff1 ==> land_6124648d, to finish the network upgrade, time spent 686 secs
all up : 1169 TPS, 3880 ms latency, 4400 ms p99 latency, no expired txns, time spent 250 secs
Logs: http://kibana.ct-0-k8s-testnet.aws.hlw3truzy4ls.com/app/kibana#/discover?_g=(time:(from:'2021-09-17T21:11:22Z',to:'2021-09-17T21:34:29Z'))
Dashboard: http://grafana.ct-0-k8s-testnet.aws.hlw3truzy4ls.com/d/performance/performance?from=1631913082000&to=1631914469000
Validator 1 logs: http://kibana.ct-0-k8s-testnet.aws.hlw3truzy4ls.com/app/kibana#/discover?_g=(time:(from:'2021-09-17T21:11:22Z',to:'2021-09-17T21:34:29Z'))&_a=(columns:!(log),query:(language:kuery,query:'kubernetes.pod_name:"val-1"'),sort:!(!('@timestamp',desc)))

Repro cmd:

./scripts/cti --tag land_3ba55ff1 --cluster-test-tag land_6124648d -E BATCH_SIZE=15 -E UPDATE_TO_TAG=land_6124648d --report report.json --suite land_blocking_compat

🎉 Land-blocking cluster test passed! 👌

@bors-libra bors-libra removed this from Testing in bors Sep 17, 2021
@bors-libra bors-libra merged commit 6124648 into diem:main Sep 17, 2021
@bors-libra bors-libra temporarily deployed to Sccache September 17, 2021 21:35 Inactive
@bors-libra bors-libra temporarily deployed to Docker September 17, 2021 21:35 Inactive
@bors-libra bors-libra temporarily deployed to Sccache September 17, 2021 21:35 Inactive
@xli xli linked an issue Sep 21, 2021 that may be closed by this pull request
50 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New Diem Dev API
3 participants