Limit memory usage about docker-php-apache-base HOT 9 CLOSED

Hi @benjaminpick, sorry for late reply...
Thank you for your feedback. For development purpose we prefer to have unlimited memory at PHP level. As you already understood memory limit should be set at container level.

from docker-php-apache-base.

from docker-php-apache-base.

Thank you @benjaminpick, I've got your point but I think that the real solution is to have a PHP ini config file loaded from the outside of the container.
In an ideal world users of this container should be able to specify their preferred PHP ini configuration. You've got a problem with memory_limit but another user could experience other issues which requires to adjust other ini settings. Isn't it?
What do you think? Would you like to propose a PR to achieve this?

Let us know.

from docker-php-apache-base.

Well, yes of course.

But loading a file from outside seems to defeat the containering
concept. Maybe adding it as a variable?

Am 10.10.2016 um 17:53 schrieb Manuele Menozzi:

Thank you @benjaminpick https://github.com/benjaminpick, I've got your
point but I think that the real solution is to have a PHP ini config
file loaded from the outside of the container.
In an ideal world users of this container should be able to specify
their preferred PHP ini configuration. You've got a problem with
memory_limit but another user could experience other issues which
requires to adjust other ini settings. Isn't it?
What do you think? Would you like to propose a PR to achieve this?

Let us know.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#2 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAO7PitV2_7GMjvFMaSn3PB1-lfSS7ZZks5qyl9zgaJpZM4KL08X.

from docker-php-apache-base.

Yes it's true, it breaks the containering concept but with a single environment variable you can only set a single ini setting (for example memory_limit). What about if someone wants to change other ini settings (error_reporting, display_errors, ecc...)?

from docker-php-apache-base.

That's what I thought at first as well - but YAML supports multi-line
text. It should work with docker-compose, but my tests failed ... let's
see if SO can shed some light on this:

http://stackoverflow.com/questions/39963838/how-can-i-pass-a-multi-line-variable-to-a-docker-container

Am 10.10.2016 um 18:44 schrieb Manuele Menozzi:

Yes it's true, it breaks the containering concept but with a single
environment variable you can only set a single ini setting (for example
memory_limit). What about if someone wants to change other ini settings
(error_reporting, display_errors, ecc...)?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#2 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAO7PuQb9gBgK7ObzgXDWHTEEjY42teGks5qymtzgaJpZM4KL08X.

from docker-php-apache-base.

Ok, let us know! Thank you!

from docker-php-apache-base.

The SO hinted me that multi-line variables do work, I only forgot the
quotes in the shell. So I will send a pull request for this later ...

Am 11.10.2016 um 08:33 schrieb Manuele Menozzi:

Ok, let us know! Thank you!

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#2 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAO7PuNikrD8ImUN0_jM1OZcsk1JQkMiks5qyy2sgaJpZM4KL08X.

from docker-php-apache-base.

Thank you!

from docker-php-apache-base.