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

Bandwidth plugin prevents NP enforcement #68

Open
jayanthvn opened this issue Sep 16, 2023 · 3 comments
Open

Bandwidth plugin prevents NP enforcement #68

jayanthvn opened this issue Sep 16, 2023 · 3 comments
Assignees
Labels
enhancement New feature or request pending triage

Comments

@jayanthvn
Copy link
Contributor

Bandwidth plugin uses TC filter and attaches TBF(token bucket filter) queuing discipline (qdisc) on both ingress and egress. But we use NP enforcement with TC filter "clsact" which will collide with B/W plugin.

We are working on providing a solution for co-existence.

@oOraph
Copy link

oOraph commented Nov 10, 2023

note that there is a pending pr in the bandwidth plugin to change the queuing disc from tbf to htb to add some feature there.

if the solution you find was compatible with this qdisc too that would be great :)

containernetworking/plugins#921

thanks

@sjastis sjastis added enhancement New feature or request and removed bug Something isn't working labels Mar 19, 2024
@benben
Copy link

benben commented Jul 10, 2024

@jayanthvn Did you manage to make any progress on this issue? We would like to use both. Thanks!

@jsferner
Copy link

jsferner commented Oct 7, 2024

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pending triage
Projects
None yet
Development

No branches or pull requests

5 participants