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

Refactor the network to support different kind of network implementation #53

Merged
merged 11 commits into from
Jul 2, 2018

Conversation

hwchiu
Copy link
Contributor

@hwchiu hwchiu commented Jun 28, 2018

No description provided.

@codecov-io
Copy link

codecov-io commented Jun 28, 2018

Codecov Report

Merging #53 into develop will increase coverage by 3.67%.
The diff coverage is 89.18%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #53      +/-   ##
===========================================
+ Coverage    76.89%   80.56%   +3.67%     
===========================================
  Files           17       20       +3     
  Lines          528      566      +38     
===========================================
+ Hits           406      456      +50     
+ Misses          92       83       -9     
+ Partials        30       27       -3
Impacted Files Coverage Δ
src/server/handler_volume.go 82.5% <ø> (-0.22%) ⬇️
src/networkcontroller/network_controller.go 82.85% <100%> (+7.24%) ⬆️
src/server/handler_network.go 83.78% <100%> (+12.59%) ⬆️
src/networkprovider/fake.go 100% <100%> (ø)
src/networkprovider/network.go 75% <75%> (ø)
src/networkprovider/ovs.go 81.25% <81.25%> (ø)

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 d5114a1...af23e23. Read the comment docs.

hwchiu added 10 commits June 29, 2018 03:09
@hwchiu hwchiu changed the title [WIP] Refactor the network to support different kind of network implementation Refactor the network to support different kind of network implementation Jun 29, 2018
Also refine the testing approach of listing network.
@hwchiu hwchiu merged commit 58af7ad into develop Jul 2, 2018
@hwchiu hwchiu deleted the hwchiu/VX-138 branch July 2, 2018 02:36
John-Lin pushed a commit that referenced this pull request Jul 25, 2018
Refactor the network to support different kind of network implementation

Former-commit-id: 7606cac98dd2df5a20e0f6758adb13f9c9a719e2 [formerly 58af7ad]
Former-commit-id: 408574af611fa3402694fb4cf469fd409bba7c4d
John-Lin pushed a commit that referenced this pull request Jul 25, 2018
Refactor the network to support different kind of network implementation

Former-commit-id: 7606cac98dd2df5a20e0f6758adb13f9c9a719e2 [formerly 7606cac98dd2df5a20e0f6758adb13f9c9a719e2 [formerly 58af7ad]]
Former-commit-id: 408574af611fa3402694fb4cf469fd409bba7c4d
Former-commit-id: 46a6e26
John-Lin pushed a commit that referenced this pull request Jul 25, 2018
Refactor the network to support different kind of network implementation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants