daniele-moro commented 2 years ago

Allow setting slice TC meter via UpfProgrammable APIs. Also, remove the no_tc apps, now TC is mandatory and we don't rely on default_tc in the physical pipeline. Depends on:

https://gerrit.onosproject.org/c/onos/+/25410

daniele-moro commented 2 years ago

retest this please

codecov[bot] commented 2 years ago

Codecov Report

Merging #503 (4270520) into main (8a103d2) will increase coverage by 0.10%. The diff coverage is 76.56%.

@@             Coverage Diff              @@
##               main     #503      +/-   ##
============================================
+ Coverage     69.85%   69.96%   +0.10%     
- Complexity      729      740      +11     
============================================
  Files            63       63              
  Lines          4771     4801      +30     
  Branches        528      530       +2     
============================================
+ Hits           3333     3359      +26     
- Misses         1161     1164       +3     
- Partials        277      278       +1

Impacted Files	Coverage Δ
...mproject/fabric/tna/behaviour/P4InfoConstants.java	`100.00% <ø> (ø)`
...umproject/fabric/tna/slicing/api/TrafficClass.java	`70.58% <37.50%> (-29.42%)`	:arrow_down:
.../fabric/tna/behaviour/upf/FabricUpfTranslator.java	`79.05% <78.12%> (+0.35%)`	:arrow_up:
...abric/tna/behaviour/upf/FabricUpfProgrammable.java	`62.26% <86.95%> (+1.53%)`	:arrow_up:
...ratumproject/fabric/tna/behaviour/FabricUtils.java	`76.47% <100.00%> (+0.47%)`	:arrow_up:
...atumproject/fabric/tna/stats/StatisticDataKey.java	`100.00% <0.00%> (+3.44%)`	:arrow_up:

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 8a103d2...4270520. Read the comment docs.

stratum / fabric-tna

[SDFAB-1104] Add slice meter support for UPF traffic #503

Codecov Report