-
Notifications
You must be signed in to change notification settings - Fork 850
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
Update run_experiment document #699
Conversation
Signed-off-by: cwen0 <[email protected]>
Signed-off-by: cwen0 <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #699 +/- ##
==========================================
- Coverage 55.78% 54.53% -1.25%
==========================================
Files 68 69 +1
Lines 4383 4652 +269
==========================================
+ Hits 2445 2537 +92
- Misses 1768 1926 +158
- Partials 170 189 +19
Continue to review full report at Codecov.
|
### Pause a chaos experiment | ||
|
||
```bash | ||
kubectl annotate networkchaos web-show-network-delay experiment.pingcap.com/pause=true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should update to experiment.chaos-mesh.org/pause
|
||
> **Note:** | ||
> | ||
> If the web-show is deployed on the server, you need use the host ip to access the application. | ||
|
||
## Step 2: Define the experiment configuration file | ||
|
||
The chaos experiment configuration is defined in a YAML file. You need to create your own experiment configuration file based on the available fields in the sample below: | ||
|
||
```yaml | ||
apiVersion: pingcap.com/v1alpha1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apiVersion: pingcap.com/v1alpha1 | |
apiVersion: chaos-mesh.org/v1alpha1 |
Signed-off-by: cwen0 <[email protected]>
Signed-off-by: cwen0 <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@WangXiangUSTC,Thanks for your review. However, LGTM is restricted to Reviewers or higher roles.See the corresponding SIG page for more information. Related SIGs: chaos-mesh(slack). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/merge |
/run-all-tests |
@cwen0 merge failed. |
/merge |
/run-all-tests |
@cwen0 merge failed. |
/merge |
Your auto merge job has been accepted, waiting for:
|
/run-all-tests |
* Update run_experiment document Signed-off-by: cwen0 <[email protected]> * make clean Signed-off-by: cwen0 <[email protected]> * address comments Signed-off-by: cwen0 <[email protected]> Co-authored-by: ti-srebot <[email protected]>
Signed-off-by: cwen0 [email protected]
What problem does this PR solve?
What is changed and how does it work?
Check List
Tests
Code changes
Side effects
Related changes
Does this PR introduce a user-facing change?