Skip to content
This repository has been archived by the owner on Mar 24, 2022. It is now read-only.

Fix error in nginx configuration #26

Merged
merged 1 commit into from
Oct 19, 2020

Conversation

koyan
Copy link
Contributor

@koyan koyan commented Oct 19, 2020

When trying to deploy the helm chart phpfpm
It was failing with:

nginx 11:21:11.85 ERROR ==> Custom server blocks files were found inside '/bitnami/nginx/conf/vhosts'. This configuration is not supported anymore. Please mount your custom server blocks config files at '/opt/bitnami/nginx/conf/server_blocks' instead.

The fix is already applied (and taken from): https://github.com/bitnami/bitnami-docker-php-fpm/pull/124/files

It was failing with: 
```
nginx 11:21:11.85 ERROR ==> Custom server blocks files were found inside '/bitnami/nginx/conf/vhosts'. This configuration is not supported anymore. Please mount your custom server blocks config files at '/opt/bitnami/nginx/conf/server_blocks' instead.
```
Copy link
Contributor

@juan131 juan131 left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@juan131 juan131 merged commit dd84dd8 into vmware-archive:master Oct 19, 2020
@koyan
Copy link
Contributor Author

koyan commented Oct 19, 2020

@juan131 I was too hasty to submit it.
While it fixes the original problem, and the pod does not get stuck in a crashLoop, the whole application does not deliver anymore the php. My knowlege of nginx configuration is not good, so I don't know else should be fixed so that the nginx points to the right place.
(when you open the app on the given ip on the browser, it points to the default nginx index.html. If you try to point it to the correct php file, it returns a "not found")

@juan131
Copy link
Contributor

juan131 commented Oct 20, 2020

@koyan please the changes I did at #28
That should fix the issue, thanks for reporting it!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants