search

lf-edge / eve

EVE is Edge Virtualization Engine
https://www.lfedge.org/projects/eve/
Apache License 2.0
478 stars 164 forks source link

[WIP] Change IPAssignments definition from map to slice #4225

Closed eriknordmark closed 2 months ago

eriknordmark commented 2 months ago

Two unrelated commits in this draft PR. Still needs to be tested.

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 19.69%. Comparing base (b88019a) to head (3ebdb94). Report is 69 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #4225 +/- ## ======================================= Coverage 19.69% 19.69% ======================================= Files 8 8 Lines 2600 2600 ======================================= Hits 512 512 Misses 1985 1985 Partials 103 103 ```

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

eriknordmark commented 2 months ago

Per @milan-zededa this is not needed since pubsub using cmp.Equal which correctly compares maps.