Add info about optimized realpath_cache_settings #7939
Conversation
| - `zip` | ||
| - **Default extensions** are the default extensions available with the PHP application. | ||
| - **Required extensions** are installed automatically and cannot be removed | ||
| - **Optional extensions**–You can install and remove these extensions as needed |
There was a problem hiding this comment.
| - **Optional extensions**–You can install and remove these extensions as needed | |
| - **Optional extensions**–You can install and remove these extensions as needed. |
There was a problem hiding this comment.
Removed these updates and reverted to original headings from the nested list.
| - `yaml` | ||
| - `zip` | ||
| - `pcntl` | ||
| | Default extensions | Required extensions | Optional extensions | |
There was a problem hiding this comment.
Required Extension list has been repeated in the Default Extensions list
Also as the Optional list is much longer that the other, the DE & RE list float in the middle of the table. Can these be aligned to the top?
There was a problem hiding this comment.
-Redid the table to begin with required extensions, followed by default and optional extensions. I also removed the default extensions from the optional list, and updated the Optional descriptions summary to state that the default extensions can be disabled.-
There was a problem hiding this comment.
I'm afraid the list is still incorrect.
The following extensions are required but appear in the optional list -
bcmath
gd
intl
pdo_mysql
soap
xsl
zip
(https://devdocs.magento.com/guides/v2.4/install-gde/system-requirements-tech.html)
There was a problem hiding this comment.
The sockets extension also became required in Magento 2.4.0 and higher btw. Should also be included in https://devdocs.magento.com/guides/v2.4/install-gde/system-requirements-tech.html in my opinion!
There was a problem hiding this comment.
@hostep I updated the PHP settings topic to add a note about the ext-sockets extension that was in the 2.3 version of the topic:
If you install Magento via cloning from the [GitHub](https://github.com/magento/magento2) repository, then make sure you have the [ext-sockets](https://github.com/php-amqplib/php-amqplib/blob/master/CHANGELOG.md#281---2018-11-13) extension installed on your instance.
In the php-settings topic, the list of required extensions is generated in this file: https://github.com/magento/devdocs/blob/master/src/_includes/install/php-extensions-template.md, but ext-sockets is not included. I think we should open a separate issue to address this, and request assistance from @dshevtsov.
There was a problem hiding this comment.
I'm afraid the list is still incorrect.
The following extensions are required but appear in the optional list -
bcmath
gd
intl
pdo_mysql
soap
xsl
zip(https://devdocs.magento.com/guides/v2.4/install-gde/system-requirements-tech.html)
I restored the original content from this topic. I realized that I was conflating enable and disable extensions with install and uninstall. In the .magento.app.yaml file, which is the subject of this topic, you only enable and disable extensions.
There was a problem hiding this comment.
@meker12: we already kind-of had an issue for this, but it was closed unfortunately: #7568 (comment), should I create a new issue for this? I know colleagues of yours are already aware of the issue for many months (internal ticket DOC-80), but maybe we need an official public ticket to track this, otherwise it might be forgotten...
There was a problem hiding this comment.
@hostep Thanks for the context about the ext-sockets extension. It's useful that the current guide now includes a specific mention of the ext-sockets extension, and why you need it. Will follow up on our internal ticket to understand the best information to provide about this extension.
There was a problem hiding this comment.
@hostep the DOC-80 is closed. It was mostly focused on the Packages topics. As it appeared, it's impossible to get accurate information about PHP versions from the Composer files. I hope that it is still possible for the list of extensions. Please open an issue for this with your recent suggestions (#7568, #7398) and assign it to me.
There was a problem hiding this comment.
@dshevtsov, done: #7960 (but I can't assign you)
Co-authored-by: Barny Shergold <[email protected]>
m2-community-project
bot
added
Progress: review
and removed
Progress: changes requested
labels
| - `yaml` | ||
| - `zip` | ||
| - `pcntl` | ||
| | Default extensions | Required extensions | Optional extensions | |
There was a problem hiding this comment.
I'm afraid the list is still incorrect.
The following extensions are required but appear in the optional list -
bcmath
gd
intl
pdo_mysql
soap
xsl
zip
(https://devdocs.magento.com/guides/v2.4/install-gde/system-requirements-tech.html)
| @@ -73,14 +73,23 @@ PHP Warning: date(): It is not safe to rely on the system's timezone settings. | |||
| - Installing and updating Magento components from Magento Marketplace, `2G` | |||
There was a problem hiding this comment.
This can probably be removed now that the web set up wizard was removed in Magento 2.4.0?
| - `zip` | ||
| - **Required extensions** install automatically and cannot be removed. | ||
| - **Default extensions** install by automatically. You can disable them as needed. | ||
| - **Optional extensions**–You can add or disable these extensions as needed. |
There was a problem hiding this comment.
This line uses an emdash and the others don't. Maybe can match with something like:
| - **Optional extensions**–You can add or disable these extensions as needed. | |
| - **Optional extensions** are not installed. You can install these extensions as needed. |
There was a problem hiding this comment.
Removed these headings
| - `sysvshm` | ||
| - `opcache` | ||
| - `zip` | ||
| - **Required extensions** install automatically and cannot be removed. |
There was a problem hiding this comment.
So, after thinking about the following two list items, you say "add or disable" a couple of times, which seems a little weird. Add = install? And enable = ?add?. Would it be correct to say the required extensions "install and enable" automatically?
Also, you say you cannot remove it, but can you disable it?
There was a problem hiding this comment.
It's confusing because the runtime section has two sections: extensions: and disabled:extensions
extensions:
- sockets
- sodium
- ssh2
disabled_extensions:
- bcmath
- bz2
- calendar
- exif
and the headings in the original topic for each section were:
- default extensions
- Extensions that are installed and cannot be uninstalled
- Extensions that can be installed and uninstalled as needed
It's not clear to me what is meant by install and uninstalled -- and how that relates to the runtime section.
I think the confusion is that with extensions, you can install them, and you can enable or disable them after they are installed.
Andrii also pointed out to me the default .magento.app.yaml in magento-cloud template:
# Enable extensions required by Magento 2
runtime:
extensions:
- xsl
- json
- newrelic
- sodium
| ```bash | ||
| To increase the PHP memory limit, add the following setting to the php.ini file: | ||
|
|
||
| ```php | ||
| memory_limit = 756M |
There was a problem hiding this comment.
Our support site is recommending something different. Is 756 the minimum? I know I have seen different values for this, and mine is set to 2G.
https://support.magento.com/hc/en-us/articles/360034599631-PHP-settings-errors
| - `pcntl` | ||
| | Default extensions | Required extensions | Optional extensions | | ||
| |--------------------|---------------------|---------------------| | ||
| | `ctype`<br> `curl`<br> `date`<br> `dom`<br> `fileinfo`<br> `filter`<br> `ftp`<br> `hash`<br> `iconv`<br> `json`<br> `mbstring`<br> `mysqlnd`<br> `openssl`<br> `pcre`<br> `pdo`<br> `pdo_sqlite`<br> `phar`<br> `posix`<br> `readline`<br> `session`<br> `sqlite3`<br> `tokenizer`<br> `xml`<br> `xmlreader`<br> `xmlwriter`<br> | `ctype`<br> `curl`<br> `date`<br> `dom`<br> `fileinfo`<br> `filter`<br> `ftp`<br> `hash`<br> `iconv`<br> `json`<br> `mbstring`<br> `mysqlnd`<br> `openssl`<br> `pcre`<br> `pdo`<br> `pdo_sqlite`<br> `phar`<br> `posix`<br> `readline`<br> `session`<br> `sqlite3`<br> `tokenizer`<br> `xml`<br> `xmlreader`<br> `xmlwriter`<br> | `bcmath`<br> `bz2`<br> `calendar`<br> `exif`<br> `gd`<br> `geoip`<br> `gettext`<br> `gmp`<br> `igbinary`<br> `imagick`<br> `imap`<br> `intl`<br> `ioncube`<br> `ldap`<br> `mailparse`<br> `mcrypt`<br> `msgpack`<br> `mysqli`<br> `oauth`<br> `opcache`<br> `pdo_mysql`<br> `propro`<br> `pspell`<br> `raphf`<br> `recode`<br> `redis`<br> `shmop`<br> `soap`<br> `sockets`<br> `sodium`<br> `ssh2`<br> `sysvmsg`<br> `sysvsem`<br> `sysvshm`<br> `tidy`<br> `xdebug`<br> `xmlrpc`<br> `xsl`<br> `yaml`<br> `zip`<br> `pcntl`<br> | |
There was a problem hiding this comment.
I think it is best to clarify here. I don't know why any extension from the Required column would be duplicated anywhere else because you note that they cannot be removed.
There was a problem hiding this comment.
Restored original column headings. My proposed updates didn't make sense.
m2-community-project
bot
added
Progress: changes requested
and removed
Progress: review
labels
…b.com:magento/devdocs into mae-mcloud-6865-update-php-customization-info
- Correct recommended memory limit and realpath_cache values - Corrected table with supported PHP extensions per review feedback
Co-authored-by: hguthrie <[email protected]>
Co-authored-by: Barny Shergold <[email protected]>
meker12
added
the
Technical
Co-authored-by: Jeff Matthews <[email protected]>
m2-community-project
bot
added
Progress: approved
and removed
Progress: changes requested
labels
| @@ -41,18 +41,25 @@ Magento requires a set of extensions to be installed: | |||
| <!--{% assign packages = site.data.codebase.v2_4.open-source.composer_lock.packages %}--> | |||
| {% include install/php-extensions-template.md %} | |||
|
|
|||
| In the command line, type: | |||
| {:.bs-callout-warning} | |||
| If you install Magento by cloning the [magento/magento2](https://github.com/magento/magento2) GitHub repository, you must have the [ext-sockets](https://github.com/php-amqplib/php-amqplib/blob/master/CHANGELOG.md#281---2018-11-13) extension installed on your system. | |||
There was a problem hiding this comment.
This is incorrect information for Magento 2.4.0. It's correct for Magento 2.3.x, but not for 2.4.x. The sockets plugin is always needed for all installation scenario's, not only when you clone Magento's repository.
There was a problem hiding this comment.
This seems to revert #7398, which is not correct.
…b.com:magento/devdocs into mae-mcloud-6865-update-php-customization-info
|
running tests |
|
Hi @meker12, thank you for your contribution! |
Purpose of this pull request
Add information from Magento Performance team about configuring the following recommended settings for PHP
realpath_cachesettings:Improve readability of PHP plugin listing in the PHP configuration section of the .magento.app.yaml, and add link to "Customize PHP.INI" topic.
Affected DevDocs pages