openyurtio / raven

provide layer 3 and layer 7 network connectivity among pods in different physical regions
Apache License 2.0
57 stars 37 forks source link

Bugfix/panic calculate ipset #167

Closed River-sh closed 8 months ago

River-sh commented 8 months ago

fix the bug of panic if cidr is empty

codecov[bot] commented 8 months ago

Codecov Report

Attention: Patch coverage is 25.00000% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 35.63%. Comparing base (d70eb16) to head (fa158b9). Report is 1 commits behind head on main.

Files Patch % Lines
pkg/networkengine/routedriver/vxlan/vxlan.go 25.00% 2 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #167 +/- ## ========================================== - Coverage 35.75% 35.63% -0.12% ========================================== Files 12 12 Lines 1474 1476 +2 ========================================== - Hits 527 526 -1 - Misses 840 842 +2 - Partials 107 108 +1 ``` | [Flag](https://app.codecov.io/gh/openyurtio/raven/pull/167/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openyurtio) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/openyurtio/raven/pull/167/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openyurtio) | `35.63% <25.00%> (-0.12%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openyurtio#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.