TrueCloudLab/frostfs-node
18
1
Fork 27
Code Issues 91 Pull requests 2 Projects Releases 1 Activity Actions

[#181] ir: Do not deposit notary GAS by non-alphabet nodes #186

Closed
carpawell wants to merge 1 commit from carpawell/frostfs-node:fix/IRs-do-not-do-deposits into master
pull from: carpawell/frostfs-node:fix/IRs-do-not-do-deposits
merge into: TrueCloudLab:master
Conversation 1 Commits 1 Files changed 2 +5 -2
2 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
CHANGELOG.md
View file

@ -53,6 +53,7 @@ Changelog for FrostFS Node
- Adding of public key for nns group `group.frostfs` at init step (#130) - Adding of public key for nns group `group.frostfs` at init step (#130)
- Iterating over just removed files by FSTree (#98) - Iterating over just removed files by FSTree (#98)
- Parts of a locked object could not be removed anymore (#141) - Parts of a locked object could not be removed anymore (#141)
- Non-alphabet nodes do not claim notary deposits (#181)
### Removed ### Removed
### Updated ### Updated

6
pkg/innerring/innerring.go
View file

@ -179,7 +179,9 @@ func (s *Server) Start(ctx context.Context, intError chan<- error) (err error) {
return err return err
} }
if !s.mainNotaryConfig.disabled { isAlpha := s.IsAlphabet()
if isAlpha && !s.mainNotaryConfig.disabled {
err = s.initNotary(ctx, err = s.initNotary(ctx,
s.depositMainNotary, s.depositMainNotary,
s.awaitMainNotaryDeposit, s.awaitMainNotaryDeposit,
@ -190,7 +192,7 @@ func (s *Server) Start(ctx context.Context, intError chan<- error) (err error) {
} }
} }
if !s.sideNotaryConfig.disabled { if isAlpha && !s.sideNotaryConfig.disabled {
err = s.initNotary(ctx, err = s.initNotary(ctx,
s.depositSideNotary, s.depositSideNotary,
s.awaitSideNotaryDeposit, s.awaitSideNotaryDeposit,