charm-telegraf:bug/1896467

Branch merges

Propose for merging

Merged into charm-telegraf:master at revision 42c814bf359e12064506ccde51088d8cdd68542d

Adam Dyess (community): Approve on 2020-10-02

Paul Goins (community): Approve on 2020-09-25

Benjamin Allot: Pending requested 2020-09-22

Diff: 168 lines (+42/-16)

4 files modified

src/reactive/telegraf.py (+14/-5)
src/tests/unit/test_mysql.py (+6/-3)
src/tests/unit/test_postgresql.py (+11/-8)
src/tests/unit/test_telegraf.py (+11/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

Related rock recipes

9ccd1a8... by Facundo Ciccioli on 2020-09-23

Files in /etc are usually owned by root; use a group to allow telegraf to access them

Allow root to edit the files to ease troubleshooting.

d8f8038... by Facundo Ciccioli on 2020-09-23

Factor out writing a telegraf-only readable file

e52205b... by Facundo Ciccioli on 2020-09-22

Write config files containing credentials as telegraf-only readable

Telegraf service runs as telegraf user, so root-only readable is not an
option. telegraf user is created with login disabled so still sudo
access is required to read those files.

ee1a92f... by Laurent Sesquès on 2020-08-28

wait for telegraf to be installed before configuring plugins

Reviewed-on: https://code.launchpad.net/~sajoupa/charm-telegraf/+git/telegraf-charm/+merge/388652
Reviewed-by: Alvaro Uria <email address hidden>
Reviewed-by: Tom Haddon <email address hidden>

f7faae9... by Laurent Sesquès on 2020-08-28

@when telegraf.installed: replace single quotes with double quotes

a38b182... by Laurent Sesquès on 2020-08-28

Merge branch 'plugins-when-telegraf-installed' of git+ssh://git.launchpad.net/~sajoupa/charm-telegraf into plugins-when-telegraf-installed

1a18488... by Laurent Sesquès on 2020-08-04

wait for telegraf to be installed before configuring plugins

efd9413... by Jose Guedez on 2020-08-26

Remove layer ntpmon

Reviewed-on: https://code.launchpad.net/~jfguedez/charm-telegraf/+git/charm-telegraf/+merge/389767
Reviewed-by: Jeremy Lounder <email address hidden>
Reviewed-by: Edin S <email address hidden>

c91d701... by Jose Guedez on 2020-08-25

Remove layer ntpmon

This is not actually being used by the charm
The ntp charm already provides monitoring via ntpmon

Fixes: LP1874245

9bdf73a... by Jose Guedez on 2020-08-21

OVS dump flows metric: query the list of bridges

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