Skip to content

Commit

Permalink
fix(clustering/rpc): do not log disconnected node for `kong.sync.v2.n…
Browse files Browse the repository at this point in the history
…otify_new_version` (#14255)

KAG-6372
  • Loading branch information
chronolaw authored Feb 12, 2025
1 parent 04f8094 commit 706a32e
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion kong/clustering/services/sync/hooks.lua
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,9 @@ function _M:notify_all_nodes()
for _, node in ipairs(get_all_nodes_with_sync_cap()) do
local res, err = kong.rpc:call(node, "kong.sync.v2.notify_new_version", msg)
if not res then
if not err:find("requested capability does not exist", nil, true) then
if not err:find("requested capability does not exist", nil, true) and
not err:find("node is not connected", nil, true)
then
ngx_log(ngx_ERR, "unable to notify ", node, " new version: ", err)
end
end
Expand Down

1 comment on commit 706a32e

@github-actions
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bazel Build

Docker image available kong/kong-dev:706a32ebec5fe40862b81f8864622971d985201c
Artifacts available https://github.com/Kong/kong/actions/runs/13276377163

Please sign in to comment.