-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Don't restart rippled during apt upgrade #3901
Conversation
Great context, thank you for that. Is it true that this will also prevent the service from starting automatically when the machine boots? @manojsdoshi, do you think that will be a problem for anyone? |
The rippled.service unit file packaged still dictates that it starts on boot. This is just some in-between uninstall/install magic. |
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.
A brief summary of our offline conversation:
- A lot of operators are configured to do automatic rippled upgrades. Until they manually intervene to restart rippled, they could have a pretty long downtime, which could be bad for them, and for the network. I would very much prefer if you could find a solution that checks after install for a running rippled, and starts it if not running (which I think we can assume indicates the old version was stopped by the old .deb config).
- Please rewrite the commit message to be more informative and link back to the issue. e.g.
Don't restart rippled during apt upgrade:
Resolves #3294
87ce0c0
to
d402b6a
Compare
d402b6a
to
0eebe86
Compare
I could have sworn the obvious |
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
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.
Looks good
"Merged as part of #3948" |
Keep outgoing version of rippled running during upgrade. Rippled must explicitly be restarted.
Context of Change
The debian packages generated now automatically restart rippled when upgrading. Installing rippled after this change will initiate a restart from 1.7 => 1.8 as a result of the outgoing package's postrm script.
Subsequent upgrades will keep the outgoing version running, requiring intervention to upgrade the running version i.e.
systemctl restart rippled
or reboot.Type of Change
I don't know why I picked "refactor" before, I now see how this could break expectations, but not really sure which of these applies.
Test Plan
What it looks like in practice
Future Tasks
Eventually all this package generation should be handled by native CMake features rather than CMake calling scripts that call CMake that call deb-helper programs.