search

John-Lin / ovs-cni

Apache License 2.0
25 stars 11 forks source link

Add new IPAM, support cluster-wide IP management. #15

Closed hwchiu closed 6 years ago

hwchiu commented 6 years ago

In this PR, we modify the centralIP architecture to support different backend implementations. For the previous one, it's node type, all the node will get different subnet. The new one, it's cluster type and it's feature request from the ONF, They want a solution to dispatch the one subnet to whole cluster node.

  1. new architecturte
  2. modify README
  3. modify the testing env, need to set TEST_ETCD env to test the centralip related testing.
codecov-io commented 6 years ago

Codecov Report

Merging #15 into master will increase coverage by 5.33%. The diff coverage is 81.19%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
+ Coverage   45.45%   50.79%   +5.33%     
==========================================
  Files           5        7       +2     
  Lines         418      504      +86     
==========================================
+ Hits          190      256      +66     
- Misses        213      225      +12     
- Partials       15       23       +8
Impacted Files Coverage Δ
ipam/centralip/backend/utils/utils.go 100% <100%> (ø)
ipam/centralip/backend/node/node.go 77.95% <77.95%> (ø)
ipam/centralip/backend/central-ip.go 85.71% <84.61%> (+2.12%) :arrow_up:
ipam/centralip/backend/cluster/cluster.go 84.93% <84.93%> (ø)

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 a77751b...7f202d5. Read the comment docs.