diff options
author | Viacheslav Hletenko <v.gletenko@vyos.io> | 2024-01-23 10:53:08 +0000 |
---|---|---|
committer | Viacheslav Hletenko <v.gletenko@vyos.io> | 2024-01-23 10:53:08 +0000 |
commit | 6ddfe6328e1cbdde1b70763b39e3a87f8374755a (patch) | |
tree | f0ea2097dcdbba29f7892961b671d57157c69bde /src/op_mode/snmp_ifmib.py | |
parent | 20106f2e827ecfa9dc347d102fe809c736d17a48 (diff) | |
download | vyos-1x-6ddfe6328e1cbdde1b70763b39e3a87f8374755a.tar.gz vyos-1x-6ddfe6328e1cbdde1b70763b39e3a87f8374755a.zip |
T5974: Fix QoS shape bandwidth and ceil calculation for default
The default `bandwidth` and `ceiling` should calculate values
based on <tag> bandwidth but currently it gets the value from
qos.base `/sys/class/net/{self._interface}/speed`
```
set qos policy shaper SHAPER bandwidth '20mbit'
set qos policy shaper SHAPER default bandwidth '95%'
set qos policy shaper SHAPER default ceiling '100%'
```
It causes wrong calculations for class `default` i.e
950Mbit for bandwidth (expected 95% of bandwidth, 19Mbit)
1Gbit for ceil (expected 100% of bandwidth, 20Mbit)
Gets incorrect values
```
r4# tc class show dev eth1
class htb 1:1 root rate 20Mbit ceil 20Mbit burst 1600b cburst 1600b
class htb 1:a parent 1:1 leaf 8053: prio 0 rate 200Kbit ceil 200Kbit burst 1Mb cburst 1600b
class htb 1:b parent 1:1 leaf 8054: prio 7 rate 950Mbit ceil 1Gbit burst 15200b cburst 1375b
```
Fix this
Diffstat (limited to 'src/op_mode/snmp_ifmib.py')
0 files changed, 0 insertions, 0 deletions