Merge lp:~niedbalski/charm-helpers/fix-1697002 into lp:charm-helpers

Proposed by Jorge Niedbalski
Status: Merged
Merged at revision: 753
Proposed branch: lp:~niedbalski/charm-helpers/fix-1697002
Merge into: lp:charm-helpers
Diff against target: 12 lines (+2/-2)
1 file modified
charmhelpers/contrib/openstack/templates/haproxy.cfg (+2/-2)
To merge this branch: bzr merge lp:~niedbalski/charm-helpers/fix-1697002
Reviewer Review Type Date Requested Status
Felipe Reyes (community) Approve
Review via email: mp+325413@code.launchpad.net

Description of the change

Modify the haproxy.cfg template to use the default
unix listen socket created by rsyslog on /e/rsyslog.d/49-haproxy.conf.

Fixes LP: #1697002.

To post a comment you must log in.
Revision history for this message
Felipe Reyes (freyes) wrote :

As we discussed over IRC, currently the logs are being sent syslog via localhost, but rsyslog's default configuration is not configured to send those messages to haproxy.log and they are sent to /dev/null . Even when it would be possible to configure rsyslog to send them to haproxy.log we would be deviating more from what the deb package does by default, so this change is more appropriate.

review: Approve
752. By Edward Hope-Morley

[thedac, r=hopem,ajkavanagh]

Update use of client.service_catalog with sessions

Due to a bug in keystone client when using keystoneauth1.sessions
keystoneclient.service_catalog is no longer populated automatically.

Manually setting client.auth_ref has the side effect of populating
the service_catalog and has become the undocumented but accepted
practice.

This change adds the manual setting of auth_ref and updates the query
arguments for url_for().

753. By Jorge Niedbalski

[niedbalski,r=freyes] Fix for LP: #1697002.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'charmhelpers/contrib/openstack/templates/haproxy.cfg'
2--- charmhelpers/contrib/openstack/templates/haproxy.cfg 2017-03-30 18:38:26 +0000
3+++ charmhelpers/contrib/openstack/templates/haproxy.cfg 2017-06-09 16:30:51 +0000
4@@ -1,6 +1,6 @@
5 global
6- log {{ local_host }} local0
7- log {{ local_host }} local1 notice
8+ log /var/lib/haproxy/dev/log local0
9+ log /var/lib/haproxy/dev/log local1 notice
10 maxconn 20000
11 user haproxy
12 group haproxy

Subscribers

People subscribed via source and target branches