Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Updated Pro and Starter architecture to reflect enhanced integration environment provisioning #7369

Merged
merged 9 commits into from
Jun 10, 2020

Conversation

meker12
Copy link
Contributor

@meker12 meker12 commented Jun 8, 2020

Purpose of this pull request

Updates to reflect changes to Integration environment provisioning

  • Updated Starter and Pro architecture diagrams to reflect new Enhanced
    Integration environment provisioning
  • Adds recommendations and caveats for using Pro Staging and Integration
    environments
  • Cleaned Starter environment topic and updated info about Integration
    environment
  • Moved Starter architecture topics to architecture directory and
    updated reference links with new path
  • Added note for existing Cloud customers about requesting upgrade
    to Enhanced Integration environments

Review tasks

  • Successful DevDocs staging build
  • Editorial review
  • Technical review

Affected DevDocs pages

whatsnew
Updated Pro architecture topics to reflect new enhanced Integration environment provisioning and added recommendations for using Pro Staging and Integration environments.

meker12 added 2 commits June 5, 2020 08:28
- Updated Starter and Pro architecture diagrams with new Enhanced
Integration environment provisioning
- Add recommendations and caveats for using Pro Staging and Integration
environments
- Cleaned Starter environment topic and updated info about Integration
environment
- Moved Starter architecture topics to architecture directory and
updated reference links with new path
- Added note for existing Cloud customers about requesting upgrade
to Enhanced Integration environments
@meker12 meker12 changed the title Mae mcloud 5863 integration environment Updated Pro and Starter architecture to reflect enhanced integration environment provisioning Jun 8, 2020
@meker12 meker12 requested a review from hguthrie June 8, 2020 12:32
@meker12 meker12 self-assigned this Jun 8, 2020
@meker12 meker12 added the 2.x label Jun 8, 2020
@meker12 meker12 added Major Update Significant original updates to existing content Internal Dev Differentiates work between community and Magento staff and removed Progress: review labels Jun 8, 2020

{{site.data.var.ece}} currently supports the following services: PHP, MySQL (MariaDB), Elasticsearch (Magento 2.2.x and later), Redis, and RabbitMQ.

Each service runs in its own secure container. Containers are managed together in the project. Some services are built-in, such as the following:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've never been a fan of "It's own". I always think of "getting out of one's own way".

Suggested change
Each service runs in its own secure container. Containers are managed together in the project. Some services are built-in, such as the following:
Each service runs in a separate, secure container. Containers are managed together in the project. Some services are standard, such as the following:

@YPyltiai
Copy link
Contributor

YPyltiai commented Jun 9, 2020

Looks good to me.

@meker12 meker12 requested a review from hguthrie June 9, 2020 18:07
@meker12
Copy link
Contributor Author

meker12 commented Jun 9, 2020

@YPyltiai Can you indicate approval on the PR. Thanks!


You use the following files to configure the software versions that you want to use in your implementation.

- [`.magento.app.yaml`][magento.app.yaml]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This link works just fine. So I suggested this for the broken one above. If you change the link definition, then you'd have to change both links. But the suggestion above should be okay.

Copy link
Contributor

@hguthrie hguthrie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yay!

@meker12
Copy link
Contributor Author

meker12 commented Jun 9, 2020

running tests

@meker12
Copy link
Contributor Author

meker12 commented Jun 10, 2020

running tests

@meker12 meker12 merged commit 583b0ca into master Jun 10, 2020
@ghost
Copy link

ghost commented Jun 10, 2020

Hi @meker12, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

@meker12 meker12 deleted the mae-mcloud-5863-integration-environment branch June 10, 2020 17:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.x Internal Dev Differentiates work between community and Magento staff Major Update Significant original updates to existing content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants