Code review comment for charm-telegraf:bug/1896467

Revision history for this message
Facundo Ciccioli (fandanbango) wrote :

As per zzehring verbal comment, I've factored out the writing of the sensitive files.

Changed also ownership to root and made group readable and root writable as per ballot's suggestion (I do agree with that that's the custom for /etc).

Unit tests: https://paste.ubuntu.com/p/zjQCY6XP3R/
Functional tests: https://paste.ubuntu.com/p/Hw25x5N2Sp/

« Back to merge proposal