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 diagram for the main chaos page sample #2484

Merged

Conversation

peter-csala
Copy link
Contributor

@peter-csala peter-csala commented Feb 7, 2025

Pull Request

The issue or feature being addressed

This is the follow up for:

Details on the issue fix or feature implementation

Preview

sequenceDiagram
   actor C as Caller
   participant P as Pipeline
   participant F as Fault
   participant L as Latency
   participant O as Outcome
   participant B as Behavior
   participant D as DecoratedUserCallback

   C->>P: Calls ExecuteAsync

   P->>F: Calls ExecuteCore

   %% Fault Chaos
   activate F
   F-->>F: Determines Injection
   deactivate F

   alt 2% chance <br/>to inject: 🙈
       F->>P: Throws injected Fault
       P->>C: Propagates Exception
   else 98% chance <br/>to continue: 🐵
       F->>L: Calls ExecuteCore
   end

   %% Delay Chaos
   activate L
   L-->>L: Determines Injection
   deactivate L

   alt 50% chance <br/>to inject: 🙈
       L->>L: Injects delay
       L->>O: Calls ExecuteCore
   else 50% chance <br/>to continue: 🐵
       L->>O: Calls ExecuteCore
   end

   %% Outcome Chaos
   activate O 
   O-->>O: Determines Injection
   deactivate  O

   alt 10% chance <br/>to inject: 🙈
       O->>O: Injects outcome
       O->>L: Returns result
       L->>F: Returns result
       F->>P: Returns result
       P->>C: Returns result
   else 90% chance <br/>to continue: 🐵
       O->>B: Calls ExecuteCore
   end

   %% Behavior Chaos
   activate B
   B-->>B: Determines Injection
   deactivate B

   alt 1% chance <br/>to inject: 🙈
       B->>B: Injects behavior
       B->>D: Invokes
   else 99% chance <br/>to continue: 🐵
       B->>D: Invokes
   end

   D->>B: Returns result
   B->>O: Returns result
   O->>L: Returns result
   L->>F: Returns result
   F->>P: Returns result
   P->>C: Returns result
Loading

Confirm the following

  • I started this PR by branching from the head of the default branch
  • I have targeted the PR to merge into the default branch
  • I have included unit tests for the issue/feature
  • I have successfully run a local build

docs/chaos/index.md Outdated Show resolved Hide resolved
Copy link

codecov bot commented Feb 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.36%. Comparing base (aab7385) to head (72da9da).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2484   +/-   ##
=======================================
  Coverage   85.36%   85.36%           
=======================================
  Files         312      312           
  Lines        7466     7466           
  Branches     1121     1121           
=======================================
  Hits         6373     6373           
  Misses        908      908           
  Partials      185      185           
Flag Coverage Δ
linux 85.36% <ø> (ø)
macos 85.36% <ø> (ø)
windows 85.32% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@martincostello martincostello merged commit ff6ed5f into App-vNext:main Feb 8, 2025
24 checks passed
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.

2 participants