You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/03/23 12:42:00 UTC

[GitHub] [pulsar] zengguan opened a new issue #14820: [broker] parameter "loadBalancerOverrideBrokerNicSpeedGbps" does not work

zengguan opened a new issue #14820:
URL: https://github.com/apache/pulsar/issues/14820


   **Describe the bug**
   machine:tencent Cloud Virtual Machine,CVM 
   os: tencent os tlinux 2.6
   i wanna set `loadBalancerOverrideBrokerNicSpeedGbps=9` to limit bandwidthout, but it does not work because `org.apache.pulsar.broker.loadbalance.impl.LinuxBrokerHostUsageImpl  getNics()` get two nics,one is physical Nic eth1,another one is virtual Nic veth51f9d919
   
   **To Reproduce**
   Steps to reproduce the behavior:
   1. set broker parameter `loadBalancerOverrideBrokerNicSpeedGbps=9`
   2. check nics path
   ```
   [root@9 ~]# cd /sys/class/net
   [root@9 /sys/class/net]# ls -l
   total 0
   lrwxrwxrwx 1 root root 0 Mar  8 21:01 cbr0 -> ../../devices/virtual/net/cbr0
   lrwxrwxrwx 1 root root 0 Mar  8 21:01 dummy0 -> ../../devices/virtual/net/dummy0
   lrwxrwxrwx 1 root root 0 Mar  8 20:59 eth1 -> ../../devices/pci0000:00/0000:00:05.0/virtio0/net/eth1
   lrwxrwxrwx 1 root root 0 Mar  8 21:01 kube-ipvs0 -> ../../devices/virtual/net/kube-ipvs0
   lrwxrwxrwx 1 root root 0 Mar  8 20:59 lo -> ../../devices/virtual/net/lo
   lrwxrwxrwx 1 root root 0 Mar  8 21:03 veth51f9d919 -> ../../devices/virtual/net/veth51f9d919
   ```
   3. Becasue `org.apache.pulsar.broker.loadbalance.impl.LinuxBrokerHostUsageImpl  isPhysicalNic(Path path)` path is relative path, so veth51f9d919 in `!path.toString().contains("/virtual/")` is true, Expected false。
   4. In this condition, `org.apache.pulsar.broker.loadbalance.impl.LinuxBrokerHostUsageImpl  getTotalNicLimitKbps()` return 18874368, Expected 9437184。the limit bandwidthout does not work 
   
   **Expected behavior**
   virtual nic `veth51f9d919` in `!path.toString().contains("/virtual/")` is  false。
   
   **Desktop (please complete the following information):**
    - OS: tencent os tlinux 2.6
   
   **Additional context**
   i wanna fix this issue


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [pulsar] codelipenghui closed issue #14820: [broker] parameter "loadBalancerOverrideBrokerNicSpeedGbps" does not work

Posted by GitBox <gi...@apache.org>.
codelipenghui closed issue #14820:
URL: https://github.com/apache/pulsar/issues/14820


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org