Skip to content

Fixes for load balancer issues

Boros Gábor requested to merge smarnach/lb-fixes into master

Created by: smarnach

This PR includes three fixes:

  1. Don't include appservers that have been turned down in the LB configuration. This has caused None appearing as an IP address in the configuration, resulting in haproxy refusing to reload the config.

  2. Convert all domain names to lower case for the LB configuration. I've already manually patched this up on the LB server, but we should do it in the IM as well. Actually, all domain names should be converted to lower case when entered, and only stored in lower case, but this fix makes sure we at least never send upper-case names to the load balancer.

  3. Add a new Huey task for activating an appserver. This takes much longer than it used to, since it now has to run a playbook to reconfigure the LB server, so we can't do it as part of a normal request response anymore.

Merge request reports