Skip to content

Commit

Permalink
Log total_home when used
Browse files Browse the repository at this point in the history
I have a theory total_home (only set on shard0's service) could be ran
using a service from another shard, but on shard0. We should add logs
for what total_allowance is because that's what the stacktrace reported
as crashing.

Signed-off-by: Tyler Rockwood <[email protected]>
  • Loading branch information
rockwotj committed May 27, 2023
1 parent fcd6345 commit da6ba58
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/v/net/conn_quota.cc
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@

#include "conn_quota.h"

#include "cluster/types.h"
#include "config/configuration.h"
#include "config/validators.h"
#include "hashing/xx.h"
Expand Down Expand Up @@ -450,8 +451,11 @@ void conn_quota::do_put(ss::net::inet_address addr) {

auto home_shard = addr_to_shard(addr);
if (home_shard == ss::this_shard_id()) {
vlog(rpc::rpclog.trace, "do_put: release directly to home");
auto allowance = get_home_allowance(addr);
vlog(
rpc::rpclog.trace,
"do_put: release directly to home allowance={}",
allowance);
allowance->put();
if (should_leave_reclaim(*allowance)) {
cancel_reclaim_to(addr, allowance);
Expand Down

0 comments on commit da6ba58

Please sign in to comment.