~hemanth-n/charm-telegraf:lp1903499

Branch merges

Propose for merging

Superseded for merging into charm-telegraf:master

Peter Sabaini (community): Needs Fixing on 2021-01-18

Benjamin Allot: Approve on 2021-01-04

BootStack Reviewers: Pending requested 2020-12-06

Diff: 87 lines (+48/-4)

2 files modified

src/files/telegraf_exec_metrics.py (+30/-1)
src/tests/unit/test_telegraf.py (+18/-3)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

8cb08fd... by Hemanth Nakkina on 2020-12-06

Fix OvsDumpFlowsCmdMetric to use correct openflow protocol

Currently ovs-ofctl dump the flows without specifying any
openflow protocol which leads to openflow version negotiation
warning messages in ovs-vswitchd daemon.

The fix gets the supported openflow protocol versions and use
the latest one for dump flows command.

Closes: #1903499

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

42c814b... by Facundo Ciccioli on 2020-10-13

Write config files containing credentials as telegraf-only readable

Reviewed-on: https://code.launchpad.net/~llama-charmers/charm-telegraf/+git/charm-telegraf/+merge/391124
Reviewed-by: Alvaro Uria <email address hidden>
Reviewed-by: Paul Goins <email address hidden>
Reviewed-by: Adam Dyess <email address hidden>

7ab8597... by Laurent Sesquès on 2020-10-13

config.yaml: use '>' for wrappable text instead of '|' for preserving whitespace

Reviewed-on: https://code.launchpad.net/~sajoupa/charm-telegraf/+git/telegraf-charm/+merge/388640
Reviewed-by: Alvaro Uria <email address hidden>
Reviewed-by: Haw Loeung <email address hidden>
Reviewed-by: Stuart Bishop <email address hidden>

32c00bb... by Paul Goins on 2020-10-13

Changed license to Apache 2.

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

4ef6c03... by Laurent Sesquès on 2020-09-24

add the internal input plugin to base inputs, with memstats collection. Fixes lp:1895661.

Reviewed-on: https://code.launchpad.net/~sajoupa/charm-telegraf/+git/telegraf-charm/+merge/391131
Reviewed-by: Paul Goins <email address hidden>
Reviewed-by: Junien Fridrick <email address hidden>
Reviewed-by: Benjamin Allot <email address hidden>

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.