-
Notifications
You must be signed in to change notification settings - Fork 6
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.
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
Build warnings when deploying #169
Comments
Need to follow https://docs.cloudfoundry.org/buildpacks/nginx/index.html#templating
|
can validate with |
Example conf files:
cloudfoundry/staticfile-buildpack#144 (comment) are all true but:
➡️ should at worst cause app staging failures in case of new versions of the buildpack
➡️ As far as we know apply only to the case in which we also have static content (as long as we use it as a pure reverse proxy you should be good)
➡️ this is how we're using it |
I’m thinking we need to accept this risk for now, since we’d need to completely re-write the proxy application and we don’t have that expertise, and might be riskier than just leaving it as-is until more people address this and we can look at their implementation. Changes that fail with buildpack errors: https://github.com/fecgov/fec-proxy/pull/174/files
|
@lbeaufort From what I can gage, we use cloud.gov's staticfile_buildpack v1.5.5. That version utilizes nginx version 1.17.9. The latest version of staticfile_buildpack v1.5.7 is using nginx version 1.17.10. As long as cloud.gov does not upgrade their staticfile_buildpack to a future version that is incompatible with the version of nginx, I see this risk as low. We should be cognizant of when cloud.gov upgrades their staticfile_buildpack to ensure that there are no issues with the upgraded nginx version that is used. |
Hi folks - so 1.5.7 is the current static-buildpack level - you can track new releases here : https://github.com/cloudfoundry/staticfile-buildpack/releases From our cloud-foundry standpoint - new releases of buildpacks come in new releases of cf-deployment: https://github.com/cloudfoundry/cf-deployment/blob/master/cf-deployment.yml - look down under the |
Closing in favor of #200 - the version warnings no longer appear, but we should move over to the NGINX buildpack. |
What we're after: This work has the potential to be problematic if/when something happens to the proxy.
Completion criteria:
Build warnings when deploying:
From the above error message,
More context on why using staticfile buildpack and custom nginx.conf (like we do): cloudfoundry/staticfile-buildpack#144 (comment)
The text was updated successfully, but these errors were encountered: