Skip to content
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

Adds init info to fix apt-get upgrade and other related stuff #138

Merged
merged 2 commits into from
Jul 6, 2017

Conversation

keeganbrown
Copy link
Contributor

@keeganbrown keeganbrown commented Jul 5, 2017

Why?

What changed?

  • This fixes the puma init script so that it doesn't cause apt-get upgrade to fail (which in turn causes a whole host of other things to fail)

@BrandonMathis
Copy link
Contributor

This is just commented out code. What are you trying to do here?

@keeganbrown
Copy link
Contributor Author

@BrandonMathis this file is being added to init.d and is causing problems when running upgrades (which the letsencrypt renew script does automatically). These comments are LSB Tags https://wiki.debian.org/LSBInitScripts/DependencyBasedBoot, but the long and short of it is that without these comments, software updating would not complete, the ssl cert would not renew, and other problems ensued.

Here's the relevant error message from the full dump on #137 :

insserv: warning: script 'puma' missing LSB tags and overrides
insserv: There is a loop between service monit and puma if stopped
insserv:  loop involving service puma at depth 2
insserv:  loop involving service monit at depth 1
insserv: Stopping puma depends on monit and therefore on system facility `$all' which can not be true!
insserv: exiting now without changing boot order!
update-rc.d: error: insserv rejected the script header

@BrandonMathis BrandonMathis merged commit 8478139 into master Jul 6, 2017
@BrandonMathis BrandonMathis deleted the bugfix/missing-init-info-breaks-apt branch July 6, 2017 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants