telegraf haproxy input broken with Juju >= 2.8.7
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Canonical Juju |
Fix Released
|
High
|
Joseph Phillips | ||
Content Cache Charm |
Fix Released
|
Low
|
Thomas Cuthbert | ||
Telegraf Charm |
Fix Released
|
Low
|
Thomas Cuthbert |
Bug Description
# Code that configures the haproxy telegraf plugin.
## notice that it hasn't been changed since 2016.
29b50c58 src/reactive/
^a7836fc reactive/
^a7836fc reactive/
# The problem
That code would work if we called unit-get public-address, see below it returns the fqdn which would satisfy line 914
juju run --unit content-cache-1ss/1 "relation-get -r haproxy-
ingress-address: darkbowser.
private-address: darkbowser.
juju run --unit telegraf-1ss/5 "relation-get -r haproxy:34 - content-
enabled: "True"
ingress-address: darkbowser.
port: "10000"
private-address: darkbowser.
user: haproxy
juju run --unit telegraf-1ss/5 "unit-get private-address"
91.189.91.43
(mojo-prod-
91.189.91.43
(mojo-prod-
(mojo-prod-
darkbowser.
Related branches
- Haw Loeung: Approve (+1)
- Canonical IS Reviewers: Pending requested
-
Diff: 13 lines (+1/-1)1 file modifiedreactive/content_cache.py (+1/-1)
- Haw Loeung: Approve (+1)
- Canonical IS Reviewers: Pending requested
-
Diff: 12 lines (+1/-0)1 file modifiedreactive/content_cache.py (+1/-0)
- Xav Paice (community): Approve
- Peter Sabaini (community): Approve
- Haw Loeung: Approve (+1)
-
Diff: 35 lines (+10/-4)2 files modifiedsrc/reactive/telegraf.py (+1/-3)
src/tests/unit/test_telegraf.py (+9/-1)
Changed in juju: | |
status: | New → In Progress |
importance: | Undecided → High |
assignee: | nobody → Joseph Phillips (manadart) |
milestone: | none → 2.8.8 |
Changed in juju: | |
status: | In Progress → Fix Committed |
Changed in charm-telegraf: | |
assignee: | nobody → Thomas Cuthbert (tcuthbert) |
status: | New → Fix Committed |
Changed in content-cache-charm: | |
assignee: | nobody → Thomas Cuthbert (tcuthbert) |
importance: | Undecided → Low |
Changed in charm-telegraf: | |
importance: | Undecided → Low |
Changed in content-cache-charm: | |
status: | Invalid → Fix Committed |
Changed in charm-telegraf: | |
status: | Fix Committed → Fix Released |
milestone: | none → 21.01 |
Changed in content-cache-charm: | |
status: | Fix Committed → Fix Released |
Changed in juju: | |
status: | Fix Committed → Fix Released |
Proposed workaround https:/ /code.launchpad .net/~tcuthbert /charm- telegraf/ +git/charm- telegraf/ +merge/ 396060