This repository has been archived by the owner on Nov 19, 2024. It is now read-only.
MCLOUD-6939: Errors occur running magento commands in composer cloud docker installation #8007
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…docker installation
BarnyShergold
suggested changes
Oct 7, 2020
src/cloud/env/variables-build.md
Outdated
| @@ -201,6 +201,19 @@ stage: | |||
| SKIP_SCD: false | |||
| ``` | |||
|
|
|||
| ### `SKIP_COMPOSER_DUMP_AUTOLOAD` | |||
|
|
|||
| Skip executing of `composer dump-autoload` command. | |||
There was a problem hiding this comment.
Suggested change
| Skip executing of `composer dump-autoload` command. | |
| Skip execution of `composer dump-autoload` command. |
There was a problem hiding this comment.
Suggest: Avoid using terms like execute and terminate:
Do not run the composer dump-autoload command.
Why do you need to set this variable? Do you only set it if you are deploying Cloud project using Magento Cloud Docker?
There was a problem hiding this comment.
@meker12 Executing composer dump-autoload command causes an error when running some Magento commands such as bin/magento deploy:mode:set production or bin/magento -n setup:upgrade. These commands can be run on writable systems, for example in the magento-cloud-docker test container. So we added a possibility to skip it.
This is a core issue magento/magento2#23251, but I don't know when it will be fixed.
meker12
suggested changes
Oct 7, 2020
src/cloud/env/variables-build.md
Outdated
| @@ -201,6 +201,19 @@ stage: | |||
| SKIP_SCD: false | |||
| ``` | |||
|
|
|||
| ### `SKIP_COMPOSER_DUMP_AUTOLOAD` | |||
|
|
|||
| Skip executing of `composer dump-autoload` command. | |||
There was a problem hiding this comment.
Suggest: Avoid using terms like execute and terminate:
Do not run the composer dump-autoload command.
Why do you need to set this variable? Do you only set it if you are deploying Cloud project using Magento Cloud Docker?
meker12
added
Internal Dev
…docker installation
m2-community-project
bot
added
Progress: review
and removed
Progress: changes requested
labels
|
Hi @oshmyheliuk, thank you for your contribution! |
bdenham
added a commit
that referenced
this pull request
Nov 4, 2020
* MCLOUD-6939: Errors occur running magento commands in composer cloud docker installation (#8007) * Added variable to skip composer dump-autoload command processing when deploying a Magento Cloud project to a Cloud Docker environment. * MCLOUD-7072: Add CLI command for generating .magento.env.yaml (#8040) * MCLOUD-7072: Add CLI commands for creating and updating .magento.env.yaml * Apply suggestions from code review Co-authored-by: Barny Shergold <[email protected]> Co-authored-by: Margaret Eker <[email protected]> * MCLOUD-6805: ECE-Tools release notes WIP: updating variables * MCLOUD-6805: ECE-Tools release notes First draft * MCLOUD-6805: ECE-Tools release notes Fixed double slash on link * added links to two hot fixes * Update services-mysql.md (#8144) Co-authored-by: Margaret Eker <[email protected]> * Update Magento Cloud Docker instructions for adding custom PHP extensions (#8145) * [Doc] Adding new PHP extensions #155 (#7929) * Clean up instructions for adding PHP extensions - Move extension configuration attribute descriptions and examples to a separate section * Fixed markdown syntax * Fix link definitions * Fix link syntax * Apply suggestions from code review Co-authored-by: Yevhen Miroshnychenko <[email protected]> Co-authored-by: Barny Shergold <[email protected]> Co-authored-by: hguthrie <[email protected]> * MCLOUD-6805: ECE-Tools release notes Updated entry for MCLOUD-7098 * MCLOUD-6805: ECE-Tools release notes Fixed merge conflicts * Added docs for SKIP_COMPOSER_DUMP_AUTOLOAD * Apply suggestions from code review Co-authored-by: Margaret Eker <[email protected]> * MCLOUD-6805: ECE-Tools release notes Added release notes entry for #7191 * Update error-codes information with current info (#8152) * Update error-codes information with current info - Replaced file with error-code.md file generated from the ece-tools package * Fixed error messages * Add source file for generating the error code ref Co-authored-by: Barny Shergold <[email protected]> * Fixed comment syntax on internal note * MCLOUD-6805: ECE-Tools release notes Added release not for MCLOUD-7193 * MCLOUD-6805: ECE-Tools release notes Fixed silly empty bullet * MCLOUD-6805: ECE-Tools release notes Per review, updated additional ece-tools docs changes that are unassociated with the release notes. * MCLOUD-6805: ECE-Tools release notes Per review, removed extra space from yaml code block. * MCLOUD-6805: ECE-Tools release notes Per review, tweaking the wording for writeable Cloud Docker environments. * MCLOUD-6805: ECE-Tools release notes Per review, forgot to clarify the pronoun that. * MCLOUD-6805: ECE-Tools release notes Per review, removed extra space, missed it first time around. * MCLOUD-6805: ECE-Tools release notes Ordered 'new' release note before fixed * MCLOUD-6805: ECE-Tools release notes Added links to documentation. * MCLOUD-6805: ECE-Tools release notes Added links to documentation. * Apply suggestions from code review Co-authored-by: Margaret Eker <[email protected]> Co-authored-by: Oleksandr Shmyheliuk <[email protected]> Co-authored-by: maeker12 <{ID}+{username}@users.noreply.github.com> Co-authored-by: jfrontain <[email protected]> Co-authored-by: Andrii Shevtsov <[email protected]> Co-authored-by: Margaret Eker <[email protected]> Co-authored-by: Yevhen Miroshnychenko <[email protected]> Co-authored-by: Barny Shergold <[email protected]> Co-authored-by: hguthrie <[email protected]>
meker12
added a commit
that referenced
this pull request
Nov 9, 2020
* ECE-Tools 2002.1.3 docs and release notes (#8161) * MCLOUD-6939: Errors occur running magento commands in composer cloud docker installation (#8007) * Added variable to skip composer dump-autoload command processing when deploying a Magento Cloud project to a Cloud Docker environment. * MCLOUD-7072: Add CLI command for generating .magento.env.yaml (#8040) * MCLOUD-7072: Add CLI commands for creating and updating .magento.env.yaml * Apply suggestions from code review Co-authored-by: Barny Shergold <[email protected]> Co-authored-by: Margaret Eker <[email protected]> * MCLOUD-6805: ECE-Tools release notes WIP: updating variables * MCLOUD-6805: ECE-Tools release notes First draft * MCLOUD-6805: ECE-Tools release notes Fixed double slash on link * added links to two hot fixes * Update services-mysql.md (#8051 * Update Magento Cloud Docker instructions for adding custom PHP extensions (#8145) * [Doc] Adding new PHP extensions #155 (#7929) * Clean up instructions for adding PHP extensions - Move extension configuration attribute descriptions and examples to a separate section * Fixed markdown syntax * Fix link definitions * Fix link syntax * Apply suggestions from code review Co-authored-by: Yevhen Miroshnychenko <[email protected]> Co-authored-by: Barny Shergold <[email protected]> Co-authored-by: hguthrie <[email protected]> * MCLOUD-6805: ECE-Tools release notes Updated entry for MCLOUD-7098 * MCLOUD-6805: ECE-Tools release notes Fixed merge conflicts * Added docs for SKIP_COMPOSER_DUMP_AUTOLOAD * Apply suggestions from code review Co-authored-by: Margaret Eker <[email protected]> * MCLOUD-6805: ECE-Tools release notes Added release notes entry for #7191 * Update error-codes information with current info (#8152) * Update error-codes information with current info - Replaced file with error-code.md file generated from the ece-tools package * Fixed error messages * Add source file for generating the error code ref Co-authored-by: Barny Shergold <[email protected]> * Fixed comment syntax on internal note * MCLOUD-6805: ECE-Tools release notes Added release not for MCLOUD-7193 * MCLOUD-6805: ECE-Tools release notes Fixed silly empty bullet * MCLOUD-6805: ECE-Tools release notes Per review, updated additional ece-tools docs changes that are unassociated with the release notes. * MCLOUD-6805: ECE-Tools release notes Per review, removed extra space from yaml code block. * MCLOUD-6805: ECE-Tools release notes Per review, tweaking the wording for writeable Cloud Docker environments. * MCLOUD-6805: ECE-Tools release notes Per review, forgot to clarify the pronoun that. * MCLOUD-6805: ECE-Tools release notes Per review, removed extra space, missed it first time around. * MCLOUD-6805: ECE-Tools release notes Ordered 'new' release note before fixed * MCLOUD-6805: ECE-Tools release notes Added links to documentation. * MCLOUD-6805: ECE-Tools release notes Added links to documentation. * Apply suggestions from code review * Cloud Docker 1.2.0 Release (#8199) * MCLOUD-6430: Xdebug not working on Magento Cloud Docker 1.1 (#7810) * Update src/cloud/docker/docker-development-debug.md * MCLOUD-6898: Add option to customize port for MailHog (#7860) * MCLOUD-6898: Add option to customize port for MailHog * MCLOUD-6660: Add option to disable MailHog * Update src/cloud/docker/docker-config.md * MCLOUD-2789: [Cloud Docker] Add custom ES plugins (#7898) * MCLOUD-2789: [Cloud Docker] Add support for adding custom ES plugins from services.yaml file to the docker-compose.yml file. * MCLOUD-6394: Help with running MFTF test generation commands (#7995) * MCLOUD-6394: Help with running MFTF test generation commands * Added `--with-test` option to `build:compose` option list * Update src/cloud/docker/docker-containers.md - Added test container * Update src/cloud/docker/docker-containers-service.md Added `--with-test` option to `build:compose` option list * Update Docker command help (#8197) * Updated Docker Quick reference - Addedd new options to the magento-docker CLI - Added listing that shows all available options for ece-docker build:compose command - Added intro to Docker Compose section * Updates for Cloud Docker 1.2.0 release (#8198) * MCLOUD-7809 Added instructions for building custom docker-compose.yaml file * Improved instructions for using the init-docker.sh installation script * Updates based on review feedback * Updated Launch developer mode instructions to add step to apply patches before deploying to a Magento Cloud Docker environment Co-authored-by: Oleksandr Shmyheliuk <[email protected]> Co-authored-by: Andrii Shevtsov <[email protected]> Co-authored-by: Margaret Eker <[email protected]> Co-authored-by: Yevhen Miroshnychenko <[email protected]> Co-authored-by: Barny Shergold <[email protected]> Co-authored-by: hguthrie <[email protected]> Co-authored-by: Jeff Matthews <[email protected]> Co-authored-by: Bohdan Korablov <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose of this pull request
Added documentation for new Cloud build variable
SKIP_COMPOSER_DUMP_AUTOLOADto skip composer dump-autoload processing when deploying a Cloud project to the Cloud Docker environment.Affected DevDocs pages
Magento source code
magento/ece-tools#784
whatsnew
Added documentation for new Cloud build variable
SKIP_COMPOSER_DUMP_AUTOLOADto skip composer dump-autoload processing when deploying a Cloud project to the Cloud Docker environment. See Build variables.