-
-
Notifications
You must be signed in to change notification settings - Fork 156
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
Cronjob causes many console.CRITICAL errors in log file #6907
Comments
@LIVID-Media What is the value of @Toflar /cc |
I can hardly imagine this error did not happen before. In 5.2 we ran the commands every minute and did not supervise whether they were already running. Now we do. Or in other words, if you now have too many connections, you should've had this before as well :) |
Have the visitors increased? This could also be the reason, though Contao 5 having lots of message workers hogging your connections is not ideal of course 🙈 . Unfortunately a low number of |
Maybe in 5.2 there were just less messages, was it used in prod? If you run We can of course reduce the defaults there. 10 is a bit high. But this has been the case in 5.2 too, so I don't think there's anything wrong in the supervision introduced in 5.3. |
I've a Contao 5.2.8 test installation on the same server with the same setup. When I call In 5.3 it's getting more and more during the cron job until the connection is lost and the site is no longer accessible. In my local system (with max_user_connections = 0), the problem also occurs. I'd already deactivated all extensions. |
@Toflar Is there an easy way to disable the supervisor without disabling the background workers, so @LIVID-Media can test if the problem is really caused by the supervisor? |
I did some local tests. When I execute But when I execute So I can confirm the issue. |
@LIVID-Media While we debug the problem, you can disable the background workers by adding this in your contao:
messenger:
workers: [] |
Careful, this will disable email sending! |
No, it won't. I have been using this for some time on contao.org and the emails are sent without any problems. Not asynchronously, of course. |
Ah sorry, did not know that. Then yes, that's the way to go. |
@leofeyer Thanks for the information. This solved the problem for now. |
My bad, in my local setup the issue was actually the lack of support for Windows from |
@LIVID-Media is there a possibility to gain test access to your server environment? |
@fritzmg Sure. Please tell me what you need. |
I can confirm the issue with the credentials provided by @LIVID-Media. It's a Webgo shared hosting environment, where the
Thus This will likely to be the case on all of Webgo's shared hosting environments (I was also able confirm this in the Webgo hostings of two of our customers). @LIVID-Media as a first step you could ask Webgo's support to make the There could of course also be other shared hosting environments where |
With Toflar/cronjob-supervisor#2 we can probably fix this as We have 2 places where we can check it: a) Check it at Container runtime. This is hard because the Supervisor instance is not created there at the moment so it means we need to do quite some work. And then the question is also what to do if the check fails. Case a): When building the container, we could disable the entire worker configuration - > but where would we log? Case b): In case of doing it in the |
Even though there may soon be a solution. Nevertheless, here is the answer from WebGo in short: |
As discussed in the Contao call, we want to set |
Description ----------- Fixes #6907. Ready for review but needs an update of the `composer.json` entries before merging. Requires Toflar/cronjob-supervisor#2 to be released as 2.0.0 which I'll only do once this PR is approved concept-wise. Commits ------- b63f645 Update to toflar/cronjob-supervisor v2 and disable workers if not sup… 7b81d99 Use static helpers c1e55c0 Update to stable release
Description ----------- Fixes contao/contao#6907. Ready for review but needs an update of the `composer.json` entries before merging. Requires Toflar/cronjob-supervisor#2 to be released as 2.0.0 which I'll only do once this PR is approved concept-wise. Commits ------- b63f6452 Update to toflar/cronjob-supervisor v2 and disable workers if not sup… 7b81d990 Use static helpers c1e55c04 Update to stable release
Affected version(s)
5.3.0
Description
Since the update from 5.2.8 to 5.3 there are a lot of critical errors in the log file after cronjob is executed. The main problem is "more than 'max_user_connections' active connections".
Here's an excerpt from the log file:
The error didn't occur in Contao 5.2. Maybe it has something to do with the new SuperviseWorkersCommand script?!
The text was updated successfully, but these errors were encountered: