Compare commits

...

9 Commits

Author SHA1 Message Date
copilot-swe-agent[bot]
06ca83cd1a Add marker for clean branch push
Co-authored-by: xemul <4498177+xemul@users.noreply.github.com>
2025-11-20 09:10:12 +00:00
copilot-swe-agent[bot]
121669560f Addressing PR comments
Co-authored-by: xemul <4498177+xemul@users.noreply.github.com>
2025-11-20 09:00:49 +00:00
copilot-swe-agent[bot]
5b1307ff9c Remove extraneous trailing newline
Co-authored-by: mykaul <4655593+mykaul@users.noreply.github.com>
2025-11-20 08:54:50 +00:00
copilot-swe-agent[bot]
0eb49a3403 Remove temporary marker file
Co-authored-by: mykaul <4655593+mykaul@users.noreply.github.com>
2025-11-20 08:53:48 +00:00
copilot-swe-agent[bot]
bfb070eabb Clean up: Single commit with comprehensive message
Co-authored-by: mykaul <4655593+mykaul@users.noreply.github.com>
2025-11-20 08:52:55 +00:00
copilot-swe-agent[bot]
d171536e9b gossip: Fix log to show host-id/ip instead of host-id/host-id
In gossiper::remove_endpoint(), the DOWN status log was printing the
host ID twice instead of host-id/ip-address.

Before:
  [shard 0: gms] gossip - InetAddress e88875d7.../e88875d7... is now DOWN

After:
  [shard 0: gms] gossip - InetAddress e88875d7.../192.168.1.10 is now DOWN

Changed line 784 in gms/gossiper.cc to use the 'ip' variable (already
extracted from endpoint state) instead of the 'endpoint' parameter
(which is a locator::host_id). This aligns with the logging pattern
used elsewhere in the codebase (e.g., line 1830).

Fixes #27113

Co-authored-by: mykaul <4655593+mykaul@users.noreply.github.com>
2025-11-20 08:50:22 +00:00
copilot-swe-agent[bot]
20ee5fca8c Trigger force push after rebase
Co-authored-by: xemul <4498177+xemul@users.noreply.github.com>
2025-11-20 08:30:38 +00:00
copilot-swe-agent[bot]
9e52e1ba1d Fix gossip logging to show host-id/ip instead of host-id/host-id
Co-authored-by: mykaul <4655593+mykaul@users.noreply.github.com>
2025-11-19 11:26:05 +00:00
copilot-swe-agent[bot]
3a32a3e000 Initial plan 2025-11-19 11:14:31 +00:00
2 changed files with 2 additions and 1 deletions

View File

@@ -0,0 +1 @@
Clean branch with single commit

View File

@@ -781,7 +781,7 @@ future<> gossiper::remove_endpoint(locator::host_id endpoint, permit_id pid) {
if (was_alive) {
try {
logger.info("InetAddress {}/{} is now DOWN, status = {}", state->get_host_id(), endpoint, get_gossip_status(*state));
logger.info("InetAddress {}/{} is now DOWN, status = {}", state->get_host_id(), ip, get_gossip_status(*state));
co_await do_on_dead_notifications(ip, std::move(state), pid);
} catch (...) {
logger.warn("Fail to call on_dead callback: {}", std::current_exception());