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

perf(scripts): improve load balancing #2165

Merged
merged 10 commits into from
Oct 14, 2024
Merged

Conversation

mabw-rte
Copy link
Contributor

@mabw-rte mabw-rte commented Oct 1, 2024

Context:
@sylvlecl reported that when gunicorn, is wrapped over an ASGI, it does not perform the load balancing as it should.

Solution:
Mount the load balancing to nginx

@mabw-rte mabw-rte self-assigned this Oct 1, 2024
@mabw-rte mabw-rte force-pushed the feature/745-load-balancing branch 4 times, most recently from cdee50d to a969ef7 Compare October 2, 2024 14:27
Copy link
Contributor

@MartinBelthle MartinBelthle left a comment

Choose a reason for hiding this comment

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

I'll let Sylvain handle this review as he knows the subject better

.gitignore Outdated Show resolved Hide resolved
scripts/start.sh Outdated Show resolved Hide resolved
scripts/start.sh Show resolved Hide resolved
@mabw-rte mabw-rte force-pushed the feature/745-load-balancing branch 4 times, most recently from 68f035d to 2aae23d Compare October 7, 2024 09:41
@pull-request-size pull-request-size bot added size/M and removed size/S labels Oct 7, 2024
@mabw-rte mabw-rte force-pushed the feature/745-load-balancing branch 4 times, most recently from 92ff838 to 49f0157 Compare October 9, 2024 12:09
@MartinBelthle MartinBelthle force-pushed the feature/745-load-balancing branch from 49f0157 to 42ecf7f Compare October 9, 2024 12:35
docker-compose.yml Outdated Show resolved Hide resolved
@mabw-rte mabw-rte force-pushed the feature/745-load-balancing branch from 295da93 to c96bf80 Compare October 10, 2024 15:26
@pull-request-size pull-request-size bot added size/S and removed size/M labels Oct 10, 2024
@mabw-rte mabw-rte force-pushed the feature/745-load-balancing branch 2 times, most recently from f869bfc to 7062c33 Compare October 11, 2024 08:16
@mabw-rte mabw-rte force-pushed the feature/745-load-balancing branch from 7062c33 to 5aae1a6 Compare October 11, 2024 15:14
@pull-request-size pull-request-size bot added size/M and removed size/S labels Oct 14, 2024
@sylvlecl sylvlecl merged commit 610c8d7 into dev Oct 14, 2024
8 of 9 checks passed
@sylvlecl sylvlecl deleted the feature/745-load-balancing branch October 14, 2024 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants