Skip to content
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

wp-content/uploads is not writable in 4.9.8 image #322

Closed
tofumatt opened this issue Aug 3, 2018 · 6 comments
Closed

wp-content/uploads is not writable in 4.9.8 image #322

tofumatt opened this issue Aug 3, 2018 · 6 comments
Labels
question Usability question, not directly related to an error with the image

Comments

@tofumatt
Copy link

tofumatt commented Aug 3, 2018

We're encountering issues over at https://github.com/wordpress/gutenberg with the latest wordpress image (4.9.8). See: WordPress/gutenberg#8418

We can't run wp-cli core update because wp-content/uploads is not writable. This line errors: https://github.com/WordPress/gutenberg/blob/master/bin/install-wordpress.sh#L52 with the bug in the above issue.

I don't know what caused this change because I'm finding it hard to track how/where this image is built from, but changing back to 4.9.7 image (changing the image we use from wordpress to wordpress:4.9.7 in our docker-compose.yml) "fixes" the issue.

Removing that update command things seem to also run okay, but we should be able to run it. I think eventually anything else that interacts with that folder will cause errors.

Was a permission changed or something?

@tofumatt
Copy link
Author

tofumatt commented Aug 3, 2018

It's possible this isn't your fault actually, I just have no idea what's up. It seems that WP CLI can't update without an error. Running wp-cli core update against the wordpress image runs into errors.

@tofumatt
Copy link
Author

tofumatt commented Aug 3, 2018

Turns out this is a permissions issue. Adding -u 33 to our wp-cli commands (using the wordpress:cli image) makes it work, but I wonder why that's needed at all.

I'll leave it to you if you want to leave this open, as I think requiring the -u 33 argument is quite weird, but it seems this isn't exactly the issue I thought it was...

@wglambert wglambert added the question Usability question, not directly related to an error with the image label Aug 3, 2018
@wglambert
Copy link

Discussion on issue here WordPress/gutenberg#8427

@tianon
Copy link
Member

tianon commented Aug 3, 2018

I think this is actually a duplicate of #256. 👍

@tofumatt
Copy link
Author

tofumatt commented Aug 7, 2018

I think this is actually a duplicate of #256. 👍

Sounds right. I get why resolving this one is tough and the solutions in #256 seem okay to me... Maybe to resolve this the docs could at least mention the solutions in #256? 🤷‍♂️

@tianon
Copy link
Member

tianon commented Aug 13, 2018

Good idea; added a little blurb over in docker-library/docs#1284. 👍

@tianon tianon closed this as completed Aug 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Usability question, not directly related to an error with the image
Projects
None yet
Development

No branches or pull requests

3 participants