Telegraf Charm

Code review comment for ~rgildein/charm-telegraf:bug1988312

  1. Git
  2. lp:~rgildein/charm-telegraf
  3. bug1988312
  4. Merge into master
Revision history for this message
Robert Gildein (rgildein) wrote :

This is covered in unit and functional tests.
Unit and functional tests were broken after the MP [1].

The changes from MP [1] was reverted by MP [2] and I
adding them back but improved.

However, I am not sure why and if we need this, as I have
not been able to reproduce the issue described and fixed
by Haw Loeung in MP [1].

---
[1]: https://code.launchpad.net/~hloeung/charm-telegraf/+git/charm-telegraf/+merge/428004
[2]: https://code.launchpad.net/~jneo8/charm-telegraf/+git/charm-telegraf/+merge/429347

« Back to merge proposal