futurewei-cloud / alcor

Alcor: Cloud native SDN platform powered by Kubernetes and Istio
MIT License
32 stars 33 forks source link

[Network Config Manager] Demo - Part 2 #566

Closed xieus closed 3 years ago

xieus commented 3 years ago

This is a follow-up change with PR #562 .

codecov-io commented 3 years ago

Codecov Report

Merging #566 (a34e9db) into master (084b3f7) will decrease coverage by 0.06%. The diff coverage is 0.00%.

:exclamation: Current head a34e9db differs from pull request most recent head e6d0aa7. Consider uploading reports for the commit e6d0aa7 to get more accurate results Impacted file tree graph

@@             Coverage Diff              @@
##             master     #566      +/-   ##
============================================
- Coverage     33.97%   33.91%   -0.07%     
- Complexity     1236     1242       +6     
============================================
  Files           508      509       +1     
  Lines         12549    12589      +40     
  Branches       1574     1576       +2     
============================================
+ Hits           4264     4269       +5     
- Misses         7707     7746      +39     
+ Partials        578      574       -4     
Impacted Files Coverage Δ Complexity Δ
...configmanager/client/gRPC/GoalStateClientImpl.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...anager/server/grpc/GoalStateProvisionerServer.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
.../netwconfigmanager/server/grpc/OnDemandServer.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
...twconfigmanager/util/NetworkConfigManagerUtil.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...alcor/elasticipmanager/dao/ElasticIpAllocator.java 63.53% <0.00%> (-0.28%) 48.00% <0.00%> (ø%)
...rewei/alcor/portmanager/processor/PortContext.java 71.62% <0.00%> (+2.70%) 33.00% <0.00%> (+2.00%)
...i/alcor/portmanager/processor/RouterProcessor.java 85.29% <0.00%> (+5.88%) 10.00% <0.00%> (+1.00%)
...com/futurewei/alcor/macmanager/utils/MacUtils.java 65.21% <0.00%> (+8.69%) 8.00% <0.00%> (+1.00%)

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 084b3f7...e6d0aa7. Read the comment docs.

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging 67bb66d9a04301e057ccd59249e26c9220893810 into 084b3f7d715f9b4a3bf5adb9a30305621aef40dd - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging 0c6302ce1b5cece14db91fa2482e7636034ce77d into 3103cd98fcb275b126379ce9540b412abb8883d6 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging 2ff4863df66b60b5394365564a889ac792c222a6 into 3103cd98fcb275b126379ce9540b412abb8883d6 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging c4bf8b11edf356f8f7b51dc570e87520e312ab8f into 3103cd98fcb275b126379ce9540b412abb8883d6 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging 531b6dc13180857dd9e12d860d5d82e0fd9c4751 into 3103cd98fcb275b126379ce9540b412abb8883d6 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging 11a833d1a0ba76ff8e86faa03f286c2ec6895e33 into 3103cd98fcb275b126379ce9540b412abb8883d6 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging 1ec56ee4a313dd1e24bb6c802c111a86a426632b into 3103cd98fcb275b126379ce9540b412abb8883d6 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging 15e06f64a1715227ad911803d73e360160d752d5 into 3103cd98fcb275b126379ce9540b412abb8883d6 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging eb81fff09a1f94a5d929d6f44b6c8b920b24795a into 3103cd98fcb275b126379ce9540b412abb8883d6 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging a486f21673b3d8489f4e947eb3b3c2e4855b92ab into bc3b4d1451626ab23f1e46b078dd862030a3696e - view on LGTM.com

new alerts: