~barryprice/charm-telegraf:prometheus-client-default-port-9103

Branch merges

Propose for merging

Merged into charm-telegraf:master at revision dd5ef38292c09d7d2530d0521bf25c31934e4152

Haw Loeung: Approve (+1) on 2021-02-18

Canonical IS Reviewers: Pending requested 2021-02-18

BootStack Reviewers: Pending requested 2021-02-18

Canonical IS Reviewers: Pending requested 2021-02-18

Diff: 19 lines (+2/-2)

1 file modified

src/tests/unit/test_telegraf.py (+2/-2)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

4301b93... by Barry Price on 2021-02-18

Fix conflicts

6296374... by Barry Price on 2021-02-18

Address review comments

ea55458... by Laurent Sesquès on 2021-01-27

when related with prometheus-client, if prometheus_output_port is unset, use :9103 by default instead of 9126

3c4a87e... by Laurent Sesquès on 2021-01-27

small fix: move flag changes to the right if stanza for config.changed

f550c15... by Laurent Sesquès on 2021-01-27

Split prometheus_client output config out of the main config file. Allow a prometheus-client relation to prometheus even if prometheus_output_port is set (and use that port in the relation).

bc10de9... by Thomas Cuthbert on 2021-01-26

Workaround for LP#1910974: telegraf haproxy input broken with Juju >= 2.8.7

Reviewed-on: https://code.launchpad.net/~tcuthbert/charm-telegraf/+git/charm-telegraf/+merge/396060
Reviewed-by: Xav Paice <email address hidden>
Reviewed-by: Peter Sabaini <email address hidden>
Reviewed-by: Haw Loeung <email address hidden>

9040361... by Paul Goins on 2021-01-15

Should use data_changed() instead of is_data_changed()

Reviewed-on: https://code.launchpad.net/~vultaire/charm-telegraf/+git/charm-telegraf/+merge/396357
Reviewed-by: Alvaro Uria <email address hidden>
Reviewed-by: Edin S <email address hidden>
Reviewed-by: Haw Loeung <email address hidden>

b7b8a51... by Paul Goins on 2021-01-14

Should use data_changed() instead of is_data_changed()

This is a bug fix for https://pad.lv/1911815.

631a400... by Thomas Cuthbert on 2021-01-13

Following Haws suggestion that we pass a listener address via relationship data and use it if present, else default to private-address as HAProxy stats listens on 0.0.0.0 by default

131c3b2... by Thomas Cuthbert on 2021-01-11

Workaround for LP#1910974: telegraf haproxy input broken with Juju >= 2.8.7