New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
removed process start function when shuffled out #2010
Conversation
log.Debug("closing node") | ||
|
||
switch endProcessArgument.Reason { | ||
case core.ShuffledOut: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
leave closing node final log
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done. also added description
to the internal signal close log
@@ -1346,7 +1345,7 @@ func startNode(ctx *cli.Context, log logger.Logger, version string) error { | |||
case <-sigs: | |||
log.Info("terminating at user's signal...") | |||
case sig = <-chanStopNodeProcess: | |||
log.Info("terminating at internal stop signal", "reason", sig.Reason) | |||
log.Info("terminating at internal stop signal", "reason", sig.Reason, "description", sig.Description) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
System tests passed.
removed
handleAppClose
andnewStartInEpoch
functions from main as they were not working as expected. When running a node usingelrond-go-scripts-v2
it caused additional log files due to multiple processes creation.