-
Notifications
You must be signed in to change notification settings - Fork 30
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
service varnish restart fails #71
Comments
I created a patch which fixes that specific problem for me, but probably breaks something else: https://gist.github.com/mgla/3e04a06bc21e6def88133917def56e9f The problems seems to occur be in the unhandled else in the outer if clause. |
Hi @mgla, thanks for the report. I'll try to reproduce the error, and test the proposed fix. Would you mind creating a pull request for the patch? |
Actually @mgla, no need to create a PR for that patch as it does indeed remove logic that we want to keep. I'll try to fix the script without stripping the configtest out. |
@denisbr Thanks. I really could not recommend that patch. |
@mgla I tried reproducing your issue, but start, stop, restart all worked fine here. Could you try using the normal initscript again, and if it still fails, paste the full output, and do |
@mgla Did you try this again? |
Can't reproduce anymore. Was probably something else. I am on it, but I will close this report for now... |
Expected Behavior
If i run service varnish restart, it should restart usually
Current Behavior
I try to restart varnish and I get this error:
Context
sudo service varnish start
works fine, same with
sudo service varnish stop
It is just the restart that does not work.
Your Environment
Version used: varnish-4.1.5 revision 2c82b1c
Operating System and version: Ubuntu 14.04
I used this package: https://packagecloud.io/varnishcache/varnish41/packages/ubuntu/trusty/varnish_4.1.5-1~trusty_amd64.deb
The text was updated successfully, but these errors were encountered: