-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[DependencyInjection] Container recompile in Symfony 4 is faulty #25177
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
Comments
Does this happen on every command? Only on a specific one? |
It's not specific to commands, same errors are on web too. Reproducer is not easy, I would already provide you one if I could. For now I am just giving out warning that there might be issue with this before full release of sf 4.
|
Would you be able to know if the file is not generated, or if it has been removed? When you don't have the error, does the file exist? If yes, I've no idea what could possibly remove it. |
I will take a snapshot of files in cache folder next time before doing risky activity which usually triggers this (mainly git pull + refresh) and will let you know. |
We are starting to have this problem a lot. Randomly on production, during deployment and plenty in dev environment. Today, I had this issue happen to me five times in sequence. So what I said before that this is always only on first load is no longer true. Error popped up, I did refresh, another error popped up which said different file could not be opened and so on.
Requested file doesn't exist before the error, nor during |
That could mean that when the container is just created, the file is referenced, but never created? |
After recent findings, this is most likely concurrency issue. We found we were running multiple php processes. We killed all of them and run only one. Now it works fine. |
…quests (nicolas-grekas) This PR was merged into the 3.4 branch. Discussion ---------- [HttpKernel] Keep legacy container files for concurrent requests | Q | A | ------------- | --- | Branch? | 3.4 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #25177 | License | MIT | Doc PR | - Because concurrent requests might still be using them, old container files should not be removed immediately, but on a next dump of the container. Commits ------- ee3b6fe [HttpKernel] Keep legacy container files for concurrent requests
We are experiencing regularly this kind of error. It errors out at first load/console run, but on second run everything works ok
The text was updated successfully, but these errors were encountered: