[#128] IR: Do not try to emit GAS to nobody #151
Labels
No Label
P0
P1
P2
P3
badger
frostfs-adm
frostfs-cli
frostfs-ir
frostfs-lens
frostfs-node
good first issue
triage
Infrastructure
blocked
bug
config
discussion
documentation
duplicate
enhancement
go
help wanted
internal
invalid
kludge
observability
perfomance
question
refactoring
wontfix
No Milestone
No Assignees
4 Participants
Notifications
Due Date
Dependencies
No dependencies set.
Reference: TrueCloudLab/frostfs-node#151
Loading…
Reference in New Issue
There is no content yet.
Delete Branch "carpawell/frostfs-node:fix/not-send-gas-to-nobody"
Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?
Fix sending GAS to an empty extra wallets receivers list. Also, send GAS to
extra wallets even if netmap is empty.
Signed-off-by: Pavel Karpy p.karpy@yadro.com
Saw
in logs.
@ -72,2 +87,3 @@
}
err = ap.morphClient.TransferGas(key.GetScriptHash(), gasPerNode)
if extraLen != 0 {
I had this condition initially but lost it in transition to batch processing. 👽
@ -71,1 +83,4 @@
}
}
} else {
ap.log.Debug("gas emission: empty network map")
Can we use 1 log message like
Debug("gas emission", zap.Int("network_map", nmLen), zap.Int("extra", extraLen))
instead on 3 (2 inelse
and 1 if both are 0).fixed
fixed
72139974f4
to32f95d5eee
32f95d5eee
to8243ac6307