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.

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

reverse-proxy-nginx.conf: use upstreams for the proxy_pass directives #7085

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

Conversation

BacLuc
Copy link
Contributor

@BacLuc BacLuc commented Mar 22, 2025

nginx seems to better retry failed upstreams again if they are defined as upstreams.
Without the paid subscription we don't have a health_checks or even new resolves https://docs.nginx.com/nginx/admin-guide/load-balancer/http-health-check/ https://docs.nginx.com/nginx/admin-guide/load-balancer/http-load-balancer/#configuring-http-load-balancing-using-dns

@BacLuc BacLuc requested review from a team, usu and pmattmann March 22, 2025 17:55
@pmattmann pmattmann added the deploy! Creates a feature branch deployment for this PR label Mar 23, 2025
Copy link

github-actions bot commented Mar 23, 2025

Feature branch deployment ready!

Name Link
😎 Deployment https://pr7085.ecamp3.ch/
🔑 Login [email protected] / test
🕒 Last deployed at Sun Mar 23 2025 21:46:11 GMT+0100
🔨 Latest commit 2e2bcceb4d8cc0ae3155de994bc20f166735f8a2
🔍 Latest deploy log https://github.com/ecamp/ecamp3/actions/runs/14022789211/job/39257020139
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy! Creates a feature branch deployment for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants