Code review comment for lp://staging/~afreiberger/landscape-client-charm/add-juju-annotations

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

First, this changes needs some test coverage (for both hooks and the externally-called `write_juju_annotations.py` script), and fixing broken tests (which might be mostly covered by my comments below).

review: Needs Fixing

« Back to merge proposal