Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update redis-service-configuration, add clarification note regarding ece-tools and port. #122

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update redis-service-configuration, add clarification note regarding ece-tools and port. #122

wants to merge 1 commit into from

Commits on Jun 13, 2024

  1. Update redis-service-configuration, add clarification note regarding … 

    …ece-tools and port.

When Platform Team (P.sh) add new redis-session service to an environment, they are not able to guarantee the port where redis-session is listening will be the same for all environments in a project.
If we "hardcode" the redis-session port in the '.magento.env.yaml' config file we could provoque an error in some other environments of the project, if the redis-session service is not listening at that port.
In this "Note", we clarify that only is needed to add the "redis.port" configuration section, if "ece-tools" are not able to get the port from RELATIONSHIPS services info environment variable (provided by Platform Team into the systems).
    @perezalo
    perezalo authored Jun 13, 2024
    Configuration menu
    Copy the full SHA
    1146786 View commit details
    Browse the repository at this point in the history