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

CLI: support for reload #113

Closed
subnetmarco opened this issue Apr 5, 2015 · 3 comments
Closed

CLI: support for reload #113

subnetmarco opened this issue Apr 5, 2015 · 3 comments
Assignees
@thibaultcha
Milestone
0.2.0

Comments

@subnetmarco
Copy link
Member

Implementing support for nginx reload to reload nginx configuration without downtime. Required to support a specific class of Plugins, like the Nginx Plus plugin: kong reload.

A few notes on the reload functionality: nginx starts a new set of workers reloading the configuration, keeping the same master process alive. Every new incoming request will be forwarded to the new workers, and once the old workers have processed all the remaining requests nginx will automatically shut them down. This means there is no downtime.
@subnetmarco subnetmarco added the cli label Apr 5, 2015
@subnetmarco subnetmarco added this to the 0.2.0 milestone Apr 5, 2015
@thibaultcha
Copy link
Member

Sounds good. Should be pretty quick to do, I'll see next week too

@thibaultcha
Copy link
Member

Is it supposed to reload the nginx.conf or also the kong.yml?

@thibaultcha thibaultcha mentioned this issue Apr 6, 2015
Closed
5 tasks
@subnetmarco
Copy link
Member Author

It would be nice if it could reload kong.yml

@thibaultcha thibaultcha mentioned this issue Apr 6, 2015
Merged
javierguerragiraldez pushed a commit that referenced this issue Sep 3, 2021
@locao
style(consistent-hashing) code and tests cleanup (#113)
9b4ac20
gszr pushed a commit that referenced this issue Oct 26, 2021
@greut
fix: balancer_latency is nil upon failure (#113)
d7e8bf1
gszr pushed a commit that referenced this issue Oct 27, 2021
@greut
fix: balancer_latency is nil upon failure (#113)
ce3619a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thibaultcha thibaultcha

Labels
None yet
Projects
None yet
Milestone
0.2.0
Development

No branches or pull requests
2 participants
@subnetmarco @thibaultcha