unit tests need privileges and integration tests use juju-deployer (via amulet)
Bug #1877284 reported by
Alvaro Uria
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Telegraf Charm |
Fix Released
|
Medium
|
Alvaro Uria | ||
Bug Description
juju-deployer is discontinued in favor of juju and zaza. It would be convenient to rewrite (and possibly expand) the current func tests.
Unit tests need sudo provileges due to calls that require systemd privileges
Related branches
~canonical-is-bootstack/charm-telegraf/+git/telegraf-charm:bug/1877284-unit
Merged
into
~canonical-is-bootstack/charm-telegraf/+git/telegraf-charm:bug/1877284-func
at
revision d06fdacc640e919014fee9a6964c0278f0fb34d8
- Paul Goins: Approve
- Zachary Zehring (community): Approve
-
Diff: 36 lines (+5/-2)2 files modifiedreactive/telegraf.py (+1/-1)
unit_tests/test_telegraf.py (+4/-1)
~canonical-is-bootstack/charm-telegraf/+git/telegraf-charm:bug/1877284-func
- Paul Goins: Approve
- Zachary Zehring (community): Approve
-
Diff: 561 lines (+214/-116)15 files modifiedMakefile (+40/-14)
dev/null (+0/-63)
reactive/telegraf.py (+2/-2)
requirements.txt (+2/-0)
tests/bundles/bionic-mysql.yaml (+14/-0)
tests/bundles/bionic-postgres.yaml (+14/-0)
tests/bundles/overlays/local-charm-overlay.yaml.j2 (+3/-0)
tests/bundles/xenial-mysql.yaml (+14/-0)
tests/bundles/xenial-postgres.yaml (+14/-0)
tests/requirements.txt (+2/-0)
tests/test_telegraf.py (+56/-0)
tests/tests.yaml (+19/-13)
tox.ini (+24/-23)
unit_tests/requirements.txt (+6/-0)
unit_tests/test_telegraf.py (+4/-1)
| Changed in charm-telegraf: | |
| status: | New → Triaged |
| importance: | Undecided → Medium |
| Changed in charm-telegraf: | |
| milestone: | none → 20.05 |
| Changed in charm-telegraf: | |
| assignee: | nobody → Alvaro Uria (aluria) |
| status: | Triaged → In Progress |
| Changed in charm-telegraf: | |
| status: | In Progress → Fix Committed |
| Changed in charm-telegraf: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
