~sajoupa/charm-k8s-telegraf:master

Branch merges

Propose for merging

Merged into charm-k8s-telegraf:master at revision 9dafe3ca0643f5e681ce704d5463ffcb1f5998b6

Tom Haddon: Approve on 2020-11-04

Facundo Batista (community): Approve on 2020-11-04

Canonical IS Reviewers: Pending requested 2020-10-07

Diff: 781 lines (+599/-66)

12 files modified

.flake8 (+1/-1)
.jujuignore (+1/-0)
Dockerfile (+38/-0)
Makefile (+11/-1)
README.md (+13/-5)
config.yaml (+72/-3)
dev/null (+0/-0)
entrypoint.sh (+8/-0)
metadata.yaml (+4/-3)
src/charm.py (+154/-21)
tests/unit/scenario.py (+184/-0)
tests/unit/test_charm.py (+113/-32)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

e64c19e... by Laurent Sesquès on 2020-11-04

Let the logger build its messages ; small details fixed following charmcrafters review

9cf0505... by Laurent Sesquès on 2020-10-07

small code simplification

f9f647a... by Laurent Sesquès on 2020-10-07

further quote fixes

18c2911... by Laurent Sesquès on 2020-10-07

scenario.py: simplify quotes / double-quotes

7fb2b14... by Laurent Sesquès on 2020-10-07

readme: fix deploy command example

4faff1e... by Laurent Sesquès on 2020-10-07

config.yaml: small wording fixes

8bcf9df... by Laurent Sesquès on 2020-10-07

update README: fix the command for local test and usage

b200b4f... by Laurent Sesquès on 2020-10-07

dockerfile: sort packages alphabetically

eb73d07... by Laurent Sesquès on 2020-10-02

update config.yaml to document required settings

0451a76... by Laurent Sesquès on 2020-10-02

actually, we shouldn't set 'service: omit' (otherwise we never get anything exposed), so remove it and make open_ports a mandatory config item