~xavpaice/charm-telegraf:bug/lp1913489

Branch merges

Propose for merging

Merged into charm-telegraf:master at revision dec0633eb0134f4222b70c4a3e4f8375baac1f61

Joe Guo (community): Approve on 2021-02-09

Canonical IS Reviewers: Pending requested 2021-01-28

BootStack Reviewers: Pending requested 2021-01-28

Diff: 43 lines (+12/-5)

2 files modified

src/reactive/telegraf.py (+6/-3)
src/tests/functional/tests/bundles/bionic-monitoring.yaml (+6/-2)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

4fc265c... by Xav Paice on 2021-01-28

Correct alert rule template names

Update the template names for alertmanager rule exports. Add the
relation to functests to exercise the part of the code.

Fixes-bug: LP1913489

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

1c088cc... by Benjamin Allot on 2020-11-26

Change the default collection_jitter value to "5s"

Reviewed-on: https://code.launchpad.net/~ballot/charm-telegraf/+git/telegraf-charm/+merge/394475
Reviewed-by: James Troup <email address hidden>

e0c262d... by Benjamin Allot on 2020-11-25

Change the default collection_jitter and flush_jitter value to "5s"

See
https://github.com/influxdata/telegraf/issues/3465#issuecomment-370510543
for the explanation.
We want to prevent all the telegraf plugins to pull their metrics
exactly at the same time.
This is even more important on a server hosting a lot of VMs with
telegraf deployed inside.

974b863... by Laurent Sesquès on 2020-11-08

make sure that extra_plugins config gets written after changing install_method

Reviewed-on: https://code.launchpad.net/~sajoupa/charm-telegraf/+git/telegraf-charm/+merge/393386
Reviewed-by: James Hebden <email address hidden>
Reviewed-by: Haw Loeung <email address hidden>

c9b21cd... by Laurent Sesquès on 2020-11-05

make sure that extra_plugins config gets written after changing install_method