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

DiemIdDomainEvents #8434

Merged
merged 3 commits into from
May 27, 2021
Merged

DiemIdDomainEvents #8434

merged 3 commits into from
May 27, 2021

Conversation

sunmilee
Copy link

@sunmilee sunmilee commented May 25, 2021

Motivation

Adding TC account type to Accounts view and adding tests for reading DiemIdDomainEvents once domains have been added or removed.

Test Plan

json rpc integration_test.rs

Related PRs

#8363

json-rpc/types/src/views.rs Outdated Show resolved Hide resolved
@sunmilee sunmilee requested a review from xli May 26, 2021 23:30
@sunmilee
Copy link
Author

/land

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

Cluster Test Result

Test runner setup time spent 254 secs
Compatibility test results for land_3e0308c3 ==> land_c41e031e (PR)
1. All instances running land_3e0308c3, generating some traffic on network
2. First full node land_3e0308c3 ==> land_c41e031e, to validate new full node to old validator node traffic
3. First Validator node land_3e0308c3 ==> land_c41e031e, to validate storage compatibility
4. First batch validators (14) land_3e0308c3 ==> land_c41e031e, to test consensus and traffic between old full nodes and new validator node
5. First batch full nodes (14) land_3e0308c3 ==> land_c41e031e
6. Second batch validators (15) land_3e0308c3 ==> land_c41e031e, to upgrade rest of the validators
7. Second batch of full nodes (15) land_3e0308c3 ==> land_c41e031e, to finish the network upgrade, time spent 711 secs
all up : 920 TPS, 4930 ms latency, 5600 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-05-27T00:39:32Z',to:'2021-05-27T01:02:37Z'))
Dashboard: http://grafana.ct-0-k8s-testnet.aws.hlw3truzy4ls.com/d/performance/performance?from=1622075972000&to=1622077357000
Validator 1 logs: http://kibana.ct-0-k8s-testnet.aws.hlw3truzy4ls.com/app/kibana#/discover?_g=(time:(from:'2021-05-27T00:39:32Z',to:'2021-05-27T01:02:37Z'))&_a=(columns:!(log),query:(language:kuery,query:'kubernetes.pod_name:"val-1"'),sort:!(!('@timestamp',desc)))

Repro cmd:

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

🎉 Land-blocking cluster test passed! 👌

@bors-libra bors-libra removed this from Testing in bors May 27, 2021
@bors-libra bors-libra merged commit c41e031 into diem:main May 27, 2021
@bors-libra bors-libra temporarily deployed to Docker May 27, 2021 01:03 Inactive
@bors-libra bors-libra temporarily deployed to Sccache May 27, 2021 01:03 Inactive
zgfzgf pushed a commit to zgfzgf/diem that referenced this pull request Jun 8, 2021
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.

None yet

3 participants