Code review comment for lp:~ericsnowcurrently/fake-juju/allow-external-connections

Revision history for this message
Chad Smith (chad.smith) wrote :

+1 on this eric. nice1!

Tested before and after your patch

JUJU_VERSION=2.0.0 make
./2.0.0/2.0.0 bootstrap myctrl
juju-2.0 deploy ubuntu

# fails without your fix on access to external websites, succeeds with your patch

review: Approve

« Back to merge proposal