Skip to content

cache:clear fails after upgrading to symfony 3.4.0-BETA4 #25066

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.

Sign up for GitHub

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

Jump to bottom
Closed
phoenixgao opened this issue Nov 21, 2017 · 4 comments
Closed

cache:clear fails after upgrading to symfony 3.4.0-BETA4 #25066

phoenixgao opened this issue Nov 21, 2017 · 4 comments
Labels
Bug Status: Needs Review Status: Waiting feedback Unconfirmed

Comments

@phoenixgao
Copy link

Q A
Bug report? yes
Feature request? no
BC Break report? no
RFC? no
Symfony version 3.4.0-BETA4

I upgraded to symfony 3.4.0-BETA4 by updating composer.json and replacing changes files according to symfony-standard repository.

Everything goes well except the php bin/console cache:clear --no-warmup constantly fails with this error:

 // Clearing the cache for the dev environment with debug true

In Filesystem.php line 181:

  Failed to remove directory "/path/to/my/project/var/cache/de~/ContainerWocmr4b": Undefined variable: lightTrace.

So there is an undefined variable in the debug component.
The actually error is:

"Failed to remove directory "/path/to/my/project/var/cache/de~/ContainerWocmr4b": rmdir(/path/to/my/project/var/cache/de~/ContainerWocmr4b): Directory not empty."

Even I manually rm all the cache folders on cli and still getting this.

btw I'm using Macos 10.13.1 and vagrant nfs shared folder to ubuntu 16.04.3 vbox. command is ran in vbox. php 7.1.13
@nicolas-grekas nicolas-grekas mentioned this issue Nov 21, 2017
Merged
nicolas-grekas added a commit that referenced this issue Nov 21, 2017
@nicolas-grekas
bug #25069 [Debug] Fix undefined variable $lightTrace (nicolas-grekas)
79a64d3 
This PR was merged into the 3.4 branch.

Discussion
----------

[Debug] Fix undefined variable $lightTrace

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

As reported in #25066

Commits
-------

21b2343 [Debug] Fix undefined variable $lightTrace
@nicolas-grekas
Copy link
Member

The undefined variable part is now fixed, see #25069.
For the rm that fails, I already see something similar when using vargant+shared folder.
Would you be able to debug that?

@phoenixgao
Copy link
Author

phoenixgao commented Nov 21, 2017 •
edited
Loading

@nicolas-grekas Thanks!

I just found similar issues:
#19851
#2600

It seems related to the "de~" folder name under var/cache, but before upgrading to 3.4 beta from 3.3.10 this directory was created as "dev" so everything worked fine, so I'm wondering if there is any related symfony changes.

I'll try to debug more on this.

@nicolas-grekas nicolas-grekas mentioned this issue Nov 21, 2017
Merged
@phoenixgao
Copy link
Author

@nicolas-grekas

I pulled #25069 the undefined error is gone now, but the error message is not showed. I got this:

In Filesystem.php line 181:

  Failed to remove directory "/var/www/sites/TNC/kaufmich2/var/cache/de~/Cont
  ainerIiegcfg": .

and the rm issue...

I was not able to debug this in symfony >< code too complex for me, so I found it's also related to
hashicorp/vagrant#8788
and
hashicorp/vagrant#8061

As described at the end of this issue, hashicorp/vagrant#8788
I tried to upgrade my macos to 10.13.2 Beta2 and the problem completely gone.

Since it's actually an apple filesystem bug and 10.13.2 will be release soon (hopefully), I think this issue could be closed
Thanks for the help

@nicolas-grekas
Copy link
Member

OK, thanks for digging

fabpot added a commit that referenced this issue Nov 21, 2017
@fabpot
minor #25078 [FrameworkBundle] Ignore failures when removing the old …
a76489b 
…cache dir (nicolas-grekas)

This PR was merged into the 3.4 branch.

Discussion
----------

[FrameworkBundle] Ignore failures when removing the old cache dir

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #25066
| License       | MIT
| Doc PR        | -

ping @phoenixgao can you please check if this improves the situation?

Commits
-------

3c4d168 [FrameworkBundle] Ignore failures when removing the old cache dir
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Status: Needs Review Status: Waiting feedback Unconfirmed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@nicolas-grekas @phoenixgao @xabbuh @carsonbot