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

[Forge] Delete test GCP workflow #8510

Merged
merged 1 commit into from
Jun 5, 2023
Merged

Conversation

sionescu
Copy link
Contributor

@sionescu sionescu commented Jun 3, 2023

Forge GCP is in production so this test flow is no longer useful.

@sionescu sionescu requested a review from a team as a code owner June 3, 2023 14:39
@sionescu sionescu enabled auto-merge (squash) June 3, 2023 14:40
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@perryjrandall
Copy link
Contributor

Thanks for cleaning up!

Forge GCP is in production so this test flow is no longer useful.
@sionescu sionescu force-pushed the stelian/delete-forge-gcp-workflow branch from 7d05067 to a54413c Compare June 5, 2023 01:53
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 5, 2023

✅ Forge suite compat success on testnet_2d8b1b57553d869190f61df1aaf7f31a8fc19a7b ==> a54413c8df5d8b624d7c460b38fd600676ae22a6

Compatibility test results for testnet_2d8b1b57553d869190f61df1aaf7f31a8fc19a7b ==> a54413c8df5d8b624d7c460b38fd600676ae22a6 (PR)
1. Check liveness of validators at old version: testnet_2d8b1b57553d869190f61df1aaf7f31a8fc19a7b
compatibility::simple-validator-upgrade::liveness-check : 9770 TPS, 3843 ms latency, 6500 ms p99 latency,no expired txns
2. Upgrading first Validator to new version: a54413c8df5d8b624d7c460b38fd600676ae22a6
compatibility::simple-validator-upgrade::single-validator-upgrade : 5722 TPS, 6901 ms latency, 8700 ms p99 latency,no expired txns
3. Upgrading rest of first batch to new version: a54413c8df5d8b624d7c460b38fd600676ae22a6
compatibility::simple-validator-upgrade::half-validator-upgrade : 5922 TPS, 6842 ms latency, 10000 ms p99 latency,no expired txns
4. upgrading second batch to new version: a54413c8df5d8b624d7c460b38fd600676ae22a6
compatibility::simple-validator-upgrade::rest-validator-upgrade : 8158 TPS, 4656 ms latency, 8000 ms p99 latency,no expired txns
5. check swarm health
Compatibility test for testnet_2d8b1b57553d869190f61df1aaf7f31a8fc19a7b ==> a54413c8df5d8b624d7c460b38fd600676ae22a6 passed
Test Ok

@github-actions
Copy link
Contributor

github-actions bot commented Jun 5, 2023

✅ Forge suite land_blocking success on a54413c8df5d8b624d7c460b38fd600676ae22a6

performance benchmark : 5819 TPS, 6826 ms latency, 27400 ms p99 latency,(!) expired 66 out of 2484920 txns
Test Ok

@github-actions
Copy link
Contributor

github-actions bot commented Jun 5, 2023

✅ Forge suite framework_upgrade success on aptos-node-v1.3.0_3fc3d42b6cfe27460004f9a0326451bcda840a60 ==> a54413c8df5d8b624d7c460b38fd600676ae22a6

Compatibility test results for aptos-node-v1.3.0_3fc3d42b6cfe27460004f9a0326451bcda840a60 ==> a54413c8df5d8b624d7c460b38fd600676ae22a6 (PR)
Upgrade the nodes to version: a54413c8df5d8b624d7c460b38fd600676ae22a6
framework_upgrade::framework-upgrade::full-framework-upgrade : 6107 TPS, 6527 ms latency, 19700 ms p99 latency,no expired txns
5. check swarm health
Compatibility test for aptos-node-v1.3.0_3fc3d42b6cfe27460004f9a0326451bcda840a60 ==> a54413c8df5d8b624d7c460b38fd600676ae22a6 passed
Test Ok

@sionescu sionescu merged commit b489087 into main Jun 5, 2023
@sionescu sionescu deleted the stelian/delete-forge-gcp-workflow branch June 5, 2023 02:42
gedigi pushed a commit that referenced this pull request Jun 6, 2023
banool pushed a commit that referenced this pull request Jul 7, 2023
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.

3 participants