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

parallelize exec command in matched pods #78

Merged
merged 1 commit into from
May 16, 2021

Conversation

StephenLiuYa
Copy link
Contributor

Describe what this PR does / why we need it

The execution in matched pods with low efficiency

Does this pull request fix one issue?

#77

Describe how you did it

  1. parallelize exec commands in matched pods
  2. set max parallel count 64

Describe how to verify it

apiVersion: chaosblade.io/v1alpha1
kind: ChaosBlade
metadata:
  name: loss-pod-by-labels
spec:
  experiments:
  - scope: pod
    target: network
    action: loss
    desc: "node network loss"
    matchers:
    - name: labels
      value: ["app=nginx"]
    - name: namespace
      value: ["default"]
    - name: percent
      value: ["100"]
    - name: interface
      value: ["eth0"]
    - name: local-port
      value: ["80"]

image
image

Special notes for reviews

@StephenLiuYa StephenLiuYa force-pushed the feature/parallelize-exec branch 5 times, most recently from ab2d44c to 1038a7e Compare May 16, 2021 04:04
…ched pods

Signed-off-by: StephenLiuYa <liuyatmac@126.com>
@tiny-x
Copy link
Member

tiny-x commented May 16, 2021

@tiny-x
Copy link
Member

tiny-x commented May 16, 2021

@tiny-x tiny-x self-requested a review May 16, 2021 11:51
@tiny-x tiny-x added the type/enhancement New feature or request label May 16, 2021
@tiny-x tiny-x added this to the v1.1.0 milestone May 16, 2021
@tiny-x
Copy link
Member

tiny-x commented May 16, 2021

Thank you very much for your contribution.

@tiny-x tiny-x merged commit e337c24 into chaosblade-io:master May 16, 2021
@tiny-x tiny-x modified the milestones: v1.1.0, v1.2.0 May 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants