Code review comment for lp:~hopem/charm-helpers/lp1337266

Revision history for this message
Jorge Niedbalski (niedbalski) wrote :

LGTM, just a quick fix required; charmhelpers/contrib/openstack/context.py:724 is missed from tests coverage, and that line appears to be a relevant condition.

review: Needs Fixing

« Back to merge proposal