bitflicker64 commented on PR #2962:
URL: https://github.com/apache/hugegraph/pull/2962#issuecomment-4007892571

   **How I tested:**
   
   1. Built a local Docker image from source with this fix applied (alongside 
the `getLeaderGrpcAddress` fix from #2961)
   2. Removed static IP workaround from docker-compose — switched back to 
hostnames (`pd0:8610`, `pd1:8610`, `pd2:8610`) for all raft peers, gRPC hosts, 
and PD addresses
   3. Brought up the full 3-node cluster (3 PD + 3 Store + 3 Server) in bridge 
network mode with no `ipam` static IP config
   4. pd2 won leader election on first boot
   
   **Results with pd2 as leader and hostnames only:**
   ```
   partitionCount:12 on all 3 stores ✅
   leaderCount:12 on all 3 stores ✅
   {"graphs":["hugegraph"]} ✅
   All 9 containers healthy ✅
   No "Blocked connection" warnings in any PD logs ✅
   ```
   
   **Before this fix:** `IpAuthHandler` compared raw hostname strings against 
incoming connection IPs — always failed in bridge mode, requiring static IPs as 
a workaround.
   
   **After this fix:** Hostnames are resolved to IPs at startup via 
`InetAddress.getAllByName()` — static IP workaround no longer needed.
   
   Related PR: #2952, #2961


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to