The charm stopped working without 'certificates' relation
Bug #2008436 reported by
Nikolay Vinogradov
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Telegraf Charm |
Fix Released
|
High
|
Sudeep Bhandari | ||
Bug Description
It seems that starting from the commit [1] telegraf charm requires TLS relations to work, see [2]. Because of that the line [3] is never called and therefore prometheus2 charm skips the target registration as per [4].
The revision of telegraf charm that still works is 62. Starting from rev. 63 the new behavior has been introduced. If this change of behavior was intended we need to update the charm docs.
[1] https:/
[2] https:/
[3] https:/
[4] https:/
Related branches
~sudeephb/charm-telegraf:bug/2008436
- Martin Kalcok (community): Approve
- 🤖 prod-jenkaas-bootstack (community): Approve (continuous-integration)
- Robert Gildein: Approve
- Eric Chen: Approve
- Tianqi Xiao (community): Needs Fixing
- BootStack Reviewers: Pending requested
- Peter Jose De Sousa: Pending requested
-
Diff: 110 lines (+45/-6)3 files modifiedsrc/files/telegraf_exec_metrics.py (+0/-1)
src/reactive/telegraf.py (+4/-4)
src/tests/unit/test_telegraf.py (+41/-1)
| summary: |
- The charm stopped working with 'certificates' relation + The charm stopped working without 'certificates' relation |
| Changed in charm-telegraf: | |
| importance: | Undecided → High |
| tags: | added: bseng-944 |
| Changed in charm-telegraf: | |
| status: | New → Triaged |
| description: | updated |
| Changed in charm-telegraf: | |
| assignee: | nobody → Sudeep Bhandari (sudeephb) |
Revision history for this message
| Nobuto Murata (nobuto) wrote | #1 |
| Changed in charm-telegraf: | |
| status: | Triaged → Fix Committed |
Revision history for this message
| Eric Chen (eric-chen) wrote | #2 |
| Changed in charm-telegraf: | |
| milestone: | none → 23.01 |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Subscribing ~field-high mainly for tracking purposes. I know it's prioritized accordingly by now.