Code review comment for lp:~natefinch/juju-ci-tools/logrot

Revision history for this message
Nate Finch (natefinch) wrote :

The test intentionally uses a charm, because it represents a real world situation where the charm is the thing outputting a lot of logging. I don't know that I could reasonably create a juju run command that would do the same thing. This seemed like the most easy to understand way to get the job done. I could use juju shh to get the size of the logs, but since I already had the charm and actions, it seemed easy enough to just make that another action.

« Back to merge proposal