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

pkg/proxy: make/simplify interface more extensible #9571

Merged
merged 1 commit into from
Apr 14, 2018
Merged

Conversation

gyuho
Copy link
Contributor

@gyuho gyuho commented Apr 14, 2018

Extend proxy for more advanced corrupt and packet drop testing.

For #5614 and #9563.

@gyuho gyuho changed the title pkg/proxy: make/simplify interface more extendible pkg/proxy: make/simplify interface more extensible Apr 14, 2018
Extend proxy for more advanced corrupt and packet drop testing.

Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@codecov-io
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (master@03cf9c4). Click here to learn what that means.
The diff coverage is 59.79%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #9571   +/-   ##
=========================================
  Coverage          ?   72.22%           
=========================================
  Files             ?      368           
  Lines             ?    31357           
  Branches          ?        0           
=========================================
  Hits              ?    22648           
  Misses            ?     7081           
  Partials          ?     1628
Impacted Files Coverage Δ
pkg/proxy/server.go 59.58% <59.79%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 03cf9c4...5db4df7. Read the comment docs.

@gyuho gyuho merged commit 9886e21 into etcd-io:master Apr 14, 2018
@gyuho gyuho deleted the net branch April 14, 2018 18:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants