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

Use internal network to fix uwsgi load balancer #403

Merged
merged 3 commits into from
May 15, 2024
Merged

Use internal network to fix uwsgi load balancer #403

merged 3 commits into from
May 15, 2024

Conversation

Xpirix
Copy link
Collaborator

@Xpirix Xpirix commented May 15, 2024 •
edited
Loading

This is an improvement for the uwsgi load balancer

  • Use an internal network to make the service distribute the load over the 2 uwsgi containers
  • Set uwsgi workers to 8
  • Increase harakiri to 100

Please find below a screenshot of the stats of uwsgi containers in a bench test (locally):

image

@Xpirix Xpirix requested a review from dimasciput May 15, 2024 07:04
timlinux
timlinux reviewed May 15, 2024
View reviewed changes
dockerize/nginx.conf Outdated
if ($http_user_agent !~* "Mozilla/5.0 QGIS") {
return 403 "Forbidden: Please use QGIS to download .zip files.";
}
}
Copy link
Member

Choose a reason for hiding this comment

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

Extra bracket?

@timlinux
Copy link
Member

LGTM

@Xpirix Xpirix merged commit 47ddc04 into qgis:master May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@timlinux timlinux timlinux left review comments

@dimasciput dimasciput Awaiting requested review from dimasciput

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Xpirix @timlinux