Skip to content

Commit

Permalink
sidechain/deploy: Do not repeat already failed actions within one block
Browse files Browse the repository at this point in the history
Some actions of committee group distribution routine are similar for all
domains, so it's redundant to repeat same failed ops for other domains.

Signed-off-by: Leonard Lyubich <[email protected]>
  • Loading branch information
cthulhu-rider committed Jul 18, 2023
1 parent e8b560d commit 41384e9
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions pkg/morph/deploy/group.go
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@ func initShareCommitteeGroupKeyAsLeaderTick(ctx context.Context, prm initCommitt
} else {
l.Error("failed to send transaction registering domain in the NNS, will try again later", zap.Error(err))
}
continue
return
}

l.Info("transaction registering domain in the NNS has been successfully sent, will wait for the outcome",
Expand All @@ -186,7 +186,7 @@ func initShareCommitteeGroupKeyAsLeaderTick(ctx context.Context, prm initCommitt
continue
} else if !errors.Is(err, errMissingDomainRecord) {
l.Error("failed to lookup NNS domain record, will try again later", zap.Error(err))
continue
return
}

l.Info("missing record of the NNS domain, needed to be set")
Expand Down Expand Up @@ -221,7 +221,7 @@ func initShareCommitteeGroupKeyAsLeaderTick(ctx context.Context, prm initCommitt
} else {
l.Error("failed to send transaction setting NNS domain record, will try again later", zap.Error(err))
}
continue
return
}

l.Info("transaction setting NNS domain record has been successfully sent, will wait for the outcome",
Expand Down

0 comments on commit 41384e9

Please sign in to comment.