Strange behaviour with haproxy_lb LWRP

[sous-chefs]

Strange behaviour with haproxy_lb LWRP

[remkade]

I actually figured this one out. Because the LWRP only sets node variables and doesn't generate a conf.d style file (HAProxy doesn't support that), you have to call all of your LWRPs first then make sure you run the haproxy::default recipe after all your LWRPs have been run. This means you can't include_recipe 'haproxy' at the top of your recipes either. In order to make it pull in haproxy but no generate the config, do include_recipe 'haproxy::YOURINSTALLMETHOD'.

This reallly should be documented.

[remkade]

I have this same issue, but the log line doesn't fix it for me.

The haproxy_lb just doesn't seem to do anything at all once I've disable the default http servers.

Chef 11.8.2 using chef server.

[thegreenrobot]

Yeah this was frustrating to find. I tried v1.6.2, v1.6.0 and 1.5.0 and it was always the same.

This is on Chef 11.10

[dennybaa]

The LWRP and cookbook implementations are buggy. The resource definitions in this case of course should precede the actual haproxy::default recipe include)
There's lazy attribute evaluation and there are also many more ways to to fix this race condition. The result of a good library cookbook must not rely on the run_list recipes order.
There's nothing to document, it's better the cookbook is fixed.

[kplimack]

@remkade's solution worked for me. Running the LWRPs before running the haproxy (default for me) recipe fixes this. My wrapper worked perfectly in vagrant with chef-solo, went to a real box and it broke here.

[remkade]

@dennybaa From what I understand this is more of a drawback of HAProxy, since you can't currently do a proxy.d/*.conf style include, which is really the way chef is best equipped to do this. Because of that it ought to be documented since there isn't likely to be a fix anytime soon. I'll submit a pull request.

[slyness]

Taking a big crack right now at making this cookbook more functional. I've struggled with this cookbook myself in many configuration scenarios. https://github.com/slyness/haproxy/compare/refactor
Hopefully we'll release a more usable cookbook soon.

Note: default.rb will be a noop recipe soon with other function recipes to supply main functionality but with the intention that it can also be used as a resource cookbook to drive wrapper cookbook custom configuration.

[calebhailey]

@slyness: is this still our top priority for cookbook maintenance? Will you have time for this this week?

[slyness]

@calebhailey This particular change set has definitely fallen off my immediate to do list. But I am willing to get back at it. I'll review the current state of things this week.

[damacus]

I'm going to close this off now as we're working towards a custom resources rewrite.

If #164 doesn't meet your needs we'd love your feedback.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.