-
Notifications
You must be signed in to change notification settings - Fork 23
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
Latest stable is crashing #31
Comments
I'm having the same issue. Running The following is my compose file:
|
Thanks to both of you for reporting. Can anyone post anything further that covers the crash? I can't reproduce anything when I use the image pulled from docker hub. Appreciate any additional information you can provide, ie @MyWay what command you're running to get your error, or maybe @DArqueBishop more context on what is or isn't happening- maybe run it in console mode to see errors? Thanks |
I ran
ETA: This is only happening with the |
@DArqueBishop is this a new/fresh instance of eggdrop, or are you using an existing config file in the data directory? |
It’s using an existing config file. |
@MyWay I would take a look at the example docker-compose file the docs provide at https://github.com/eggheads/eggdrop-docker/blob/master/docker-compose.yml and add in the required environmental variables (and other required settings), or mount an existing eggdrop configuration file. @DArqueBishop I would double-check that the path and config file you are using exist where docker thinks it is- my working theory on this is the NICK and SERVER variables are not set in the docker-compose file, and there is no eggdrop config file it can find to load these variables from. For fun, you could try adding the environment variables into your compose file and see if that fixes it to narrow down the issue. |
Keep in mind that it's working for both of us with the previous version. Any var break change? Any different setting for config file location? Because otherwise it can't be that. |
@MyWay If what you pasted is truly your full docker-compose file, then it should never have started with any version. What would be most helpful here is running it in foreground as I suggested above (no -d) and sharing the output. |
It was just to give you the command I used:
Change 1.9.5 to latest version to make it crash when starting. No error shown. Logs are those I pasted in the initial post.
|
@DArqueBishop thank you for your help testing the fix. I think (hope?) the most recent update just pushed to the eggdrop:develop image holds the fix. It takes about 24-48 hours for these changes to make it onto docker hub, so feel free to grab eggdrop:develop shortly and try it out. The issue was some poor PID-checking logic that was added a version or two back that broke when 'set pidfile' wasn't present in the config file, as can be the case with a custom config file. There's probably still room to break this though, so please let me know if you encounter any further issues (or if this fixes it for you). Thanks! |
Hi, I'm running eggdrop on arm, until 1.9.5 everything works fine, but when updating to latest stable, I get the following and then the container is crashing:
Unfortunately the log isn't very helpful.
My docker compose:
The text was updated successfully, but these errors were encountered: