Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enforce policing of red traffic in the queues table #499

Merged
merged 2 commits into from
Feb 24, 2022
Merged

Conversation

ccascone
Copy link
Member

Policing is enforced only for elastic and real-time TCs. This is required to implement per-app and per-session MBR policing of UPF traffic. We also fix entries for the control TC to include handling of yellow packets (before we were handling only green and red).

@codecov
Copy link

codecov bot commented Feb 24, 2022

Codecov Report

Merging #499 (e976b65) into main (e30fb04) will increase coverage by 0.00%.
The diff coverage is 89.47%.

Impacted file tree graph

@@            Coverage Diff            @@
##               main     #499   +/-   ##
=========================================
  Coverage     69.85%   69.85%           
- Complexity      727      729    +2     
=========================================
  Files            63       63           
  Lines          4763     4771    +8     
  Branches        527      528    +1     
=========================================
+ Hits           3327     3333    +6     
- Misses         1160     1161    +1     
- Partials        276      277    +1     
Impacted Files Coverage Δ
...atumproject/fabric/tna/slicing/SlicingManager.java 83.81% <89.47%> (-0.20%) ⬇️

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 e30fb04...e976b65. Read the comment docs.

@ccascone ccascone merged commit 50d1b83 into main Feb 24, 2022
@ccascone ccascone deleted the red-policing branch February 24, 2022 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants