[TASK] Updated user.ini according to Magento DevDocs

[lewisvoncken]

Description

The Magento DevDocs recommend to set the memory_limit to 1G or at least 2G for debugging.
http://devdocs.magento.com/guides/v2.2/install-gde/trouble/php/tshoot_php-set.html

Fixed Issues (if relevant)

1. User.ini files specify 768M - Docs recommend at least 1G #11322: User.ini files specify 768M - Docs recommend at least 1G

Contribution checklist

• Pull request has a meaningful description of its purpose
• All commits are accompanied by meaningful commit messages
• All new or changed code is covered with unit/integration tests (if applicable)
• All automated tests passed successfully (all builds on Travis CI are green)

[orlangur]

Minimum required memory limit can be found here: http://devdocs.magento.com/guides/v2.2/install-gde/prereq/php-settings.html
And here in the code: https://github.com/magento/magento2/blob/2.1/setup/src/Magento/Setup/Model/PhpReadinessCheck.php#L189

Troubleshooting doc is wrong about 1Gb.

Please always start your contributions from default repo branch, 2.2-develop. You can change target branch and create a new branch based on 2.2-develop in this PR.

According to my understanding all occurrences of 768M in code must be replaced with recommended 756M

[lewisvoncken]

@orlangur thank you for your feedback I immediately applied your feedback. Next time I will create a pull request first for the base branch

[orlangur]

Great! Thanks for collaboration.

Let me clarify the thing regarding target branch. You need to create a clean branch based on freshest 2.2-develop and apply similar changes, then force push to the same branch you used to create PR and then change target branch via GitHub UI.

All occurrences may be found with Ctrl+Shift+F in PhpStorm or via grep in CLI. Besides changes you made there are also in .htaccess.sample, nginx.conf.sample.

Here we need to change condition also:

if ($memoryLimit != -1 && $this->getMemoryInBytes($memoryLimit) < 768 * 1024 * 1024) {
                @ini_set('memory_limit', '768M');
            }

Please make changes as a single clean commit. If you need any assistance with git to achieve what I described, just let me know.

[magento-cicd2]

All committers have signed the CLA.

[lewisvoncken]

@orlangur I think it is correct now. Thanks for the feedback and quick replies! If it is accepted I will also backport it to 2.1 and forward for 2.3

[orlangur]

@lewisvoncken almost correct :) 768 * 1024 * 1024 here we need 756 too (please amend commit, as always). Besides that looks perfect to me 👍

[lewisvoncken]

@orlangur I searched for 768M but I have updated it in the same commit to keep the pull request clean

[orlangur]

Thanks, I mentioned this condition in "Here we need to change condition also:", remarked it accidentally.

[LilyBergonzatOld]

Just found out that Magento was recommending 756M as PHP memory limit. Can I ask you why is that? Why not 768? 756 isn't a power of 2, so it really seems weird.

[orlangur]

@YannBergonzat,

Our detailed recommendations are:
Compiling code or deploying static assets, 756M
Installing and updating Magento components from Magento Marketplace, 2G
Testing, 2G

Looks like just an empirical number as to me. Probably was 256M and then increased until it was enough :)