Skip to content

updated the service name in inital docker compose file due to configu…#437

Merged
ezodude merged 1 commit intomasterfrom
update-wordpress-tutorial-docs
Mar 30, 2021
Merged

updated the service name in inital docker compose file due to configu…#437
ezodude merged 1 commit intomasterfrom
update-wordpress-tutorial-docs

Conversation

@Alhinde93
Copy link
Contributor

…ration conflicts in latest wordpress image

…ration conflicts in latest wordpress image
@Alhinde93
Copy link
Contributor Author

Alhinde93 commented Mar 30, 2021

When running through the getting started documentation, after completing the initial Kubernetes deployment, i was getting database connection error when port forwarding the service. After looking at the Wordpress configuration, there were two pieces of configuration present. This is due to the default Kubernetes env variables being named after the service.
The latest image will check to see if there are any environment variables present with the Wordpress prefix and generate a new config file.
entrypoint config here
I have renamed the service to avoid this, can you please review?

@Alhinde93 Alhinde93 requested review from marcinc and myjaffoo March 30, 2021 14:29
@ezodude ezodude merged commit bec7ddd into master Mar 30, 2021
@ezodude ezodude deleted the update-wordpress-tutorial-docs branch March 30, 2021 15:57
marcinc pushed a commit that referenced this pull request Apr 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants