Code review comment for ~sajoupa/charm-telegraf:snap-support

Revision history for this message
Laurent Sesquès (sajoupa) wrote :

> Hi Laurent,
>
> Did you run `make test` as root?
> I just ran `make unittest` as non-root, it will pop up dialogues to ask for
> authentication.
> You may need to add this line in unit_tests/test_telegraf.py::setup:
>
> monkeypatch.setattr(telegraf.host, 'service_pause', lambda svc: None)

Hi Joe,

I've always run it successfully as a normal user.
But your suggestion makes sense, I added it.

« Back to merge proposal