Skip to content
This repository has been archived by the owner on Feb 13, 2023. It is now read-only.

PHP memory limit config value not respected for FPM on Ubuntu #441

Closed
kevinquillen opened this issue Feb 16, 2016 · 8 comments
Closed

PHP memory limit config value not respected for FPM on Ubuntu #441

kevinquillen opened this issue Feb 16, 2016 · 8 comments
Milestone

Comments

@kevinquillen
Copy link

In my config.yml, I have 256M set for memory, but Drupal reports it as 128M.

I am doing this with a fresh install of 2.2.1 of DrupalVM.

@kevinquillen
Copy link
Author

I checked the PHP role and its default is also 256M. Is it possible the PHP FPM additions has caused a change?

@oxyc
Copy link
Collaborator

oxyc commented Feb 16, 2016

I checked my boxes and I can confirm I have the same issue. It's filed upstream already geerlingguy/ansible-role-php#53

@geerlingguy
Copy link
Owner

It's because of Debian/Ubuntu's pathing insanity. I do need to fix this sometime, but I've been going through all my roles and updating them for Galaxy 2.0 and clearing out some cobwebs... so the PHP role is a little later in that list. Help is welcome :D

@geerlingguy geerlingguy changed the title PHP memory limit config value not respected PHP memory limit config value not respected for FPM on Ubuntu Feb 16, 2016
@geerlingguy geerlingguy added this to the 2.3.0 milestone Feb 20, 2016
@geerlingguy
Copy link
Owner

This is the last issue I have on my chopping block for the next minor release of Drupal VM, so I hope to get to it upstream this weekend :)

@oxyc
Copy link
Collaborator

oxyc commented Feb 20, 2016

I have a thoroughly tested PR for it now geerlingguy/ansible-role-php#83 :)

@geerlingguy
Copy link
Owner

Thanks @oxyc - if you haven't already, I'll work more on getting it to green in an hour or two when I'm back from errands, or tonight after finishing out the day's work. It's a beautiful day in STL so doing lots of outdoor things while it's not freezing cold :)

@geerlingguy
Copy link
Owner

This should now be fixed upstream; I just tagged 1.9.1 of the PHP role, and am testing with my local copy of Drupal VM. Will close this once fix is confirmed.

@geerlingguy
Copy link
Owner

Tested and fixed. Found a new bug that resulted in the xdebug role, but fixed that as well, and just pushed a new tag to Galaxy. I'll be working on polishing up the 2.3.0 release soon.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants