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

Reload HAProxy configuration without pod restart #33

Open
starteleport opened this issue Oct 7, 2021 · 1 comment
Open

Reload HAProxy configuration without pod restart #33

starteleport opened this issue Oct 7, 2021 · 1 comment
Labels
question Further information is requested

Comments

@starteleport
Copy link

Hi!

I'm using Fluxv2 to deploy haproxy-ingress chart to my cluster. I've noticed that each time I change values that are destined for ConfigMap I have ingress controller pods recreated. AFAIK Flux just issues helm upgrade on values change, so I think it's not related to Flux itself, hence posting here.

I was wondering how could I apply HAProxy config changes without pod recreation? For example, I want to enable logs and have new configuration reloaded by HAProxy itself.

Thanks.

@starteleport starteleport changed the title Reload HAProxy configuration without pod restart with Flux Reload HAProxy configuration without pod restart Oct 7, 2021
@jcmoraisjr
Copy link
Member

Hi, it depends on where the configuration changes. For example if you change how logs are configured, you'll instruct helm to configure a distinct pod template (logging is configured as a sidecar container), and Kubernetes will reflect this reissuing new pods. Please send another examples of config change that restarts the controller, so we can verify if this is the expected behavior.

@jcmoraisjr jcmoraisjr added the question Further information is requested label Oct 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants