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

chore(chaos): Update 'experiment' to 'fault' in fault docs and update tutorials #150

Merged
merged 87 commits into from
Dec 7, 2022

Conversation

neelanjan00
Copy link
Member

Harness Developer Pull Request

  • Update 'experiment' to 'fault' in all fault docs
  • Update chaos tutorials to update demo app repo to githuub.com/chaosnative/harness-chaos-demo

What Type of PR is This?

  • Issue
  • Feature
  • Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

  • Issue(s)

House Keeping

Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

  • Tested Locally
  • Optional Screen Shoot.

uditgaurav and others added 30 commits November 8, 2022 15:45
Signed-off-by: Akash Shrivastava <[email protected]>
Signed-off-by: Adarsh kumar <[email protected]>
Signed-off-by: Akash Shrivastava <[email protected]>
Signed-off-by: Akash Shrivastava <[email protected]>
Signed-off-by: Akash Shrivastava <[email protected]>
docs(kubelet-density): adding kubelet density experiment doc
docs(ec2): adding ec2 network,dns  and process-kill faults docs
Signed-off-by: Adarsh kumar <[email protected]>
Signed-off-by: Adarsh kumar <[email protected]>
Signed-off-by: Adarsh kumar <[email protected]>
Chore(Docs): AWS EC2 Stress Chaos
Signed-off-by: Neelanjan Manna <[email protected]>
Signed-off-by: Neelanjan Manna <[email protected]>
Signed-off-by: Neelanjan Manna <[email protected]>
Signed-off-by: Neelanjan Manna <[email protected]>
Signed-off-by: Neelanjan Manna <[email protected]>
@neelanjan00 neelanjan00 added the Content - enhancement Content flag: The PR includes new feature documentation label Nov 28, 2022
@neelanjan00 neelanjan00 self-assigned this Nov 28, 2022
Signed-off-by: Neelanjan Manna <[email protected]>
@ravilach
Copy link
Collaborator

Looks like a few broken links. Can always run yarn build locally to find also.

- On source page path = /docs/chaos-engineering/chaos-faults/aws/ecs-agent-stop:

   -> linking to ../common-tunables-for-all-experiments (resolved as: /docs/chaos-engineering/chaos-faults/common-tunables-for-all-experiments)

   -> linking to ./aws-experiments-tunables (resolved as: /docs/chaos-engineering/chaos-faults/aws/aws-experiments-tunables)


- On source page path = /docs/chaos-engineering/chaos-faults/aws/ecs-container-cpu-hog:

   -> linking to ../common-tunables-for-all-experiments (resolved as: /docs/chaos-engineering/chaos-faults/common-tunables-for-all-experiments)

   -> linking to ./aws-experiments-tunables (resolved as: /docs/chaos-engineering/chaos-faults/aws/aws-experiments-tunables)


- On source page path = /docs/chaos-engineering/chaos-faults/aws/ecs-container-io-stress:

   -> linking to ../common-tunables-for-all-experiments (resolved as: /docs/chaos-engineering/chaos-faults/common-tunables-for-all-experiments)

   -> linking to ./aws-experiments-tunables (resolved as: /docs/chaos-engineering/chaos-faults/aws/aws-experiments-tunables)


- On source page path = /docs/chaos-engineering/chaos-faults/aws/ecs-container-memory-hog:

   -> linking to ../common-tunables-for-all-experiments (resolved as: /docs/chaos-engineering/chaos-faults/common-tunables-for-all-experiments)

   -> linking to ./aws-experiments-tunables (resolved as: /docs/chaos-engineering/chaos-faults/aws/aws-experiments-tunables)


- On source page path = /docs/chaos-engineering/chaos-faults/aws/ecs-container-network-latency:

   -> linking to ../common-tunables-for-all-experiments (resolved as: /docs/chaos-engineering/chaos-faults/common-tunables-for-all-experiments)

   -> linking to ./aws-experiments-tunables (resolved as: /docs/chaos-engineering/chaos-faults/aws/aws-experiments-tunables)


- On source page path = /docs/chaos-engineering/chaos-faults/aws/ecs-container-network-loss:

   -> linking to ../common-tunables-for-all-experiments (resolved as: /docs/chaos-engineering/chaos-faults/common-tunables-for-all-experiments)

   -> linking to ./aws-experiments-tunables (resolved as: /docs/chaos-engineering/chaos-faults/aws/aws-experiments-tunables)


- On source page path = /docs/chaos-engineering/chaos-faults/aws/ecs-instance-stop:

   -> linking to ../common-tunables-for-all-experiments (resolved as: /docs/chaos-engineering/chaos-faults/common-tunables-for-all-experiments)

   -> linking to ./aws-experiments-tunables (resolved as: /docs/chaos-engineering/chaos-faults/aws/aws-experiments-tunables)


- On source page path = /docs/chaos-engineering/chaos-faults/azure/azure-web-app-access-restrict:

   -> linking to ../common-tunables-for-all-experiments (resolved as: /docs/chaos-engineering/chaos-faults/common-tunables-for-all-experiments)


- On source page path = /docs/chaos-engineering/chaos-faults/azure/azure-web-app-stop:

   -> linking to ../common-tunables-for-all-experiments (resolved as: /docs/chaos-engineering/chaos-faults/common-tunables-for-all-experiments)

Signed-off-by: Neelanjan Manna <[email protected]>
@neelanjan00
Copy link
Member Author

neelanjan00 commented Dec 1, 2022

Thanks for pointing out @ravilach! These faults were added later and hence had not been updated for the changed file names. FIxed.

@bot-gitexp-user
Copy link

Preview environment: https://ci.pr.harness.io/pr-150

Signed-off-by: Neelanjan Manna <[email protected]>
@bot-gitexp-user
Copy link

Preview environment: https://ci.pr.harness.io/pr-150

Signed-off-by: Neelanjan Manna <[email protected]>
@SmritiSatya SmritiSatya merged commit 9e1787f into main Dec 7, 2022
@ravilach ravilach deleted the fix-all-faults branch December 28, 2022 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content - enhancement Content flag: The PR includes new feature documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants