Skip to content

Commit

Permalink
fix(clustering): fix the bug in compatible logic in this [PR](#10400) (
Browse files Browse the repository at this point in the history
…#10501)

that forgetting to set the `has_updated` flag
  • Loading branch information
liverpool8056 authored Mar 29, 2023
1 parent 1e610b7 commit 96fc3b5
Showing 1 changed file with 8 additions and 5 deletions.
13 changes: 8 additions & 5 deletions kong/clustering/compat/init.lua
Original file line number Diff line number Diff line change
Expand Up @@ -368,11 +368,14 @@ function _M.update_compatible_payload(payload, dp_version, log_suffix)

for _, name in ipairs(entity_names) do
for _, config_entity in ipairs(config_table[name] or {}) do
ngx_log(ngx_WARN, _log_prefix, "Kong Gateway v" .. KONG_VERSION ..
" contains configuration '" .. name .. ".updated_at'",
" which is incompatible with dataplane version " .. dp_version .. " and will",
" be removed.", log_suffix)
config_entity.updated_at = nil
if config_entity["updated_at"] ~= nil then
ngx_log(ngx_WARN, _log_prefix, "Kong Gateway v" .. KONG_VERSION ..
" contains configuration '" .. name .. ".updated_at'",
" which is incompatible with dataplane version " .. dp_version .. " and will",
" be removed.", log_suffix)
config_entity["updated_at"] = nil
has_update = true
end
end
end
end
Expand Down

1 comment on commit 96fc3b5

@khcp-gha-bot
Copy link

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:96fc3b56a17436b9936ed65dae2c219045b3c2c1
Artifacts available https://github.com/Kong/kong/actions/runs/4555633707

Please sign in to comment.