Code review comment for lp:~johnsca/charms/trusty/cloudfoundry/catchup

Revision history for this message
Benjamin Saller (bcsaller) wrote :

These changes LGTM. Its nice that you found the dangling reference issue with placement and the clean up on interface types, like http->login.

I'd like to hold off landing this until we've identified the remaining issue with CATs, even if we can't fix that one remaining test run today I'd like some clarity as to why. If it works on a fresh run this morning its most likely a GC issue and we'll need to address that separately.

Thanks

review: Approve

« Back to merge proposal