Code review comment for lp:~exsdev/landscape-client-charm/juju-app-annotator

Revision history for this message
Simon Poirier (simpoir) wrote :

The first thing which comes to mind is we should do without client restarts (and the sudoers bit).
Annotations already get monitored at regular intervals (5min) and pushed with other monitor exchanges (as frequently as configured). There is no need to force restart services.

Bonus is that preserves the throttling of server-bound exchanges.
Extra bonus, the code would be way simpler.

« Back to merge proposal