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

[#181] ir: Do not deposit notary GAS by non-alphabet nodes
All checks were successful
ci/woodpecker/pr/pre-commit Pipeline was successful
Details

Browse source 
Signed-off-by: Pavel Karpy <p.karpy@yadro.com>
This commit is contained in:
Pavel Karpy 2023-03-29 17:55:11 +03:00
parent e843e7f090
commit e71b6582b5
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,