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

Multiple rules per interface #43

Closed
konetzed opened this issue Jan 12, 2017 · 5 comments
Closed

Multiple rules per interface #43

konetzed opened this issue Jan 12, 2017 · 5 comments
Milestone

Comments

@konetzed
Copy link

konetzed commented Jan 12, 2017

I would like to do something like this

tcset --device bond0.310 --rate 500M --network 192.168.2.0/24
tcset --device bond0.310 --rate 100M --network 192.168.0.0/24

The first one succeeds w/o issue but the second command fails. Is this not something tcconfig can do?

root@ubuntu:~# tcset --device bond0.310 --rate 500M --network 192.168.2.0/24
root@ubuntu:~# tcset --device bond0.310 --rate 100M --network 192.168.0.0/24
[2017-01-12 20:53:30.357743] NOTICE: subprocrunner: failed to add qdisc: prio qdisc already exists. execute with --overwrite option if you want to overwrite the existing settings.
[2017-01-12 20:53:30.367540] NOTICE: subprocrunner: failed to add qdisc: netem qdisc already exists. execute with --overwrite option if you want to overwrite the existing settings.
[2017-01-12 20:53:30.377499] WARNING: subprocrunner: returncode=2, stderr=RTNETLINK answers: File exists
root@ubuntu:~#
@konetzed
Copy link
Author

Also using tcconfig (0.7.0a2) installed via pip on ubuntu 16.04

@thombashi
Copy link
Owner

thombashi commented Jan 15, 2017

@konetzed
tcconfig has no capability to do that until 0.7.0-alpha2.
So, I've added a prototype of the feature with 0.7.0-alpha3.
In this version, you can add multiple rules to a same interface with --add option like the below.

tcset --device bond0.310 --rate 500M --network 192.168.2.0/24
tcset --device bond0.310 --rate 100M --network 192.168.0.0/24 --add

Could you try this version?
Does it fulfill your requirements?

@thombashi thombashi added this to the 0.7.0 milestone Jan 19, 2017
@thombashi
Copy link
Owner

I'll close the issue since the future added to the tcconfig 0.7.0.
Any feedback would be welcome.
Feel free to reopen.

Thanks,

@konetzed
Copy link
Author

konetzed commented Feb 1, 2017

Wanted to say thanks for this, I didn't see the update on the issue until now!

@thombashi
Copy link
Owner

You are welcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants