You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by pe...@apache.org on 2022/04/19 09:51:05 UTC

[pulsar] 03/26: [fix][broker] filter the virtual NIC with relative path (#14829)

This is an automated email from the ASF dual-hosted git repository.

penghui pushed a commit to branch branch-2.9
in repository https://gitbox.apache.org/repos/asf/pulsar.git

commit 1c2eb897fb49c8ad3e70f88d104979d197564b57
Author: grayson <91...@qq.com>
AuthorDate: Thu Mar 24 23:08:50 2022 +0800

    [fix][broker] filter the virtual NIC with relative path (#14829)
    
    (cherry picked from commit 9e919c71a46a2b936a44ed0ea9f2592b872cee86)
---
 .../pulsar/broker/loadbalance/impl/LinuxBrokerHostUsageImpl.java    | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/LinuxBrokerHostUsageImpl.java b/pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/LinuxBrokerHostUsageImpl.java
index 53d8d3e379d..9bd5f4e6524 100644
--- a/pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/LinuxBrokerHostUsageImpl.java
+++ b/pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/LinuxBrokerHostUsageImpl.java
@@ -227,10 +227,10 @@ public class LinuxBrokerHostUsageImpl implements BrokerHostUsage {
     }
 
     private boolean isPhysicalNic(Path path) {
-        if (path.toString().contains("/virtual/")) {
-            return false;
-        }
         try {
+            if (path.toRealPath().toString().contains("/virtual/")) {
+                return false;
+            }
             // Check the type to make sure it's ethernet (type "1")
             String type = new String(Files.readAllBytes(path.resolve("type")), StandardCharsets.UTF_8).trim();
             // wireless NICs don't report speed, ignore them.