charm-haproxy

Merge lp://qastaging/~mthaddon/charm-haproxy/doc-updates into lp://qastaging/charm-haproxy

  1. doc-updates
  2. Merge into trunk
Proposed by Tom Haddon
Status: Merged
Approved by: Tom Haddon
Approved revision: 144
Merged at revision: 145
Proposed branch: lp://qastaging/~mthaddon/charm-haproxy/doc-updates
Merge into: lp://qastaging/charm-haproxy
Diff against target: 39 lines (+15/-2)
1 file modified
README.md (+15/-2)
To merge this branch: bzr merge lp://qastaging/~mthaddon/charm-haproxy/doc-updates
Reviewer Review Type Date Requested Status
Loïc Gomez Approve
James Troup (community) Needs Information
Canonical IS Reviewers Pending
Review via email: mp+409521@code.qastaging.launchpad.net

Commit message

Update README to document monitoring via Telegraf, use 'relate' rather than 'add-relation' and remove personal pronoun

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
James Troup (elmo) wrote :

See comment inline

review: Needs Information
Revision history for this message
Tom Haddon (mthaddon) :
Revision history for this message
James Troup (elmo) wrote :

Tom Haddon <email address hidden> writes:

> I think that would need some config option updates to the charm to
> enable it per
> https://www.haproxy.com/blog/haproxy-exposes-a-prometheus-metrics-endpoint/. I've
> confirmed that the version of HAProxy installed by the charm on focal
> does at least support this:
>
> ```
> $ haproxy -vv | grep Prom
> Built with the Prometheus exporter as a service
> ```

I believe Alvaro and/or his team has done some work on this for the
hacluster subordinate FWIW.

--
James

Revision history for this message
Alvaro Uria (aluria) wrote :

Hi! The DevOps Centres work around haproxy monitoring is still WIP and focused on the OpenStack charms (which don't use charm-haproxy). The plan is for the OpenStack charms (API charms) to support Juju interfaces to relate to prometheus and Grafana. If possible, the OS charms implementation would support both the old charms (old LMA stack) and the new ones (new LMA stack from the Observability team).

As mentioned, we're using haproxy 2.0+ built-in support so we don't need an external exporter.

Revision history for this message
Loïc Gomez (kotodama) wrote :

+1

review: Approve
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision 145

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: