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

Add chaos testing for peer gateway into the fabric-test builds #354

Open
4 of 9 tasks
davidkel opened this issue Dec 17, 2021 · 1 comment
Open
4 of 9 tasks

Add chaos testing for peer gateway into the fabric-test builds #354

davidkel opened this issue Dec 17, 2021 · 1 comment

Comments

@davidkel
Copy link

davidkel commented Dec 17, 2021

Code base

  • add operator based network
  • add chaos chaincode from fabric-chaos testing into chaincodes/chaos/node
  • add chaos client (don't forget the lint dot file) from fabric-chaos-testing into tools/chaos/client/node
  • add chaos engine (don't forget the lint dot file and sample scenarios) from fabric-chaos-testing into tools/chaos/engine

Integration

  • build task to run chaos killing non gateway peers (this could fail occasionally) - see requirements later for this : PR AVAILABLE
  • build task to run complete chaos and ensure client can still recover - see requirements later for this
  • tune the tasks to the build environment they will run in when adding to the CI pipeline

Misc

  • task to do a smoke test of chaos engine when PR submitted
  • provide a readme to document the chaos environment and how to use standalone

For each build task you will need to

  • update Makefile to to build chaos chaincode, chaos client, chaos engine (on time only)
  • create a directory in regression
  • add in an appropriate suite_test.go + test.go files (+go.mod, go.sum)
  • create a chaos launcher script (based on coord.sh) to manage syncing the client and chaos engine
  • add a scenarios directory with the required scenarios for that build task
  • add a .env file which configures the client and is appropriate for that task (probably require tuning)
@denyeart
Copy link
Contributor

denyeart commented Oct 3, 2022

@davidkel Remind me, how far did we get with this? Wondering if we should close it now...

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

No branches or pull requests

2 participants