From 4d857cb5154bc6e00b1a5b767b6c8d237a23b7b2 Mon Sep 17 00:00:00 2001 From: bdchatham Date: Wed, 8 Jul 2026 15:41:09 -0700 Subject: [PATCH] =?UTF-8?q?fix(logging):=20demote=20per-mint=20bank=20log?= =?UTF-8?q?=20to=20Debug=20=E2=80=94=20mock=5Fbalances=20top-off=20spam?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Every mock_balances account top-off logs "minted coins from module account" at Info, once per funded sender — on a seiload chain that is a log line per fresh account. The mock path used to suppress this with a ctx NopLogger, but the package-level slog migration (#3050) removed ctx-scoped loggers, silently re-enabling the spam. Demote the bank keepers' per-mint Info log to Debug in both copies (sei-cosmos/x/bank, giga/deps/xbank). Block-mint visibility is unaffected: x/mint already logs "Minted coins" at Info per release. Also drop the now-stale NopLogger comment and dead ctx alias in the mock_balances top-off. Co-Authored-By: Claude Fable 5 --- giga/deps/xbank/keeper/keeper.go | 2 +- giga/deps/xevm/state/mock_balances.go | 6 ++---- sei-cosmos/x/bank/keeper/keeper.go | 2 +- x/evm/state/mock_balances.go | 6 ++---- 4 files changed, 6 insertions(+), 10 deletions(-) diff --git a/giga/deps/xbank/keeper/keeper.go b/giga/deps/xbank/keeper/keeper.go index 54cff2ef59..57d74e0744 100644 --- a/giga/deps/xbank/keeper/keeper.go +++ b/giga/deps/xbank/keeper/keeper.go @@ -494,7 +494,7 @@ func (k BaseKeeper) createCoins(ctx sdk.Context, moduleName string, amounts sdk. k.SetSupply(ctx, supply) } - logger.Info("minted coins from module account", "amount", amounts, "from", moduleName) + logger.Debug("minted coins from module account", "amount", amounts, "from", moduleName) // emit mint event ctx.EventManager().EmitEvent( diff --git a/giga/deps/xevm/state/mock_balances.go b/giga/deps/xevm/state/mock_balances.go index 5fe2bfef28..d2b6a4ba3d 100644 --- a/giga/deps/xevm/state/mock_balances.go +++ b/giga/deps/xevm/state/mock_balances.go @@ -59,13 +59,11 @@ func (s *DBImpl) topOffAccount(seiAddr sdk.AccAddress, amt *big.Int) { s.k.AccountKeeper().SetAccount(s.ctx, s.k.AccountKeeper().NewAccountWithAddress(s.ctx, seiAddr)) } - // Mint and send (use NopLogger to suppress log spam) usei, wei := SplitUseiWeiAmount(amt) coinsAmt := sdk.NewCoins(sdk.NewCoin(s.k.GetBaseDenom(s.ctx), usei.Add(sdk.OneInt()))) - ctx := s.ctx - if err := s.k.BankKeeper().MintCoins(ctx, types.ModuleName, coinsAmt); err != nil { + if err := s.k.BankKeeper().MintCoins(s.ctx, types.ModuleName, coinsAmt); err != nil { return } moduleAddr := s.k.AccountKeeper().GetModuleAddress(types.ModuleName) - _ = s.k.BankKeeper().SendCoinsAndWei(ctx, moduleAddr, seiAddr, usei, wei) + _ = s.k.BankKeeper().SendCoinsAndWei(s.ctx, moduleAddr, seiAddr, usei, wei) } diff --git a/sei-cosmos/x/bank/keeper/keeper.go b/sei-cosmos/x/bank/keeper/keeper.go index eff2b7b1ca..63b69461b2 100644 --- a/sei-cosmos/x/bank/keeper/keeper.go +++ b/sei-cosmos/x/bank/keeper/keeper.go @@ -586,7 +586,7 @@ func (k BaseKeeper) createCoins(ctx sdk.Context, moduleName string, amounts sdk. k.SetSupply(ctx, supply) } - logger.Info("minted coins from module account", "amount", amounts, "from", moduleName) + logger.Debug("minted coins from module account", "amount", amounts, "from", moduleName) // emit mint event ctx.EventManager().EmitEvent( diff --git a/x/evm/state/mock_balances.go b/x/evm/state/mock_balances.go index 3735cc6d20..6154d6bb15 100644 --- a/x/evm/state/mock_balances.go +++ b/x/evm/state/mock_balances.go @@ -59,13 +59,11 @@ func (s *DBImpl) topOffAccount(seiAddr sdk.AccAddress, amt *big.Int) { s.k.AccountKeeper().SetAccount(s.ctx, s.k.AccountKeeper().NewAccountWithAddress(s.ctx, seiAddr)) } - // Mint and send (use NopLogger to suppress log spam) usei, wei := SplitUseiWeiAmount(amt) coinsAmt := sdk.NewCoins(sdk.NewCoin(s.k.GetBaseDenom(s.ctx), usei.Add(sdk.OneInt()))) - ctx := s.ctx - if err := s.k.BankKeeper().MintCoins(ctx, types.ModuleName, coinsAmt); err != nil { + if err := s.k.BankKeeper().MintCoins(s.ctx, types.ModuleName, coinsAmt); err != nil { return } moduleAddr := s.k.AccountKeeper().GetModuleAddress(types.ModuleName) - _ = s.k.BankKeeper().SendCoinsAndWei(ctx, moduleAddr, seiAddr, usei, wei) + _ = s.k.BankKeeper().SendCoinsAndWei(s.ctx, moduleAddr, seiAddr, usei, wei) }