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] refactory Forge CLI interface - (132) #10682

Merged
merged 10 commits into from Jan 4, 2023

Conversation

ankitkacn
Copy link
Contributor

[forge] refactory Forge CLI interface #9061

Forge Cli is increasingly complicate since it supports more and more operations. Refactory CLI to different operation categories for better user experience.

Test Plan

For cluster Opertions:

  • cargo run -p forge-cli -- operator clean-up
  • cargo run -p forge-cli -- operator resize
  • cargo run -p forge-cli -- operator set-validator
  • for Testing:

cargo run -p forge-cli -- test local-swarm

  • ./scripts/fgi/run --local-swarm
  • ./script/fgi/run --pr xxx
  • ./scripts/fgi/run -T dev_czh_pull_9061 --suite land_blocking_compat

smoke-test: finish conversion to using local forge backend #9045

This PR finishes out the conversion of smoke-tests from using the old DiemSwarm to using the local forge backend as well as removing the now unused DiemSwarm infrastructure.

Test Plan

  • CI/CD tests are covered

[CI] update the min_tps to 1100 to prevent regressions #9063

  • CI/CD tests are covered

@bors-diem bors-diem added this to In Review in bors Jan 3, 2023
@ankitkacn
Copy link
Contributor Author

/canary

@bors-diem bors-diem moved this from In Review to Canary in bors Jan 3, 2023
bors-diem pushed a commit that referenced this pull request Jan 3, 2023
@bors-diem bors-diem moved this from Canary to In Review in bors Jan 3, 2023
@bors-diem
Copy link
Contributor

☀️ Canary successful

@ankitkacn
Copy link
Contributor Author

/land

@bors-diem bors-diem moved this from In Review to Queued in bors Jan 4, 2023
@bors-diem bors-diem moved this from Queued to Testing in bors Jan 4, 2023
@bors-diem bors-diem removed this from Testing in bors Jan 4, 2023
@bors-diem bors-diem closed this in c57d18d Jan 4, 2023
@bors-diem bors-diem merged commit c57d18d into diem:latest Jan 4, 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.

None yet

5 participants