Code review comment for lp:~thumper/juju-core/ubuntu-profile-upgrade-step

Revision history for this message
Tim Penhey (thumper) wrote :

On 2014/03/21 04:56:00, wallyworld wrote:
> LGTM. I wonder if we should to grep for more than just
".juju-profile", but I
> guess that is all we need.

Thanks. The only thing that should have put .juju-profile into the
/home/ubuntu/.profile should have been us.

This is exactly what the cloud-init script does too.

https://codereview.appspot.com/78660043/

« Back to merge proposal