stratum / fabric-tna

The SD-Fabric data plane
https://docs.sd-fabric.org/
30 stars 15 forks source link

[SDFAB-1009] Return only pending add and added UPF entities when reading #489

Closed daniele-moro closed 2 years ago

daniele-moro commented 2 years ago

When reading UPF entities via the UpfProgrammable, we should get only entities that are installed or being installed on the UPF device. For this reason, we filter the flow entries we get from the flow rule service and keep only those in PENDING_ADDor ADDED state.

codecov[bot] commented 2 years ago

Codecov Report

Merging #489 (5d42071) into main (d4c1749) will decrease coverage by 0.04%. The diff coverage is 58.62%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #489      +/-   ##
============================================
- Coverage     69.90%   69.85%   -0.05%     
- Complexity      722      727       +5     
============================================
  Files            63       63              
  Lines          4748     4754       +6     
  Branches        523      527       +4     
============================================
+ Hits           3319     3321       +2     
- Misses         1157     1158       +1     
- Partials        272      275       +3     
Impacted Files Coverage Δ
...abric/tna/behaviour/upf/FabricUpfProgrammable.java 60.16% <58.62%> (-0.34%) :arrow_down:

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 d4c1749...5d42071. Read the comment docs.