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.

Sign up for GitHub

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

Jump to bottom

fixbug: Multiple exclude-port is invalid #63

Merged
merged 1 commit into from
Oct 29, 2020
Merged

fixbug: Multiple exclude-port is invalid #63

merged 1 commit into from
Oct 29, 2020

Conversation

tiny-x
Copy link
Member

@tiny-x tiny-x commented Oct 23, 2020 •
edited
Loading

Signed-off-by: tiny.x 185120555@qq.com

Describe what this PR does / why we need it

#chaosblade-io/chaosblade#402

exec comannd

./blade create network loss --percent 100 --interface eth0 --exclude-port 22,9526,80 --debug

Nothing takes effect except port 22

@tiny-x
fixbug: Multiple exclude-port is invalid
1bea58c 
Signed-off-by: tiny.x <185120555@qq.com>
@tiny-x tiny-x added the type/bug Something isn't working label Oct 29, 2020
@tiny-x tiny-x added this to the v0.8.0 milestone Oct 29, 2020
@tiny-x tiny-x requested a review from xcaspar October 29, 2020 02:36
@xcaspar xcaspar merged commit 71edaef into master Oct 29, 2020
@tiny-x tiny-x deleted the feature-chaosblade-402 branch November 30, 2020 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xcaspar xcaspar xcaspar approved these changes

Assignees
No one assigned
Labels
type/bug Something isn't working
Projects
None yet
Milestone
v0.8.0
Development

Successfully merging this pull request may close these issues.
2 participants
@tiny-x @xcaspar