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
updating startup scripts #3873
updating startup scripts #3873
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #3873 +/- ##
========================================
Coverage 0.16% 0.16%
========================================
Files 337 337
Lines 286759 286759
========================================
Hits 469 469
Misses 286290 286290 📣 We’re building smart automated test selection to slash your CI/CD build times. 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.
Just a couple minor comments, but otherwise LGTM!
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.
Blindly approving this. I famously don't use systemd
;) But if it works on your end and Charlie's end, I'm sure it's fine.
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.
Send it!
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.
LGTM!
With the
stacks.system
unitfile, stopping the service during when not at chain tip was resulting in a panic (thread stopped).The methods used in the current systemd unit no longer appear to be needed where a PID is recorded via forking the process using
sh
.Using a standard execstart line of
ExecStart=/usr/local/bin/stacks-node start --config=/etc/stacks-blockchain/Config.toml
seems more than sufficient now - start/stop/restart all work as expected during a sync as well as when the chain has reached tip.
Minor correction was made in the
stacks.init
script to use the default logfile mentioned in the comment vs a non-existent/stacks-blockchain
directory (this is configurable by the user).